summaryrefslogtreecommitdiff
path: root/seckey.h
diff options
context:
space:
mode:
authorJeffrey Walton <noloader@gmail.com>2016-09-21 12:47:26 -0400
committerJeffrey Walton <noloader@gmail.com>2016-09-21 12:47:26 -0400
commit08a206f3eb2713a17eda636ebb96ed8142a4036d (patch)
tree044e409267a5e37ae196f02360dfd29af9e8e293 /seckey.h
parentcaf8aef8fe91229d9df370596519e8e80eba6eb7 (diff)
downloadcryptopp-git-08a206f3eb2713a17eda636ebb96ed8142a4036d.tar.gz
Rewrite 'keylength < MIN_KEYLENGTH' to avoid -Wtautological-compare
The warning surfaces under earlier Clnag's and GCC's
Diffstat (limited to 'seckey.h')
-rw-r--r--seckey.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/seckey.h b/seckey.h
index bbf1ea7a..e033c8df 100644
--- a/seckey.h
+++ b/seckey.h
@@ -215,7 +215,8 @@ public:
// TODO: Figure out how to make this CRYPTOPP_CONSTEXPR
static size_t CRYPTOPP_API StaticGetValidKeyLength(size_t keylength)
{
- if (keylength < (size_t)MIN_KEYLENGTH)
+ // Rewrite 'keylength < MIN_KEYLENGTH' to avoid -Wtautological-compare
+ if ((int)keylength - MIN_KEYLENGTH < 0)
return MIN_KEYLENGTH;
else if (keylength > (size_t)MAX_KEYLENGTH)
return (size_t)MAX_KEYLENGTH;