Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(824)

Unified Diff: tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc

Issue 1617253002: rewrite_to_chrome_style: exclude conversion functions from being renamed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
diff --git a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
index 719acfbd331add8111387cd64e7a84825f3b8b57..0b35d834a63bd59fdcfe948b60b48a9f623d0a30 100644
--- a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
+++ b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
@@ -16,7 +16,12 @@ class Task {
// Note: this is purposely copyable and assignable, to make sure the Clang
// tool doesn't try to emit replacements for things that aren't explicitly
// written.
- // TODO(dcheng): Add an explicit test for something like operator+.
+
+ // Overloaded operators should not be rewritten.
+ Task& operator++() { return *this; }
+
+ // Conversion functions should not be rewritten.
+ explicit operator int() const { return 42; }
};
// Test that the actual method definition is also updated.

Powered by Google App Engine
This is Rietveld 408576698