svn commit: r209011 - stable/8/sys/dev/ath/ath_hal/ar5211

Daniel O'Connor doconnor at gsoft.com.au
Fri Jun 11 02:24:10 UTC 2010


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.

--
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
  -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C








More information about the svn-src-stable-8 mailing list