On Wed, Mar 12, 2014 at 14:53 +0100, Linus Walleij wrote: > > On Fri, Mar 7, 2014 at 12:25 AM, Gary Servin <garyser...@gmail.com> wrote: > > > This coding style issue was detected using the checkpatch.pl script > > > > Signed-off-by: Gary Servin <garyser...@gmail.com> > > Sometimes the compiler is just too forgiving :-/ > Thanks a lot for fixing this, patch applied.
I think the code had no syntax or language issue before. The cleanup really was about style exclusively (which still is a good thing). 'sizeof' is an operator, very much like a unary minus or unary ampersand which neither require parentheses. So either of "sizeof(x)" as well as "sizeof x" are legal with regard to the C language. It's just that the community prefers "sizeof(x)" for the improved readability. virtually yours Gerhard Sittig -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr. 5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
