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"

Reply via email to