[PATCH 01/14] staging: brcm80211: use wait queues instead of semaphores in wl_cfg80211.c
Johannes Berg
johannes at sipsolutions.net
Fri Aug 12 18:59:12 UTC 2011
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
More information about the devel
mailing list