From b4eb214d56478c7851819c3985e657c848dec0eb Mon Sep 17 00:00:00 2001 From: Mark Olesen Date: Thu, 26 Nov 2020 01:04:53 +0100 Subject: [PATCH] BUG: transformPoints flagged zero or negative scaling as an error - additional check was introduced by 51b249025802, but scaling with zero is useful for flattening a mesh --- .../manipulation/transformPoints/transformPoints.C | 11 +---------- .../surfaceTransformPoints/surfaceTransformPoints.C | 11 +---------- 2 files changed, 2 insertions(+), 20 deletions(-) diff --git a/applications/utilities/mesh/manipulation/transformPoints/transformPoints.C b/applications/utilities/mesh/manipulation/transformPoints/transformPoints.C index 769f3e79d9..29de0f2f53 100644 --- a/applications/utilities/mesh/manipulation/transformPoints/transformPoints.C +++ b/applications/utilities/mesh/manipulation/transformPoints/transformPoints.C @@ -195,16 +195,7 @@ List getScalingOpt(const word& optName, const argList& args) scaling.clear(); } - for (const scalar scale : scaling) - { - if (scale <= 0) - { - FatalError - << "Invalid scaling value, must be positive." << nl - << " -" << optName << ' ' << args[optName].c_str() << endl - << exit(FatalError); - } - } + // Zero and negative scaling are permitted return scaling; } diff --git a/applications/utilities/surface/surfaceTransformPoints/surfaceTransformPoints.C b/applications/utilities/surface/surfaceTransformPoints/surfaceTransformPoints.C index a2302190ce..85948f878d 100644 --- a/applications/utilities/surface/surfaceTransformPoints/surfaceTransformPoints.C +++ b/applications/utilities/surface/surfaceTransformPoints/surfaceTransformPoints.C @@ -135,16 +135,7 @@ List getScalingOpt(const word& optName, const argList& args) scaling.clear(); } - for (const scalar scale : scaling) - { - if (scale <= 0) - { - FatalError - << "Invalid scaling value, must be positive." << nl - << " -" << optName << ' ' << args[optName].c_str() << endl - << exit(FatalError); - } - } + // Zero and negative scaling are permitted return scaling; }