Changeset 194471 in webkit
- Timestamp:
- Dec 31, 2015 2:51:40 PM (8 years ago)
- Location:
- trunk/Tools
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Tools/ChangeLog
r194470 r194471 1 2015-12-31 Andy Estes <aestes@apple.com> 2 3 Update the style checker to advise using WTFMove() instead of WTF::move() 4 https://bugs.webkit.org/show_bug.cgi?id=152601 5 6 Reviewed by Daniel Bates. 7 8 * Scripts/webkitpy/style/checkers/cpp.py: 9 (check_wtf_move): 10 * Scripts/webkitpy/style/checkers/cpp_unittest.py: 11 (WebKitStyleTest.test_wtf_move): 12 1 13 2015-12-31 Andy Estes <aestes@apple.com> 2 14 -
trunk/Tools/Scripts/webkitpy/style/checkers/cpp.py
r192274 r194471 2257 2257 2258 2258 def check_wtf_move(clean_lines, line_number, file_state, error): 2259 """Looks for use of 'std::move()' which should be replaced with 'WTF ::move()'.2259 """Looks for use of 'std::move()' which should be replaced with 'WTFMove()'. 2260 2260 2261 2261 Args: … … 2277 2277 return 2278 2278 2279 error(line_number, 'runtime/wtf_move', 4, "Use 'WTF ::move()' instead of 'std::move()'.")2279 error(line_number, 'runtime/wtf_move', 4, "Use 'WTFMove()' instead of 'std::move()'.") 2280 2280 2281 2281 -
trunk/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
r192274 r194471 4802 4802 def test_wtf_move(self): 4803 4803 self.assert_lint( 4804 'A a = WTF ::move(b);',4804 'A a = WTFMove(b);', 4805 4805 '', 4806 4806 'foo.cpp') … … 4808 4808 self.assert_lint( 4809 4809 'A a = std::move(b);', 4810 "Use 'WTF ::move()' instead of 'std::move()'."4810 "Use 'WTFMove()' instead of 'std::move()'." 4811 4811 " [runtime/wtf_move] [4]", 4812 4812 'foo.cpp') … … 4814 4814 self.assert_lint( 4815 4815 'A a = std::move(b);', 4816 "Use 'WTF ::move()' instead of 'std::move()'."4816 "Use 'WTFMove()' instead of 'std::move()'." 4817 4817 " [runtime/wtf_move] [4]", 4818 4818 'foo.mm')
Note: See TracChangeset
for help on using the changeset viewer.