svn commit: r240549 - head/sys/arm/tegra

[Chris Rees]

On 16 September 2012 18:41, Ian Lepore <freebsd at damnhippie.dyndns.org> wrote:
> On Sun, 2012-09-16 at 18:30 +0100, Chris Rees wrote:
>> On 16 Sep 2012 16:19, "David Chisnall" <theraven at freebsd.org> wrote:
>> >
>> > On 16 Sep 2012, at 10:37, Chris Rees wrote:
>> >
>> > > Actually:
>> > >
>> > > for (;;)
>> > >        ;
>> >
>> > This form will generate a compiler warning, because it looks like a
>> missing loop body.  You can silence the warning by this form:
>> >
>> > for (;;) {}
>> >
>> > This makes it clear that you have an explicit body containing no
>> statements.
>>
>> Please fix the style man page then, and make sure bde approves!
>
> One of the examples allows
>
>     for (;;)
>         stmt;
>
> Using "continue;" as the statement is consistant with that.

Ok, so with approval from a docs/src guy I'd like to commit [1] to
update the style page to use continue; instead of a lone ;

Anyone like to approve/commit?

Chris

[1] http://www.bayofrum.net/~crees/patches/style-empty-loop.diff