| 1 | // RUN: %check_clang_tidy --match-partial-fixes -std=c++17 %s modernize-use-integer-sign-comparison %t -- \ |
| 2 | // RUN: -config="{CheckOptions: {modernize-use-integer-sign-comparison.EnableQtSupport: true}}" |
| 3 | |
| 4 | // CHECK-FIXES: #include <QtCore/q20utility.h> |
| 5 | |
| 6 | // The code that triggers the check |
| 7 | #define MAX_MACRO(a, b) (a < b) ? b : a |
| 8 | |
| 9 | unsigned int FuncParameters(int bla) { |
| 10 | unsigned int result = 0; |
| 11 | if (result == bla) |
| 12 | return 0; |
| 13 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 14 | // CHECK-FIXES: if (q20::cmp_equal(result , bla)) |
| 15 | |
| 16 | return 1; |
| 17 | } |
| 18 | |
| 19 | template <typename T> |
| 20 | void TemplateFuncParameter(T val) { |
| 21 | unsigned long uL = 0; |
| 22 | if (val >= uL) |
| 23 | return; |
| 24 | // CHECK-MESSAGES-NOT: warning: |
| 25 | } |
| 26 | |
| 27 | template <typename T1, typename T2> |
| 28 | int TemplateFuncParameters(T1 val1, T2 val2) { |
| 29 | if (val1 >= val2) |
| 30 | return 0; |
| 31 | // CHECK-MESSAGES-NOT: warning: |
| 32 | return 1; |
| 33 | } |
| 34 | |
| 35 | int AllComparisons() { |
| 36 | unsigned int uVar = 42; |
| 37 | unsigned short uArray[7] = {0, 1, 2, 3, 9, 7, 9}; |
| 38 | |
| 39 | int sVar = -42; |
| 40 | short sArray[7] = {-1, -2, -8, -94, -5, -4, -6}; |
| 41 | |
| 42 | enum INT_TEST { |
| 43 | VAL1 = 0, |
| 44 | VAL2 = -1 |
| 45 | }; |
| 46 | |
| 47 | char ch = 'a'; |
| 48 | unsigned char uCh = 'a'; |
| 49 | signed char sCh = 'a'; |
| 50 | bool bln = false; |
| 51 | |
| 52 | if (bln == sVar) |
| 53 | return 0; |
| 54 | // CHECK-MESSAGES-NOT: warning: |
| 55 | |
| 56 | if (ch > uCh) |
| 57 | return 0; |
| 58 | // CHECK-MESSAGES-NOT: warning: |
| 59 | |
| 60 | if (sVar <= INT_TEST::VAL2) |
| 61 | return 0; |
| 62 | // CHECK-MESSAGES-NOT: warning: |
| 63 | |
| 64 | if (uCh < sCh) |
| 65 | return -1; |
| 66 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 67 | // CHECK-FIXES: if (q20::cmp_less(uCh , sCh)) |
| 68 | |
| 69 | if ((int)uVar < sVar) |
| 70 | return 0; |
| 71 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 72 | // CHECK-FIXES: if (q20::cmp_less(uVar, sVar)) |
| 73 | |
| 74 | (uVar != sVar) ? uVar = sVar |
| 75 | : sVar = uVar; |
| 76 | // CHECK-MESSAGES: :[[@LINE-2]]:6: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 77 | // CHECK-FIXES: (q20::cmp_not_equal(uVar , sVar)) ? uVar = sVar |
| 78 | |
| 79 | while (uArray[0] <= sArray[0]) |
| 80 | return 0; |
| 81 | // CHECK-MESSAGES: :[[@LINE-2]]:12: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 82 | // CHECK-FIXES: while (q20::cmp_less_equal(uArray[0] , sArray[0])) |
| 83 | |
| 84 | if (uArray[1] > sArray[1]) |
| 85 | return 0; |
| 86 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 87 | // CHECK-FIXES: if (q20::cmp_greater(uArray[1] , sArray[1])) |
| 88 | |
| 89 | MAX_MACRO(uVar, sArray[0]); |
| 90 | // CHECK-MESSAGES: :[[@LINE-1]]:15: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 91 | |
| 92 | if (static_cast<unsigned int>(uArray[2]) < static_cast<int>(sArray[2])) |
| 93 | return 0; |
| 94 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 95 | // CHECK-FIXES: if (q20::cmp_less(uArray[2],sArray[2])) |
| 96 | |
| 97 | if ((unsigned int)uArray[3] < (int)sArray[3]) |
| 98 | return 0; |
| 99 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 100 | // CHECK-FIXES: if (q20::cmp_less(uArray[3],sArray[3])) |
| 101 | |
| 102 | if ((unsigned int)(uArray[4]) < (int)(sArray[4])) |
| 103 | return 0; |
| 104 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 105 | // CHECK-FIXES: if (q20::cmp_less((uArray[4]),(sArray[4]))) |
| 106 | |
| 107 | if (uArray[5] > sArray[5]) |
| 108 | return 0; |
| 109 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 110 | // CHECK-FIXES: if (q20::cmp_greater(uArray[5] , sArray[5])) |
| 111 | |
| 112 | #define VALUE sArray[6] |
| 113 | if (uArray[6] > VALUE) |
| 114 | return 0; |
| 115 | // CHECK-MESSAGES: :[[@LINE-2]]:9: warning: comparison between 'signed' and 'unsigned' integers [modernize-use-integer-sign-comparison] |
| 116 | // CHECK-FIXES: if (q20::cmp_greater(uArray[6] , VALUE)) |
| 117 | |
| 118 | |
| 119 | FuncParameters(bla: uVar); |
| 120 | TemplateFuncParameter(val: sVar); |
| 121 | TemplateFuncParameters(val1: uVar, val2: sVar); |
| 122 | |
| 123 | return 0; |
| 124 | } |
| 125 | |