diff options
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/bugprone-swapped-arguments.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/bugprone-swapped-arguments.cpp | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/clang-tools-extra/test/clang-tidy/bugprone-swapped-arguments.cpp b/clang-tools-extra/test/clang-tidy/bugprone-swapped-arguments.cpp new file mode 100644 index 00000000000..06fb2d869f6 --- /dev/null +++ b/clang-tools-extra/test/clang-tidy/bugprone-swapped-arguments.cpp @@ -0,0 +1,53 @@ +// RUN: %check_clang_tidy %s bugprone-swapped-arguments %t + +void F(int, double); + +int SomeFunction(); + +template <typename T, typename U> +void G(T a, U b) { + F(a, b); // no-warning + F(2.0, 4); +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// CHECK-FIXES: F(4, 2.0) +} + +void foo() { + F(1.0, 3); +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// CHECK-FIXES: F(3, 1.0) + +#define M(x, y) x##y() + + double b = 1.0; + F(b, M(Some, Function)); +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// CHECK-FIXES: F(M(Some, Function), b); + +#define N F(b, SomeFunction()) + + N; +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// In macro, don't emit fixits. +// CHECK-FIXES: #define N F(b, SomeFunction()) + + G(b, 3); + G(3, 1.0); + G(0, 0); + + F(1.0, 1.0); // no-warning + F(3, 1.0); // no-warning + F(true, false); // no-warning + F(0, 'c'); // no-warning + +#define APPLY(f, x, y) f(x, y) + APPLY(F, 1.0, 3); +// CHECK-MESSAGES: :[[@LINE-1]]:9: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// CHECK-FIXES: APPLY(F, 3, 1.0); + +#define PARAMS 1.0, 3 +#define CALL(P) F(P) + CALL(PARAMS); +// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: argument with implicit conversion from 'int' to 'double' followed by argument converted from 'double' to 'int', potentially swapped arguments. +// In macro, don't emit fixits. +} |