Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 16 Sep 2012 10:37:56 +0100
From:      Chris Rees <crees@FreeBSD.org>
To:        Alexey Dokuchaev <danfe@freebsd.org>
Cc:        Andrew Turner <andrew@freebsd.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r240549 - head/sys/arm/tegra
Message-ID:  <CADLo83-5zQBdf3ndBXO4qbkXJgaFPxGfE86LSDGqJ4g3k_4gkw@mail.gmail.com>
In-Reply-To: <20120916092818.GA81952@FreeBSD.org>
References:  <201209160755.q8G7tnMH036332@svn.freebsd.org> <20120916092818.GA81952@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 16 September 2012 10:28, Alexey Dokuchaev <danfe@freebsd.org> wrote:
> On Sun, Sep 16, 2012 at 07:55:49AM +0000, Andrew Turner wrote:
>> New Revision: 240549
>> URL: http://svn.freebsd.org/changeset/base/240549
>>
>> Log:
>>   The cpu_reset function is noreturn, make sure this is true on Tegra 2.
>>   While here fix a typo.
>>
>> +
>> +     while(1);
>
> I thought preferred and more style(9) compliant way to code this is:
>
>         for (;;)
>                 continue;

Actually:

for (;;)
        ;

Chris



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo83-5zQBdf3ndBXO4qbkXJgaFPxGfE86LSDGqJ4g3k_4gkw>