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"