[Webkit-unassigned] [Bug 27497] New: Feature request: cpplint should check for braces - rule 2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 21 07:26:46 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=27497
Summary: Feature request: cpplint should check for braces -
rule 2
Product: WebKit
Version: 528+ (Nightly build)
Platform: PC
OS/Version: All
Status: UNCONFIRMED
Severity: Minor
Priority: P2
Component: Tools / Tests
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: luciano.wolf at openbossa.org
CC: kenneth.christiansen at openbossa.org
According to Braces rule 2 of the Webkit coding style guidelines[1]:
"Other braces: place the open brace on the line preceding the code block; place
the close brace on its own line."
cpplint doesn't complain about non-compliance with this rule when using the Qt
foreach macro, as illustrated below
WRONG CODING STYLE:
foreach (AbstractMetaFunction *func, lst)
{
qDebug() << "*" << func->ownerClass()->name()
<< "Private: " << func->isPrivate()
<< "Empty: " << func->isEmptyFunction();
}
[1]
http://webkit.org/coding/coding-style.html
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list