John Baldwin wrote:
> Also, one final note about using do { } while(0). If you actually read
> style(9), you will see that you are supposed to use it for compound statement
s,
> not just for any macro that happens to be more than one line long. If the
> macro's body is a single statement, it doesn't need the do { } while (0) bit.
It was there so that the macro didn't have a "value". Since you're
changing this so they all have a meaningful return (eg: TUNABLE_STR_FETCH()
did not before) then removing this makes sense.
Cheers,
-Peter
--
Peter Wemm - [EMAIL PROTECTED]; [EMAIL PROTECTED]; [EMAIL PROTECTED]
"All of this is for nothing if we don't go to the stars" - JMS/B5
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message
