From 90ba706b062f5ddcb970a999ab8365e921b864ba Mon Sep 17 00:00:00 2001 From: Mark Olesen Date: Thu, 21 Apr 2022 16:17:54 +0200 Subject: [PATCH] ENH: make op names unambiguous * lessEqOp -> lessEqualOp * greaterEqOp -> greaterEqualOp to avoid ambiguitity with other forms such as 'plusEqOp' where the 'Eq' implies an assigment. The name change also aligns better with C++ names such as std::less_equal, std::greater_equal ENH: simple labelRange predicates gt0/ge0/lt0/le0 - mirrors scalarRange tests. Lower overhead than using labelMinMax::ge(0) etc since it does not create an intermediate (is stateless) and can be used as a constexpr --- .../scalarPredicates/Test-scalarPredicates.C | 4 +- .../expressions/fields/fieldExprLemonParser.h | 4 +- .../expressions/fields/fieldExprScanner.cc | 4 +- .../expressions/fields/fieldExprScanner.rl | 4 +- .../include/m4/lemon/operator-precedence.m4 | 6 +-- .../include/m4/lemon/rules-operations.m4 | 6 +-- .../linearInterpolationWeights.C | 2 +- src/OpenFOAM/primitives/ops/ops.H | 13 +++--- .../predicates/scalar/scalarPredicates.C | 18 ++++---- .../predicates/scalar/scalarPredicates.H | 12 ++++-- .../primitives/ranges/IntRange/IntRange.H | 43 +++++++++++++++++++ .../expressions/patch/patchExprLemonParser.h | 4 +- .../expressions/patch/patchExprScanner.cc | 4 +- .../expressions/patch/patchExprScanner.rl | 4 +- .../volume/volumeExprLemonParser.h | 4 +- .../expressions/volume/volumeExprScanner.cc | 4 +- .../expressions/volume/volumeExprScanner.rl | 4 +- .../averageNeighbourFvGeometryScheme.C | 15 ++----- .../field/blendingFactor/blendingFactor.C | 2 +- .../lagrangian/common/parcelSelectionDetail.H | 4 +- .../buoyancyTurbSource/buoyancyTurbSource.C | 2 +- 21 files changed, 103 insertions(+), 60 deletions(-) diff --git a/applications/test/scalarPredicates/Test-scalarPredicates.C b/applications/test/scalarPredicates/Test-scalarPredicates.C index 688114f7d2..e2f5459f48 100644 --- a/applications/test/scalarPredicates/Test-scalarPredicates.C +++ b/applications/test/scalarPredicates/Test-scalarPredicates.C @@ -5,7 +5,7 @@ \\ / A nd | www.openfoam.com \\/ M anipulation | ------------------------------------------------------------------------------- - Copyright (C) 2018 OpenCFD Ltd. + Copyright (C) 2018-2022 OpenCFD Ltd. ------------------------------------------------------------------------------- License This file is part of OpenFOAM. @@ -184,7 +184,7 @@ int main(int argc, char *argv[]) testPredicate(values, greaterOp1(100)); // Also with dissimilar type - testPredicate(values, lessEqOp1