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

Reply via email to