From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Mon, 10 Jan 2011 19:31:22 -0500 Subject: [U-Boot] [PATCH 15/28] Blackfin: fix bd_t handling In-Reply-To: <20110110222823.776D4150A4A@gemini.denx.de> References: <1293425300-27644-1-git-send-email-vapier@gentoo.org> <201012271142.17334.vapier@gentoo.org> <20110110222823.776D4150A4A@gemini.denx.de> Message-ID: <201101101931.23412.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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