Changeset 53387 in webkit
- Timestamp:
- Jan 17, 2010 7:28:09 PM (14 years ago)
- Location:
- trunk/WebKitTools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebKitTools/ChangeLog
r53386 r53387 1 2010-01-17 Adam Barth <abarth@webkit.org> 2 3 Reviewed by Darin Adler. 4 5 style-queue complains about one-line macros that include multiple statements 6 https://bugs.webkit.org/show_bug.cgi?id=33173 7 8 Add an exception for multiple statements on a line that starts a macro. 9 10 * Scripts/webkitpy/style/cpp_style.py: 11 * Scripts/webkitpy/style/cpp_style_unittest.py: 12 1 13 2010-01-17 Adam Barth <abarth@webkit.org> 2 14 -
trunk/WebKitTools/Scripts/webkitpy/style/cpp_style.py
r53386 r53387 1939 1939 and not ((cleansed_line.find('case ') != -1 1940 1940 or cleansed_line.find('default:') != -1) 1941 and cleansed_line.find('break;') != -1)): 1941 and cleansed_line.find('break;') != -1) 1942 and not cleansed_line.startswith('#define ')): 1942 1943 error(filename, line_number, 'whitespace/newline', 4, 1943 1944 'More than one command on the same line') -
trunk/WebKitTools/Scripts/webkitpy/style/cpp_style_unittest.py
r53386 r53387 3019 3019 '}\n', 3020 3020 '') 3021 3021 self.assert_multi_line_lint( 3022 '#define TEST_ASSERT(expression) do { if (!(expression)) { TestsController::shared().testFailed(__FILE__, __LINE__, #expression); return; } } while (0)\n', 3023 '') 3022 3024 self.assert_multi_line_lint( 3023 3025 'if (condition) {\n'
Note: See TracChangeset
for help on using the changeset viewer.