Skip to content

Commit 41e9bb0

Browse files
committed
M4-10-2 fix compilation
1 parent f253bad commit 41e9bb0

File tree

2 files changed

+48
-49
lines changed

2 files changed

+48
-49
lines changed
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
| test.cpp:21:5:25:5 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:21:5:25:5 | if (...) ... | `if...else` |
2-
| test.cpp:41:7:45:7 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:41:7:45:7 | if (...) ... | `if...else` |
3-
| test.cpp:55:5:65:5 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:55:5:65:5 | if (...) ... | `if...else` |
1+
| test.cpp:21:3:25:3 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:21:3:25:3 | if (...) ... | `if...else` |
2+
| test.cpp:41:5:45:5 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:41:5:45:5 | if (...) ... | `if...else` |
3+
| test.cpp:55:3:65:3 | if (...) ... | The $@ construct does not terminate with else statement. | test.cpp:55:3:65:3 | if (...) ... | `if...else` |

cpp/common/test/rules/ifelseterminationconstruct/test.cpp

Lines changed: 45 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -10,58 +10,57 @@ void test_ifelse_valid(int expression) {
1010
} else {
1111
k = j * j;
1212
}
13-
void test_ifelse_mix_validity(int expression) {
14-
int i = 4;
15-
int j = 7;
16-
int k;
17-
18-
if (expression > 0) { // GOOD
19-
k = i * i;
20-
}
21-
if (expression > 10) { // BAD
22-
k = i + j;
23-
} else if (expression < 0) {
24-
k = i * 2;
25-
}
13+
}
14+
void test_ifelse_mix_validity(int expression) {
15+
int i = 4;
16+
int j = 7;
17+
int k;
18+
if (expression > 0) { // GOOD
19+
k = i * i;
2620
}
21+
if (expression > 10) { // BAD
22+
k = i + j;
23+
} else if (expression < 0) {
24+
k = i * 2;
25+
}
26+
}
2727

28-
void test_ifelse_nested_invalid(int expression) {
29-
int i = 5;
30-
int j = 7;
31-
int k;
28+
void test_ifelse_nested_invalid(int expression) {
29+
int i = 5;
30+
int j = 7;
31+
int k;
3232

33-
if (expression > 0) { // GOOD
34-
k = i * i * i;
35-
} else {
36-
k = i * j;
37-
}
38-
if (expression > 10) { // GOOD
39-
k = i;
40-
} else if (expression < 0) {
41-
if (expression < -10) { // BAD
42-
k = 5 + j;
43-
} else if (expression < -20) {
44-
k = i * 3;
45-
}
46-
} else {
47-
k = 3;
33+
if (expression > 0) { // GOOD
34+
k = i * i * i;
35+
} else {
36+
k = i * j;
37+
}
38+
if (expression > 10) { // GOOD
39+
k = i;
40+
} else if (expression < 0) {
41+
if (expression < -10) { // BAD
42+
k = 5 + j;
43+
} else if (expression < -20) {
44+
k = i * 3;
4845
}
46+
} else {
47+
k = 3;
4948
}
49+
}
5050

51-
void test_ifelse_nested_valid(int expression) {
52-
int i = 3;
53-
int j = 1;
54-
int k;
55-
if (expression > 10) { // BAD
56-
k = i + j;
57-
} else if (expression < 0) {
58-
if (i > 3) { // GOOD
59-
k = j;
60-
} else if (i < 10) {
61-
k = i % 3;
62-
} else {
63-
i = i % 2;
64-
}
51+
void test_ifelse_nested_valid(int expression) {
52+
int i = 3;
53+
int j = 1;
54+
int k;
55+
if (expression > 10) { // BAD
56+
k = i + j;
57+
} else if (expression < 0) {
58+
if (i > 3) { // GOOD
59+
k = j;
60+
} else if (i < 10) {
61+
k = i % 3;
62+
} else {
63+
i = i % 2;
6564
}
6665
}
6766
}

0 commit comments

Comments
 (0)