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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #define MACRO_FROM_CPP INLINE_CTORS_IN_A_MACRO(InlineCtorsInvolvingCppAreOK)
6
5 #include "inline_ctor.h" 7 #include "inline_ctor.h"
6 8
7 #include <string> 9 #include <string>
8 #include <vector> 10 #include <vector>
9 11
10 // We don't warn on classes that are in CPP files. 12 // We don't warn on classes that are in CPP files.
11 class InlineInCPPOK { 13 class InlineInCPPOK {
12 public: 14 public:
13 InlineInCPPOK() {} 15 InlineInCPPOK() {}
14 ~InlineInCPPOK() {} 16 ~InlineInCPPOK() {}
15 17
16 private: 18 private:
17 std::vector<int> one_; 19 std::vector<int> one_;
18 std::vector<std::string> two_; 20 std::vector<std::string> two_;
19 }; 21 };
20 22
23 INLINE_CTORS_IN_A_MACRO(InlineCtorsBehindAMacroAreOKInCpp);
24
21 int main() { 25 int main() {
22 InlineInCPPOK one; 26 InlineInCPPOK one;
23 InlineCtorsArentOKInHeader two; 27 InlineCtorsArentOKInHeader two;
28 InlineCtorsBehindAMacroArentOKInHeader three;
29 InlineCtorsBehindAMacroAreOKInCpp four;
30 InlineCtorsInvolvingCppAreOK five;
24 return 0; 31 return 0;
25 } 32 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698