According to http://webkit.org/coding/coding-style.html, rule #5 (and the
check-webkit-style script), if-statements should look like this:
if (a
&& b)
and not like this:
if (a &&
b)
However,
find WebCore/ -name *.cpp | xargs grep -E '^.*(&&|\|\|)\s*$' | wc -l = 925
incorrect uses
find WebCore/ -name *.h | xargs grep -E '^.*(&&|\|\|)\s*$' | wc -l = 81
incorrect uses
find WebCore/ -name *.cpp | xargs grep -E '^\s*(&&|\|\|).*$' | wc -l = 0
correct uses
find WebCore/ -name *.h | xargs grep -E '^\s*(&&|\|\|).*$' | wc -l = 0
correct uses
Should the rule be changed? Sorry if this was discussed already and I missed
the thread.
Thanks,
Dumi
_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
