1 | // RUN: %check_clang_tidy -format-style=LLVM -check-suffix=DEFAULT %s \ |
2 | // RUN: modernize-replace-disallow-copy-and-assign-macro %t |
3 | |
4 | // RUN: %check_clang_tidy -format-style=LLVM -check-suffix=DIFFERENT-NAME %s \ |
5 | // RUN: modernize-replace-disallow-copy-and-assign-macro %t \ |
6 | // RUN: -config="{CheckOptions: { \ |
7 | // RUN: modernize-replace-disallow-copy-and-assign-macro.MacroName: \ |
8 | // RUN: MY_MACRO_NAME}}" |
9 | |
10 | // RUN: %check_clang_tidy -format-style=LLVM -check-suffix=FINALIZE %s \ |
11 | // RUN: modernize-replace-disallow-copy-and-assign-macro %t \ |
12 | // RUN: -config="{CheckOptions: { \ |
13 | // RUN: modernize-replace-disallow-copy-and-assign-macro.MacroName: \ |
14 | // RUN: DISALLOW_COPY_AND_ASSIGN_FINALIZE}}" |
15 | |
16 | // RUN: clang-tidy %s -checks="-*,modernize-replace-disallow-copy-and-assign-macro" \ |
17 | // RUN: -config="{CheckOptions: { \ |
18 | // RUN: modernize-replace-disallow-copy-and-assign-macro.MacroName: \ |
19 | // RUN: DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS}}" -- -Wno-extra-semi | count 0 |
20 | |
21 | // RUN: clang-tidy %s -checks="-*,modernize-replace-disallow-copy-and-assign-macro" \ |
22 | // RUN: -config="{CheckOptions: { \ |
23 | // RUN: modernize-replace-disallow-copy-and-assign-macro.MacroName: \ |
24 | // RUN: DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION}}" -- -Wno-extra-semi | count 0 |
25 | |
26 | // Note: the last two tests expect no diagnostics, but FileCheck cannot handle |
27 | // that, hence the use of | count 0. |
28 | |
29 | #define DISALLOW_COPY_AND_ASSIGN(TypeName) |
30 | |
31 | class TestClass1 { |
32 | private: |
33 | DISALLOW_COPY_AND_ASSIGN(TestClass1); |
34 | }; |
35 | // CHECK-MESSAGES-DEFAULT: :[[@LINE-2]]:3: warning: prefer deleting copy constructor and assignment operator over using macro 'DISALLOW_COPY_AND_ASSIGN' [modernize-replace-disallow-copy-and-assign-macro] |
36 | // CHECK-FIXES-DEFAULT: {{^}} TestClass1(const TestClass1 &) = delete;{{$}} |
37 | // CHECK-FIXES-DEFAULT-NEXT: {{^}} const TestClass1 &operator=(const TestClass1 &) = delete;{{$}} |
38 | |
39 | #define MY_MACRO_NAME(TypeName) |
40 | |
41 | class TestClass2 { |
42 | private: |
43 | MY_MACRO_NAME(TestClass2); |
44 | }; |
45 | // CHECK-MESSAGES-DIFFERENT-NAME: :[[@LINE-2]]:3: warning: prefer deleting copy constructor and assignment operator over using macro 'MY_MACRO_NAME' [modernize-replace-disallow-copy-and-assign-macro] |
46 | // CHECK-FIXES-DIFFERENT-NAME: {{^}} TestClass2(const TestClass2 &) = delete;{{$}} |
47 | // CHECK-FIXES-DIFFERENT-NAME-NEXT: {{^}} const TestClass2 &operator=(const TestClass2 &) = delete;{{$}} |
48 | |
49 | #define DISALLOW_COPY_AND_ASSIGN_FINALIZE(TypeName) \ |
50 | TypeName(const TypeName &) = delete; \ |
51 | const TypeName &operator=(const TypeName &) = delete; |
52 | |
53 | class TestClass3 { |
54 | private: |
55 | // Notice, that the macro allows to be used without a semicolon because the |
56 | // macro definition already contains one above. Therefore our replacement must |
57 | // contain a semicolon at the end. |
58 | DISALLOW_COPY_AND_ASSIGN_FINALIZE(TestClass3) |
59 | }; |
60 | // CHECK-MESSAGES-FINALIZE: :[[@LINE-2]]:3: warning: prefer deleting copy constructor and assignment operator over using macro 'DISALLOW_COPY_AND_ASSIGN_FINALIZE' [modernize-replace-disallow-copy-and-assign-macro] |
61 | // CHECK-FIXES-FINALIZE: {{^}} TestClass3(const TestClass3 &) = delete;{{$}} |
62 | // CHECK-FIXES-FINALIZE-NEXT: {{^}} const TestClass3 &operator=(const TestClass3 &) = delete;{{$}} |
63 | |
64 | #define DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS(A, B) |
65 | |
66 | class TestClass4 { |
67 | private: |
68 | DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS(TestClass4, TestClass4); |
69 | }; |
70 | // CHECK-MESSAGES-MORE-ARGUMENTS-NOT: warning: prefer deleting copy constructor and assignment operator over using macro 'DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS' |
71 | |
72 | #define DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION(A) |
73 | #define TESTCLASS TestClass5 |
74 | |
75 | class TestClass5 { |
76 | private: |
77 | DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION(TESTCLASS); |
78 | }; |
79 | // CHECK-MESSAGES-MORE-ARGUMENTS-NOT: warning: prefer deleting copy constructor and assignment operator over using macro 'DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION' |
80 | |