diff options
| author | Luis A. Lozano <llozano@google.com> | 2017-10-25 19:00:47 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2017-10-26 09:29:20 +0200 |
| commit | 71f0fc6821fd0c5fc67f99e2588bf4e9fd5bd396 (patch) | |
| tree | b88d9413d3c9191ee6acffd1bf9db885f6590b07 | |
| parent | 098c4c0271e04b564da57bb5c9e4e8e067bd58c9 (diff) | |
| download | ModemManager-71f0fc6821fd0c5fc67f99e2588bf4e9fd5bd396.tar.gz | |
sms-cdma-part: fix tautological-constant-compare compiler warning
This patch fixes the following compiler warning issued by clang:
mm-sms-part-cdma.c:301:46: mcomparison 'guint8' (aka 'unsigned char') <= 255 is always true
[-Werror,-Wtautological-constant-compare]
| -rw-r--r-- | src/mm-sms-part-cdma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mm-sms-part-cdma.c b/src/mm-sms-part-cdma.c index 167eda83b..5379c58b9 100644 --- a/src/mm-sms-part-cdma.c +++ b/src/mm-sms-part-cdma.c @@ -298,7 +298,7 @@ cause_code_to_delivery_state (guint8 error_class, else if (cause_code == 101) /* 101 reserved */ delivery_state += CAUSE_CODE_GENERAL_PROBLEM_OTHER; - else if (cause_code >= 108 && cause_code <= 255) + else if (cause_code >= 108) /* cause_code <= 255 is always true */ /* 108 to 223 reserved, treat as CAUSE_CODE_GENERAL_PROBLEM_OTHER * 224 to 255 reserved for TIA/EIA-41 extension, otherwise treat as CAUSE_CODE_GENERAL_PROBLEM_OTHER */ delivery_state += CAUSE_CODE_GENERAL_PROBLEM_OTHER; |