On Fri, 2011-08-12 at 20:55 +0200, Arend van Spriel wrote: > >>> + } while ((e = brcmf_deq_event(cfg_priv))); > >> if (((((care_coding_style))))) > >> fix(); > >> > >> :-) > >> > > I assume you refer to the indents above. Not sure what exactly happened > > there, but the patch email as I received it looks. Could it be a mail > > reader issue? > > > > Gr. AvS > > A colleague pointed out you probably meant the while condition. The > extra braces may be there because an assignment is done, but in this > case it does not make much sense. Doesn't gcc warn if you don't have two parentheses? johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
