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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot