Changeset 70600 in webkit
- Timestamp:
- Oct 26, 2010 7:01:07 PM (13 years ago)
- Location:
- trunk/WebKitTools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebKitTools/ChangeLog
r70590 r70600 1 2010-10-26 Ademar de Souza Reis Jr. <ademar.reis@openbossa.org> 2 3 Reviewed by Dumitru Daniliuc. 4 5 check-webkit-style fails on operator+=, operator-=, ... methods 6 https://bugs.webkit.org/show_bug.cgi?id=48258 7 8 * Scripts/webkitpy/style/checkers/cpp.py: Added exceptions 9 * Scripts/webkitpy/style/checkers/cpp_unittest.py: Added unit-tests 10 1 11 2010-10-26 Ojan Vafai <ojan@chromium.org> 2 12 -
trunk/WebKitTools/Scripts/webkitpy/style/checkers/cpp.py
r69953 r70600 1294 1294 1295 1295 # Don't try to do spacing checks for operator methods 1296 line = sub(r'operator(==|!=|<|<<|<=|>=|>>|> )\(', 'operator\(', line)1296 line = sub(r'operator(==|!=|<|<<|<=|>=|>>|>|\+=|-=|\*=|/=|%=|&=|\|=|^=|<<=|>>=)\(', 'operator\(', line) 1297 1297 # Don't try to do spacing checks for #include or #import statements at 1298 1298 # minimum because it messes up checks for spacing around / -
trunk/WebKitTools/Scripts/webkitpy/style/checkers/cpp_unittest.py
r69953 r70600 1314 1314 '') 1315 1315 1316 def test_operator_methods(self): 1317 self.assert_lint('String operator+(const String&, const String&);', '') 1318 self.assert_lint('bool operator==(const String&, const String&);', '') 1319 self.assert_lint('String& operator-=(const String&, const String&);', '') 1320 self.assert_lint('String& operator+=(const String&, const String&);', '') 1321 self.assert_lint('String& operator*=(const String&, const String&);', '') 1322 self.assert_lint('String& operator%=(const String&, const String&);', '') 1323 self.assert_lint('String& operator&=(const String&, const String&);', '') 1324 self.assert_lint('String& operator<<=(const String&, const String&);', '') 1325 self.assert_lint('String& operator>>=(const String&, const String&);', '') 1326 self.assert_lint('String& operator|=(const String&, const String&);', '') 1327 self.assert_lint('String& operator^=(const String&, const String&);', '') 1328 1316 1329 def test_spacing_before_last_semicolon(self): 1317 1330 self.assert_lint('call_function() ;',
Note: See TracChangeset
for help on using the changeset viewer.