[U-Boot] [PATCH 15/28] Blackfin: fix bd_t handling

Mike Frysinger vapier at gentoo.org
Tue Jan 11 01:31:22 CET 2011


On Monday, January 10, 2011 17:28:23 Wolfgang Denk wrote:
> 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.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110110/0a5bef41/attachment.pgp 


More information about the U-Boot mailing list