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

2010-06-11 Thread Rui Paulo
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

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

2010-06-10 Thread Daniel O'Connor
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

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

2010-06-10 Thread Rui Paulo
Author: rpaulo Date: Thu Jun 10 20:34:22 2010 New Revision: 209011 URL: http://svn.freebsd.org/changeset/base/209011 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.