summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Turbov <i.zaufi@gmail.com>2021-08-02 16:22:07 (GMT)
committerBrad King <brad.king@kitware.com>2021-08-03 14:55:47 (GMT)
commitf37d6a31703bd74dddfbc82b41f3d97633edb70b (patch)
treeb55c35fb647bc781388338e77685b766377fb1c4
parent00961a4782d9b22adce4d75ea61eb2f1d16ad0e5 (diff)
downloadCMake-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.cxx10
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;
}