WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
150440
Style checker didn't catch single-line if statement
https://bugs.webkit.org/show_bug.cgi?id=150440
Summary
Style checker didn't catch single-line if statement
Antoine Quint
Reported
2015-10-22 02:05:05 PDT
My patch at
https://bugs.webkit.org/attachment.cgi?id=263680&action=review
had the following code: } else if (name == SVGNames::onendAttr) { setAttributeEventListener(eventNames().endEventEvent, name, value); } else SVGElement::parseAttribute(name, value); I added the else if which should not have used braces, but the style bot didn't report a failure. This sounds like
https://bugs.webkit.org/show_bug.cgi?id=28164
but that bug is marked as fixed.
Attachments
Add attachment
proposed patch, testcase, etc.
Sam Sneddon [:gsnedders]
Comment 1
2024-01-30 04:33:47 PST
The test is still there:
https://github.com/WebKit/WebKit/blob/b28edead7c0c1615e7f7f934c475245e61742fe7/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py#L5433-L5438
So presumably something specific in this case (the else if?) making it not apply.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug