On Tue, 2011-07-26 at 11:26 +0300, Dan Carpenter wrote:
> On Fri, Jul 22, 2011 at 08:57:00PM +0200, Arend van Spriel wrote:
> > @@ -3345,7 +3345,7 @@ void brcms_c_init(struct brcms_c_info *wlc)
> > FOREACH_BSS(wlc, i, bsscfg)
> > brcms_c_set_mac(bsscfg);
> > brcms_c_set_bssid(bsscfg);
> > - END_FOREACH_BSS
> > + END_FOREACH_BSS()
> These would look nicer with a semicolon on the end.
> END_FOREACH_BSS();
> That looks like c.
> I quite liked my way with the extra { } chars.
> FOREACH_BSS(wlc, i, bsscfg) {
> brcms_c_set_mac(bsscfg);
> brcms_c_set_bssid(bsscfg);
> } END_FOREACH_BSS();
> Emacs gets the indenting wrong without the braces. We use that
> style of for each loop in the Sparse project.
I too think it's better with braces and it's not
just emacs. Anything that looks at indentation gets
this wrong without the braces.
> But I'm fairly sick of this macro so at this point, I'm fine with
> whatever. :P
I'm with Dan, but I'm not sick like he is...
--
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
