WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
33814
WebKitTools/Scripts/check-webkit-style gives false positives in single-line functions.
https://bugs.webkit.org/show_bug.cgi?id=33814
Summary
WebKitTools/Scripts/check-webkit-style gives false positives in single-line f...
Gavin Barraclough
Reported
2010-01-18 15:35:51 PST
Some examples in:
https://bugs.webkit.org/show_bug.cgi?id=33731
We allow single-line functions declared in the header (within the class definition) for fairly trivial accessors. Not a big issue, but would be awesome if the script was aware of these cases.
Attachments
Patch
(4.94 KB, patch)
2010-07-28 08:57 PDT
,
Kenichi Ishibashi
no flags
Details
Formatted Diff
Diff
Hamaji-san, Thank you for your quick review. I put semicolons after brackets.
(4.94 KB, patch)
2010-07-28 17:43 PDT
,
Kenichi Ishibashi
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Kenichi Ishibashi
Comment 1
2010-07-28 08:57:59 PDT
Created
attachment 62831
[details]
Patch
Shinichiro Hamaji
Comment 2
2010-07-28 12:30:57 PDT
Comment on
attachment 62831
[details]
Patch Looks great. I'll make the following changes when I'll land this patch. WebKitTools/Scripts/webkitpy/style/checkers/cpp_unittest.py:2600 + '}\n', I'd put a semicolon so this code is valid as C++. WebKitTools/Scripts/webkitpy/style/checkers/cpp_unittest.py:2608 + '}\n', ditto
Gavin Barraclough
Comment 3
2010-07-28 16:23:55 PDT
Yay! this bug has been annoying me so much I've been considering learning python! This is constantly biting me, I'm so happy to see this fixed, thank you!
Kenichi Ishibashi
Comment 4
2010-07-28 17:43:20 PDT
Created
attachment 62901
[details]
Hamaji-san, Thank you for your quick review. I put semicolons after brackets.
Shinichiro Hamaji
Comment 5
2010-07-28 18:05:22 PDT
Comment on
attachment 62901
[details]
Hamaji-san, Thank you for your quick review. I put semicolons after brackets. Thanks for your update!
WebKit Commit Bot
Comment 6
2010-07-28 20:36:52 PDT
Comment on
attachment 62901
[details]
Hamaji-san, Thank you for your quick review. I put semicolons after brackets. Clearing flags on attachment: 62901 Committed
r64263
: <
http://trac.webkit.org/changeset/64263
>
WebKit Commit Bot
Comment 7
2010-07-28 20:36:57 PDT
All reviewed patches have been landed. Closing bug.
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