mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2026-01-07 22:30:13 -06:00
Code Issues Packages Projects Releases Wiki Activity

Merge topic 'genex-fix-short-circuit'

Browse Source 
27244a8f73 cmGeneratorExpressionNode: Fix short-circuit logic

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: buildbot <buildbot@kitware.com>
Merge-request: !8966
This commit is contained in:
Brad King
2023-11-14 14:07:01 +00:00
committed by Kitware Robot
parent 120d6b81f0 27244a8f73
commit 7b2f771077
3 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Source/cmGeneratorExpressionNode.cxx
View File

@@ -131,7 +131,7 @@ struct BooleanOpNode : public cmGeneratorExpressionNode
bool ShouldEvaluateNextParameter(const std::vector<std::string>& parameters,
std::string& def_value) const override
{
if (!parameters.empty() && parameters[0] == failureVal) {
if (!parameters.empty() && parameters.back() == failureVal) {
def_value = failureVal;
return false;
}

3
Tests/RunCMake/GeneratorExpressionShortCircuit/GoodAND.cmake
View File

@@ -1,4 +1,7 @@
set(error $<0>)
add_custom_target(check ALL COMMAND check
$<AND:0,${error}>
$<AND:0,1,${error}>
$<AND:1,0,${error}>
$<AND:0,0,${error}>
)

2
Tests/RunCMake/GeneratorExpressionShortCircuit/GoodOR.cmake
View File

@@ -1,4 +1,6 @@
set(error $<0>)
add_custom_target(check ALL COMMAND check
$<OR:1,${error}>
$<OR:0,1,${error}>
$<OR:1,0,${error}>
)