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