On Saturday 03 June 2006 18:12, Chen, Kenneth W wrote:
> Ingo Molnar wrote on Saturday, June 03, 2006 12:58 AM
>
> > * Con Kolivas <[email protected]> wrote:
> > > Could we make this neater with extra braces such as:
> > >
> > > for_each_domain(this_cpu, tmp) {
> > > if (tmp->flags & SD_SHARE_CPUPOWER) {
> > > sd = tmp;
> > > break;
> > > }
> > > }
> > >
> > > and same for the other uses of for_each ? I know it's redundant but
> > > it's neater IMO when there are multiple lines of code below it.
> >
> > yep, that's the preferred style when there are multiple lines below a
> > loop.
>
> OK, thanks for the tips. Here is an incremental coding-style fix:
Great!
Thanks Chris, Nick and Ken for your input.
Signed-off-by: Con Kolivas <[email protected]>
for both
--
-ck
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
