diff options
author | Alex Turbov <i.zaufi@gmail.com> | 2021-08-02 16:22:07 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2021-08-03 14:55:47 (GMT) |
commit | f37d6a31703bd74dddfbc82b41f3d97633edb70b (patch) | |
tree | b55c35fb647bc781388338e77685b766377fb1c4 | |
parent | 00961a4782d9b22adce4d75ea61eb2f1d16ad0e5 (diff) | |
download | CMake-f37d6a31703bd74dddfbc82b41f3d97633edb70b.zip CMake-f37d6a31703bd74dddfbc82b41f3d97633edb70b.tar.gz CMake-f37d6a31703bd74dddfbc82b41f3d97633edb70b.tar.bz2 |
Refactor: Remove redundant checks for `0` and `1` literals
They are gonna be checked by `cmIsOn()` anyway.
-rw-r--r-- | Source/cmConditionEvaluator.cxx | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/Source/cmConditionEvaluator.cxx b/Source/cmConditionEvaluator.cxx index ea5dce5..9485d04 100644 --- a/Source/cmConditionEvaluator.cxx +++ b/Source/cmConditionEvaluator.cxx @@ -355,15 +355,7 @@ bool cmConditionEvaluator::IsKeyword( bool cmConditionEvaluator::GetBooleanValue( cmExpandedCommandArgument& arg) const { - // Check basic constants. - if (arg == "0") { - return false; - } - if (arg == "1") { - return true; - } - - // Check named constants. + // Check basic and named constants. if (cmIsOn(arg.GetValue())) { return true; } |