On Wed, 23 Aug 2006 09:05:30 +0200, Johannes Berg wrote:
> On Tue, 2006-08-22 at 21:00 +0200, Jiri Benc wrote:
> > When you're touching this, could you #ifdef out wpa_test when
> > CONFIG_HOSTAPD_WPA_TESTING is not defined? This could be a part of this
> > patch.
> 
> I thought about it but deemed it too ugly. And wouldn't the compiler
> optimize it away anyway? Thing is, it's used in tests, and having
> #ifdefs within if (...) tests is hugely ugly.

#ifdefs within if tests are ugly, I agree. I thought about restructuring
these ifs. Okay, we will go with your patch and address this later.

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to