summaryrefslogtreecommitdiffstats
path: root/Source/cmConditionEvaluator.cxx
Commit message (Collapse)AuthorAgeFilesLines
* Source: Fix -Wdangling-reference warnings exposed by gcc 13Brad King2023-05-221-2/+2
|
* CMake code rely on cmList class for CMake lists management (part. 1)Marc Chevrier2023-04-241-1/+4
|
* if command: Add PATH_EQUAL operatorMarc Chevrier2022-06-031-0/+26
|
* Merge topic 'reduce-policy-eval-scope'Brad King2022-05-241-4/+2
|\ | | | | | | | | | | | | cb19a63499 cmConditionEvaluator: Reduce the scope of the CMP0064 evaluation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !7287
| * cmConditionEvaluator: Reduce the scope of the CMP0064 evaluationAlex Turbov2022-05-231-4/+2
| | | | | | | | | | When evaluatig conditions, avoid testing CMP0064 unless it's really needed (Cuz most of the time it doesn't)
* | clang-tidy: address `google-readability-casting` lintsBen Boeckel2022-05-241-17/+26
|/ | | | At least those involving `static_cast`.
* Source: Fix clang -Wimplicit-fallthrough warningsSean McBride2021-09-281-0/+1
|
* Source: Fix clang -Wextra-semi warningsSean McBride2021-09-281-1/+1
|
* Rename cmProp in cmValueMarc Chevrier2021-09-211-19/+19
|
* cmSystemTools::VersionCompare: use std::string for argumentsMarc Chevrier2021-09-171-4/+3
|
* Fix: Regression in the `cmConditionEvaluator::HandleLevel0`Alex Turbov2021-08-091-1/+7
| | | | | | | | As reported in the BUG #22524, mismatched parenthesis reported differently for `while()` and `if()`. The problem was in the double loop (over "handlers" and the arguments), where the outer loop didn't check the result of the running handler.
* cmProp: refactoring: transform alias in classMarc Chevrier2021-08-081-4/+4
| | | | | | To handle safely the values used by CMake variables and properties, introduce the class cmProp as a replacement from the simple pointer to std::string instance.
* Refactor: Move version compare op table out of the functionAlex Turbov2021-08-031-7/+7
| | | | It's static data and never changed.
* Refactor: Remove redundant checks for `0` and `1` literalsAlex Turbov2021-08-031-9/+1
| | | | They are gonna be checked by `cmIsOn()` anyway.
* Refactor: Copy exactly required count of args and avoid `pop_back()`Alex Turbov2021-08-031-6/+7
|
* Style: Better names of local vars for `IN_LIST` handlerAlex Turbov2021-08-031-4/+6
|
* Refactor: Extract the logic of testing for special variable to funcAlex Turbov2021-08-031-14/+27
| | | | Introduce `looksLikeSpecialVariable`. It makes `if()` blocks readable.
* Refactor: Introduce `cmArgumentList` container classAlex Turbov2021-08-031-141/+197
| | | | | | | | | | | | | | | | | The `cmArgumentList` has been turned into a class (forward declared in the header). It inherits from the `std::list` (yeah, but we don't intend to store polymorphic classes in it). In addition to the standard methods, now it's possible to move `HandlePredicate` (renamed to `ReduceOneArg`) and `HandleBinaryOp` (renamed to `ReduceTwoArgs`) as its members. Additionally, iterators managements (`IncrementArguments`) have been refactored into two separate classes mimicking iterators. This also allows having a uniform `for` loop and concentrates the logic of iterators advancing in it instead of the loop's body. The arguments processing algorithms operate with "windows" over a collection of arguments. Hence there are two kinds of "iteration windows" -- allowing to observe 2 or 3 elements per loop iteration. These iteration "windows" also passed to reducers.
* Refactor: Reduce one more condition checking on handling math compareAlex Turbov2021-08-031-12/+13
|
* Refactor: Avoid `if` → `else if` → … for compare operatorsAlex Turbov2021-08-031-62/+95
| | | | | | When trying to match one of compare operator key inside `if()` condition, remember the index of matched operator. Later this index used to select the operation to perform instead of strings compare again.
* Refactor: Set `const` to `cmConditionEvaluator::IsKeyword` parametersAlex Turbov2021-08-031-2/+3
|
* Refactor: No need to move iterators after matchAlex Turbov2021-08-031-40/+28
| | | | | | | | | | A handler's loop of all levels gonna restart after calls to `HandlePredcate` or `HandleBinaryOp`... And the first action in the loop is to setup iterators. So, no need to move 'em inside the functions. Also, no need to pass iterators by reference. Also, reorder parameters of both functions so iterators followed after a reference to the newArgs container.
* Refactor: Replace `std::to_string` w/ more specialized `bool2string`Alex Turbov2021-08-031-5/+9
|
* Refactor: Reorder `MATCHES` handler from top below to the fail-fastAlex Turbov2021-08-031-18/+18
|
* Refactor: Reduce variables scope in `HandleLevel2`Alex Turbov2021-08-031-14/+12
|
* Refactor: Simplify boolean to string result assignmentsAlex Turbov2021-08-031-5/+4
|
* Refactor: Replace `if` block w/ boolean expressionAlex Turbov2021-08-031-6/+2
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Avoid redundant `operator<<` on printing messagesAlex Turbov2021-08-031-9/+15
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Opt-out do+while loops and reduce nesting level in handlersAlex Turbov2021-08-031-309/+287
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Remove `reducible` flag from `handleLevelN()` functionsAlex Turbov2021-08-031-45/+45
| | | | | | | The indicator that smth has been done is the `newArgs` size get differ after an iteration. Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Make `cmConditionEvaluator::IsTrue` a bit more compactAlex Turbov2021-08-031-19/+13
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Generalize policy checking in `HandleLevel1`Alex Turbov2021-07-261-18/+25
| | | | Also, move OLD policy checking out of the loop to evaluate it once.
* Refactor: When handle predicates, there is no need to check 2nd argAlex Turbov2021-07-261-30/+34
| | | | | Introduce an overload for `IncrementArguments()` w/ one iterator and use it in the handler level 1.
* Refactor: Initialize args vector from iterators instead of copyAlex Turbov2021-07-261-4/+2
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Speedup predicates and binary operationAlex Turbov2021-07-261-49/+60
| | | | | | | | | | Before predicates and binary ops reducers use series of `if ()` blocks to match the keywords. However, if matched the currect `arg` get replaced with evaluation result, so further `if (<match-another-keyword>)` is just wasting time/resources. This patch introduce a chain of `if` → `else if` → ..., so after first match the loop restarts w/ the next argument.
* Style: Add empty lines to increase readabilityAlex Turbov2021-07-261-0/+20
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Deduplicate code for `AND` and `OR` handling in `if()` commandAlex Turbov2021-07-261-16/+9
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Rewrite parenthesis scanner to avoid `if`sAlex Turbov2021-07-261-12/+5
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Make `HandleBinaryOp` and `HandlePredicate` free functionsAlex Turbov2021-07-261-60/+52
|
* Refactor: More `auto` and constnessAlex Turbov2021-07-261-33/+27
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Make `IncrementArguments()` the free functionAlex Turbov2021-07-261-21/+22
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Turn the innter `while` loop in `HandleLevelN()` into `for`Alex Turbov2021-07-261-30/+10
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Turn `reducible` flag into booleanAlex Turbov2021-07-261-16/+16
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Use `cm::string_view` for static const literalsAlex Turbov2021-07-261-46/+47
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Opt-out `if` stataments to select 1st paramAlex Turbov2021-07-261-18/+10
| | | | | | ... for some calls to `cmExpandedCommandArgument`. Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Add constnessAlex Turbov2021-07-261-10/+10
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Refactor: Use anonymous namespace instead of `static` keyword for constsAlex Turbov2021-07-261-32/+34
| | | | Signed-off-by: Alex Turbov <i.zaufi@gmail.com>
* Update CMake code using KWSys to account for Status return valuesBrad King2021-04-141-2/+2
| | | | | | KWSys as of 2021-04-14 changed the return type of `SystemTools` operations from `bool` to `Status`. Update our call sites. This may improve error reporting accuracy in a few places.
* Code style: add missed explicit 'this->'Oleksandr Koval2021-01-051-8/+8
| | | | | CMake uses explicit 'this->' style. Using custom clang-tidy check we can detect and fix places where 'this->' was missed.
* cmConditionEvaluator: Remove extra copy of execution contextBrad King2020-09-281-6/+2
| | | | | | The execution context passed to the constructor always matches the top of the backtrace, so the former can be removed in favor of using only the latter.
generated by cgit v0.12 at 2025-01-14 14:06:17 (GMT)