diff options
| author | Brad King <brad.king@kitware.com> | 2022-02-09 19:16:50 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2022-02-09 19:24:43 (GMT) |
| commit | 41adfc6b041474f6985ab72efcfb02b2b64a6b8d (patch) | |
| tree | 3624c5d8c5f34145f26709cf164ee840210f1d19 /Tests | |
| parent | a15cc7706da8f4a1833539be3f37fbc63ee20e36 (diff) | |
| download | CMake-41adfc6b041474f6985ab72efcfb02b2b64a6b8d.zip CMake-41adfc6b041474f6985ab72efcfb02b2b64a6b8d.tar.gz CMake-41adfc6b041474f6985ab72efcfb02b2b64a6b8d.tar.bz2 | |
Help: Clarify precedence of AND and OR in 'if' conditions
The wording update in commit b74819e4fe (Help: Format 'if' command
documentation, 2013-12-18, v3.0.0-rc1~227^2~1) incorrectly implied that
`AND` has higher precedence than `OR`. Although this is common in many
languages, it has never been true in CMake's implementation. Revise
the wording to clarify the precedence.
Add a test case demonstrating the order.
Fixes: #23207
Diffstat (limited to 'Tests')
| -rw-r--r-- | Tests/RunCMake/if/AndOr-stdout.txt | 1 | ||||
| -rw-r--r-- | Tests/RunCMake/if/AndOr.cmake | 6 | ||||
| -rw-r--r-- | Tests/RunCMake/if/RunCMakeTest.cmake | 2 |
3 files changed, 9 insertions, 0 deletions
diff --git a/Tests/RunCMake/if/AndOr-stdout.txt b/Tests/RunCMake/if/AndOr-stdout.txt new file mode 100644 index 0000000..9fd395b --- /dev/null +++ b/Tests/RunCMake/if/AndOr-stdout.txt @@ -0,0 +1 @@ +-- OR and AND correctly evaluated left to right diff --git a/Tests/RunCMake/if/AndOr.cmake b/Tests/RunCMake/if/AndOr.cmake new file mode 100644 index 0000000..847d6f2 --- /dev/null +++ b/Tests/RunCMake/if/AndOr.cmake @@ -0,0 +1,6 @@ +# AND and OR are the same precedence +if(1 OR 0 AND 0) # equivalent to ((1 OR 0) AND 0) + message(FATAL_ERROR "AND incorrectly evaluated before OR") +else() + message(STATUS "OR and AND correctly evaluated left to right") +endif() diff --git a/Tests/RunCMake/if/RunCMakeTest.cmake b/Tests/RunCMake/if/RunCMakeTest.cmake index 6baa840..de9cb57 100644 --- a/Tests/RunCMake/if/RunCMakeTest.cmake +++ b/Tests/RunCMake/if/RunCMakeTest.cmake @@ -17,3 +17,5 @@ run_cmake(IncompleteMatchesFail) run_cmake(TestNameThatExists) run_cmake(TestNameThatDoesNotExist) + +run_cmake_script(AndOr) |