On 11 Jun 2010, at 02:46, Daniel O'Connor wrote:
>
> On 11/06/2010, at 6:04, Rui Paulo wrote:
>> Log:
>> MFC r208644:
>> Due to the way HALDEBUG() is defined, we need to add curly brackets
>> when using it as a sole if clause instruction.
>> While there, fix 'const static' typo.
>
> Why not fix HALDEBUG? ie wrap it in do { } while (0)
>
> Although that said when I looked at HALDEBUG on 8.0 it was defined as a
> function so it is a non issue there.
>
> HAL_DEBUG is different but I can't actually find where that is defined.
There's actually no problem in the code. This was to shut up a compiler warning
on NetBSD. I probably should've said that in the commit log.
Regards,
--
Rui Paulo
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
