Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/crypto/rijndael



On 5/14/11 12:31 PM, Christos Zoulas wrote:
On May 14, 12:29pm, jmmv%julipedia.org@localhost (Julio Merino) wrote:
-- Subject: Re: CVS commit: src/sys/crypto/rijndael

|>  Declare for-loop control variable outside of the for statement to prevent
|>  a warning and therefore fix the build.
|
| Ah!  I just saw your warns=4 change.  I presume my 'fix' goes against
| it, right?

Yes, you should revert it. Kernel code is supposed to use c99 features.

OK; I'll do a test build before submitting the revert. I just realized that my tree already had your change when I committed this, yet the build still failed.

Home | Main Index | Thread Index | Old Index