blob: 3b3175ab958e3b448171f8706ddf7c91e44bc968 [file] [log] [blame]
// RUN: %check_clang_tidy %s bugprone-suspicious-string-compare %t -- \
// RUN: -config='{CheckOptions: \
// RUN: [{key: bugprone-suspicious-string-compare.WarnOnImplicitComparison, value: 1}, \
// RUN: {key: bugprone-suspicious-string-compare.WarnOnLogicalNotComparison, value: 1}]}' \
// RUN: -- -std=c99
static const char A[] = "abc";
int strcmp(const char *, const char *);
int test_warning_patterns() {
if (strcmp(A, "a"))
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' is called without explicitly comparing result [bugprone-suspicious-string-compare]
// CHECK-FIXES: if (strcmp(A, "a") != 0)
if (strcmp(A, "a") != 0 ||
strcmp(A, "b"))
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' is called without explicitly comparing result
// CHECK-FIXES: strcmp(A, "b") != 0)
if (strcmp(A, "a") == 1)
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' is compared to a suspicious constant
if (strcmp(A, "a") == -1)
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' is compared to a suspicious constant
if (strcmp(A, "a") < '0')
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' is compared to a suspicious constant
if (strcmp(A, "a") < 0.)
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:7: warning: function 'strcmp' has suspicious implicit cast
if (!strcmp(A, "a"))
return 0;
// CHECK-MESSAGES: [[@LINE-2]]:8: warning: function 'strcmp' is compared using logical not operator
// CHECK-FIXES: if (strcmp(A, "a") == 0)
}
void test_structure_patterns() {
if (strcmp(A, "a")) {}
// CHECK-MESSAGES: [[@LINE-1]]:7: warning: function 'strcmp' is called without explicitly comparing result
// CHECK-FIXES: if (strcmp(A, "a") != 0) {}
while (strcmp(A, "a")) {}
// CHECK-MESSAGES: [[@LINE-1]]:10: warning: function 'strcmp' is called without explicitly comparing result
// CHECK-FIXES: while (strcmp(A, "a") != 0) {}
for (;strcmp(A, "a");) {}
// CHECK-MESSAGES: [[@LINE-1]]:9: warning: function 'strcmp' is called without explicitly comparing result
// CHECK-FIXES: for (;strcmp(A, "a") != 0;) {}
}
int test_valid_patterns() {
// The following cases are valid.
if (strcmp(A, "a") < 0) return 0;
if (strcmp(A, "a") == 0) return 0;
if (strcmp(A, "a") <= 0) return 0;
if (strcmp(A, "a") == strcmp(A, "b")) return 0;
return 1;
}
int wrapper(const char* a, const char* b) {
return strcmp(a, b);
}
int assignment_wrapper(const char* a, const char* b) {
int cmp = strcmp(a, b);
return cmp;
}
int condexpr_wrapper(const char* a, const char* b) {
return (a < b) ? strcmp(a, b) : strcmp(b, a);
}