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

Unified Diff: tools/clang/plugins/tests/inline_ctor.cpp

Issue 1504033010: Follow macro invocations when checking if a violation happened in a .cc file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkcr
Patch Set: Check every file in the macro instantiation chain. Created 5 years 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/plugins/tests/inline_ctor.cpp
diff --git a/tools/clang/plugins/tests/inline_ctor.cpp b/tools/clang/plugins/tests/inline_ctor.cpp
index 6a751fb405072641b9acb62b0e2657a1f54801b1..f209b289f9a38b4ba361764ffb68090d88265a97 100644
--- a/tools/clang/plugins/tests/inline_ctor.cpp
+++ b/tools/clang/plugins/tests/inline_ctor.cpp
@@ -2,6 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#define MACRO_FROM_CPP INLINE_CTORS_IN_A_MACRO(InlineCtorsInvolvingCppAreOK)
+
#include "inline_ctor.h"
#include <string>
@@ -18,8 +20,13 @@ class InlineInCPPOK {
std::vector<std::string> two_;
};
+INLINE_CTORS_IN_A_MACRO(InlineCtorsBehindAMacroAreOKInCpp);
+
int main() {
InlineInCPPOK one;
InlineCtorsArentOKInHeader two;
+ InlineCtorsBehindAMacroArentOKInHeader three;
+ InlineCtorsBehindAMacroAreOKInCpp four;
+ InlineCtorsInvolvingCppAreOK five;
return 0;
}

Powered by Google App Engine
This is Rietveld 408576698