12 Jan
2011
12 Jan
'11
2:10 p.m.
On 11.01.2011 3:31, Mike Frysinger wrote:
it isnt a problem to have the parens, and it keeps things sane if someone does something like: #define CONFIG_SYS_BD_INFO_ADDR SOME_DEFINE + 0x1000
This would be a violation of basic rules of defensive coding.
Please drop these parens.
i dont see how these two statements are compatible. defensive coding would mean i keep the parens.
I think Wolfgang meant that defining a macro without parens would be a violation -- which should be fixed, not worked around.
-mike
WBR, Sergei