[PATCH] D42120: [clang-tidy] Fixing Fuchsia overloaded operator warning message

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 16 11:01:42 PST 2018


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: test/clang-tidy/fuchsia-overloaded-operator.cpp:6
   int operator+(int);
-  // CHECK-MESSAGES: [[@LINE-1]]:3: warning: cannot overload 'operator+' [fuchsia-overloaded-operator]
+  // CHECK-MESSAGES: [[@LINE-1]]:3: warning: overloading 'operator+' is disallowed [fuchsia-overloaded-operator]
 };
----------------
Can you remove the `[fuchsia-overloaded-operator]` from all of the comments in this file as well? That can be done in a follow-up commit and no review is needed for it.


https://reviews.llvm.org/D42120





More information about the cfe-commits mailing list