| 1 | // RUN: %check_clang_tidy -std=c++98-or-later %s bugprone-non-zero-enum-to-bool-conversion %t -- \ |
| 2 | // RUN: -config="{CheckOptions: {bugprone-non-zero-enum-to-bool-conversion.EnumIgnoreList: '::without::issue::IgnoredEnum;IgnoredSecondEnum'}}" |
| 3 | |
| 4 | namespace with::issue { |
| 5 | |
| 6 | typedef enum EStatus { |
| 7 | SUCCESS = 1, |
| 8 | FAILURE = 2, |
| 9 | INVALID_PARAM = 3, |
| 10 | UNKNOWN = 4 |
| 11 | } Status; |
| 12 | |
| 13 | bool testEnumConversion(EStatus value) { |
| 14 | // CHECK-MESSAGES: :[[@LINE+1]]:10: warning: conversion of 'EStatus' into 'bool' will always return 'true', enum doesn't have a zero-value enumerator [bugprone-non-zero-enum-to-bool-conversion] |
| 15 | return value; |
| 16 | } |
| 17 | |
| 18 | bool testTypedefConversion(Status value) { |
| 19 | // CHECK-MESSAGES: :[[@LINE+1]]:10: warning: conversion of 'EStatus' into 'bool' will always return 'true', enum doesn't have a zero-value enumerator [bugprone-non-zero-enum-to-bool-conversion] |
| 20 | return value; |
| 21 | } |
| 22 | |
| 23 | bool testExplicitConversion(EStatus value) { |
| 24 | // CHECK-MESSAGES: :[[@LINE+1]]:28: warning: conversion of 'EStatus' into 'bool' will always return 'true', enum doesn't have a zero-value enumerator [bugprone-non-zero-enum-to-bool-conversion] |
| 25 | return static_cast<bool>(value); |
| 26 | } |
| 27 | |
| 28 | bool testInIfConversion(EStatus value) { |
| 29 | // CHECK-MESSAGES: :[[@LINE+1]]:7: warning: conversion of 'EStatus' into 'bool' will always return 'true', enum doesn't have a zero-value enumerator [bugprone-non-zero-enum-to-bool-conversion] |
| 30 | if (value) { |
| 31 | return false; |
| 32 | } |
| 33 | return true; |
| 34 | } |
| 35 | |
| 36 | bool testWithNegation(EStatus value) { |
| 37 | // CHECK-MESSAGES: :[[@LINE+1]]:14: warning: conversion of 'EStatus' into 'bool' will always return 'true', enum doesn't have a zero-value enumerator [bugprone-non-zero-enum-to-bool-conversion] |
| 38 | return not value; |
| 39 | } |
| 40 | |
| 41 | } |
| 42 | |
| 43 | namespace without::issue { |
| 44 | |
| 45 | enum StatusWithZero { |
| 46 | UNK = 0, |
| 47 | OK = 1, |
| 48 | NOT_OK = 2 |
| 49 | }; |
| 50 | |
| 51 | bool testEnumConversion(StatusWithZero value) { |
| 52 | return value; |
| 53 | } |
| 54 | |
| 55 | enum WithDefault { |
| 56 | Value0, |
| 57 | Value1 |
| 58 | }; |
| 59 | |
| 60 | bool testEnumConversion(WithDefault value) { |
| 61 | return value; |
| 62 | } |
| 63 | |
| 64 | enum WithNegative : int { |
| 65 | Nen2 = -2, |
| 66 | Nen1, |
| 67 | Nen0 |
| 68 | }; |
| 69 | |
| 70 | bool testEnumConversion(WithNegative value) { |
| 71 | return value; |
| 72 | } |
| 73 | |
| 74 | enum EStatus { |
| 75 | SUCCESS = 1, |
| 76 | FAILURE, |
| 77 | INVALID_PARAM, |
| 78 | UNKNOWN |
| 79 | }; |
| 80 | |
| 81 | bool explicitCompare(EStatus value) { |
| 82 | return value == SUCCESS; |
| 83 | } |
| 84 | |
| 85 | bool explicitBitUsage1(EStatus value) { |
| 86 | return (value & SUCCESS); |
| 87 | } |
| 88 | |
| 89 | bool explicitBitUsage2(EStatus value) { |
| 90 | return (value | SUCCESS); |
| 91 | } |
| 92 | |
| 93 | bool testEnumeratorCompare() { |
| 94 | return SUCCESS; |
| 95 | } |
| 96 | |
| 97 | enum IgnoredEnum { |
| 98 | IGNORED_VALUE_1 = 1, |
| 99 | IGNORED_VALUE_2 |
| 100 | }; |
| 101 | |
| 102 | enum IgnoredSecondEnum { |
| 103 | IGNORED_SECOND_VALUE_1 = 1, |
| 104 | IGNORED_SECOND_VALUE_2 |
| 105 | }; |
| 106 | |
| 107 | bool testIgnored(IgnoredEnum value) { |
| 108 | return value; |
| 109 | } |
| 110 | |
| 111 | bool testIgnored(IgnoredSecondEnum value) { |
| 112 | return value; |
| 113 | } |
| 114 | |
| 115 | enum CustomOperatorEnum { |
| 116 | E0 = 0x1, |
| 117 | E1 = 0x2, |
| 118 | E2 = 0x4 |
| 119 | }; |
| 120 | |
| 121 | CustomOperatorEnum operator&(CustomOperatorEnum a, CustomOperatorEnum b) { return static_cast<CustomOperatorEnum>(a & b); } |
| 122 | |
| 123 | void testCustomOperator(CustomOperatorEnum e) { |
| 124 | if (e & E1) {} |
| 125 | if ((e & E1)) {} |
| 126 | if (!(e & E1)) {} |
| 127 | } |
| 128 | |
| 129 | } |
| 130 | |