RESOLVED FIXED Bug 275309
Code style misses an entry about a space in empty braces, check-webkit-style checks
https://bugs.webkit.org/show_bug.cgi?id=275309
Summary Code style misses an entry about a space in empty braces, check-webkit-style ...
Kohei Asano
Reported 2024-06-10 01:29:45 PDT
check-webkit-style checks a space in every empty braces. We can check this for empty braces, i.e. in https://github.com/WebKit/WebKit/commit/736ea9e03ccb296098339b8766404e3c34d6ffe3. void f() {} struct Unit {}; union U2 {}; class U3 {}; enum U4 {}; int x {}; auto a = [] {}; $ python ./Tools/Scripts/check-webkit-style --git-index ERROR: Tools/MiniBrowser/win/MainWindow.cpp:410: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:411: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:412: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:413: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:414: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:415: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:416: Missing space inside { }. [whitespace/braces] [5] ERROR: Tools/MiniBrowser/win/MainWindow.cpp:424: Missing space inside { }. [whitespace/braces] [5] but Code Style Guidelines misses about this rule. https://webkit.org/code-style-guidelines/#braces
Attachments
Kohei Asano
Comment 1 2024-06-10 02:06:34 PDT
Radar WebKit Bug Importer
Comment 2 2024-06-17 01:30:18 PDT
EWS
Comment 3 2024-06-19 11:11:27 PDT
Committed 280182@main (528d5d1f396a): <https://commits.webkit.org/280182@main> Reviewed commits have been landed. Closing PR #29668 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.