Skip to content

Commit 176d195

Browse files
committed
Remove specific warning type
on-behalf-of: @amd <[email protected]>
1 parent 7dcf216 commit 176d195

File tree

2 files changed

+14
-18
lines changed

2 files changed

+14
-18
lines changed

clang-tools-extra/clang-tidy/bugprone/DefaultLambdaCaptureCheck.cpp

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,8 @@ void DefaultLambdaCaptureCheck::check(const MatchFinder::MatchResult &Result) {
3333
if (DefaultCaptureLoc.isInvalid())
3434
return;
3535

36-
const char *CaptureKind =
37-
(Lambda->getCaptureDefault() == LCD_ByCopy) ? "by-copy" : "by-reference";
38-
39-
diag(DefaultCaptureLoc, "lambda %0 default capture is discouraged; "
40-
"prefer to capture specific variables explicitly")
41-
<< CaptureKind;
36+
diag(DefaultCaptureLoc, "lambda default captures are discouraged; "
37+
"prefer to capture specific variables explicitly");
4238
}
4339

4440
} // namespace clang::tidy::bugprone

clang-tools-extra/test/clang-tidy/checkers/bugprone/default-lambda-capture.cpp

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,16 @@ void test_default_captures() {
55
int another = 10;
66

77
auto lambda1 = [=](int x) { return value + x; };
8-
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
8+
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
99

1010
auto lambda2 = [&](int x) { return value + x; };
11-
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda by-reference default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
11+
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
1212

1313
auto lambda3 = [=, &another](int x) { return value + another + x; };
14-
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
14+
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
1515

1616
auto lambda4 = [&, value](int x) { return value + another + x; };
17-
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda by-reference default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
17+
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
1818
}
1919

2020
void test_acceptable_captures() {
@@ -42,10 +42,10 @@ void test_nested_lambdas() {
4242
int inner_var = 3;
4343

4444
auto outer = [=]() {
45-
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
45+
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
4646

4747
auto inner = [&](int x) { return outer_var + middle_var + inner_var + x; };
48-
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda by-reference default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
48+
// CHECK-MESSAGES: :[[@LINE-1]]:19: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
4949

5050
return inner(10);
5151
};
@@ -55,15 +55,15 @@ void test_lambda_returns() {
5555
int a = 1, b = 2, c = 3;
5656

5757
auto create_adder = [=](int x) {
58-
// CHECK-MESSAGES: :[[@LINE-1]]:24: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
58+
// CHECK-MESSAGES: :[[@LINE-1]]:24: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
5959
return [x](int y) { return x + y; }; // Inner lambda is fine - explicit capture
6060
};
6161

6262
auto func1 = [&]() { return a; };
63-
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda by-reference default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
63+
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
6464

6565
auto func2 = [=]() { return b; };
66-
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
66+
// CHECK-MESSAGES: :[[@LINE-1]]:17: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
6767
}
6868

6969
class TestClass {
@@ -74,10 +74,10 @@ class TestClass {
7474
int local = 10;
7575

7676
auto lambda1 = [=]() { return member + local; };
77-
// CHECK-MESSAGES: :[[@LINE-1]]:21: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
77+
// CHECK-MESSAGES: :[[@LINE-1]]:21: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
7878

7979
auto lambda2 = [&]() { return member + local; };
80-
// CHECK-MESSAGES: :[[@LINE-1]]:21: warning: lambda by-reference default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
80+
// CHECK-MESSAGES: :[[@LINE-1]]:21: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
8181

8282
auto lambda3 = [this, local]() { return member + local; };
8383
auto lambda4 = [this, &local]() { return member + local; };
@@ -89,7 +89,7 @@ void test_template_lambdas() {
8989
T value{};
9090

9191
auto lambda = [=](T x) { return value + x; };
92-
// CHECK-MESSAGES: :[[@LINE-1]]:18: warning: lambda by-copy default capture is discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
92+
// CHECK-MESSAGES: :[[@LINE-1]]:18: warning: lambda default captures are discouraged; prefer to capture specific variables explicitly [bugprone-default-lambda-capture]
9393
}
9494

9595
void instantiate_templates() {

0 commit comments

Comments
 (0)