On Fri, Dec 31, 2010 at 12:26:00PM -0800, Joe Perches wrote: > On Fri, 2010-12-31 at 20:28 +0100, Javier Martinez Canillas wrote: > if (!(!memcmp(redundant+0x0D, EccBuf, 3) && > !memcmp(redundant+0x08, EccBuf+0x03, 3))) ^^^ ^^^ Also we should add spaces around the '+' character. But this is dead code anyway so let's just merge it. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html