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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
|
// RUN: %check_clang_tidy %s bugprone-argument-comment %t -- \
// RUN: -config="{CheckOptions: [ \
// RUN: {key: bugprone-argument-comment.IgnoreSingleArgument, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentBoolLiterals, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentIntegerLiterals, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentFloatLiterals, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentUserDefinedLiterals, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentStringLiterals, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentNullPtrs, value: true}, \
// RUN: {key: bugprone-argument-comment.CommentCharacterLiterals, value: true}]}" --
struct A {
void foo(bool abc);
void foo(bool abc, bool cde);
void foo(const char *, bool abc);
void foo(int iabc);
void foo(float fabc);
void foo(double dabc);
void foo(const char *strabc);
void fooW(const wchar_t *wstrabc);
void fooPtr(A *ptrabc);
void foo(char chabc);
};
#define FOO 1
void g(int a);
void h(double b);
void i(const char *c);
double operator"" _km(long double);
void test() {
A a;
a.foo(true);
a.foo(false);
a.foo(true, false);
// CHECK-MESSAGES: [[@LINE-1]]:9: warning: argument comment missing for literal argument 'abc' [bugprone-argument-comment]
// CHECK-MESSAGES: [[@LINE-2]]:15: warning: argument comment missing for literal argument 'cde' [bugprone-argument-comment]
// CHECK-FIXES: a.foo(/*abc=*/true, /*cde=*/false);
a.foo(false, true);
// CHECK-MESSAGES: [[@LINE-1]]:9: warning: argument comment missing for literal argument 'abc' [bugprone-argument-comment]
// CHECK-MESSAGES: [[@LINE-2]]:16: warning: argument comment missing for literal argument 'cde' [bugprone-argument-comment]
// CHECK-FIXES: a.foo(/*abc=*/false, /*cde=*/true);
a.foo(/*abc=*/false, true);
// CHECK-MESSAGES: [[@LINE-1]]:24: warning: argument comment missing for literal argument 'cde' [bugprone-argument-comment]
// CHECK-FIXES: a.foo(/*abc=*/false, /*cde=*/true);
a.foo(false, /*cde=*/true);
// CHECK-MESSAGES: [[@LINE-1]]:9: warning: argument comment missing for literal argument 'abc' [bugprone-argument-comment]
// CHECK-FIXES: a.foo(/*abc=*/false, /*cde=*/true);
bool val1 = true;
bool val2 = false;
a.foo(val1, val2);
a.foo("", true);
// CHECK-MESSAGES: [[@LINE-1]]:13: warning: argument comment missing for literal argument 'abc' [bugprone-argument-comment]
// CHECK-FIXES: a.foo("", /*abc=*/true);
a.foo(0);
a.foo(1.0f);
a.foo(1.0);
int val3 = 10;
a.foo(val3);
float val4 = 10.0;
a.foo(val4);
double val5 = 10.0;
a.foo(val5);
a.foo("Hello World");
a.fooW(L"Hello World");
a.fooPtr(nullptr);
a.foo(402.0_km);
a.foo('A');
g(FOO);
h(1.0f);
i(__FILE__);
g((1));
}
void f(bool _with_underscores_);
void ignores_underscores() {
f(false);
f(true);
}
|
