OpenFOAM: Rationalized the naming of scalar limits
In early versions of OpenFOAM the scalar limits were simple macro replacements and the
names were capitalized to indicate this. The scalar limits are now static
constants which is a huge improvement on the use of macros and for consistency
the names have been changed to camel-case to indicate this and improve
readability of the code:
GREAT -> great
ROOTGREAT -> rootGreat
VGREAT -> vGreat
ROOTVGREAT -> rootVGreat
SMALL -> small
ROOTSMALL -> rootSmall
VSMALL -> vSmall
ROOTVSMALL -> rootVSmall
The original capitalized are still currently supported but their use is
deprecated.
This commit is contained in:
@ -143,7 +143,7 @@ bool Foam::checkWedges
|
||||
const point& pt = p[pp.meshPoints()[i]];
|
||||
scalar d = mag((pt - p0) & pp.n());
|
||||
|
||||
if (d > ROOTSMALL)
|
||||
if (d > rootSmall)
|
||||
{
|
||||
if (report)
|
||||
{
|
||||
@ -176,7 +176,7 @@ bool Foam::checkWedges
|
||||
vector d(p[p1]-p[p0]);
|
||||
scalar magD = mag(d);
|
||||
|
||||
if (magD > ROOTVSMALL)
|
||||
if (magD > rootVSmall)
|
||||
{
|
||||
d /= magD;
|
||||
|
||||
@ -666,7 +666,7 @@ Foam::label Foam::checkGeometry
|
||||
|
||||
{
|
||||
faceSet faces(mesh, "wrongOrientedFaces", mesh.nFaces()/100 + 1);
|
||||
if (mesh.checkFacePyramids(true, -SMALL, &faces))
|
||||
if (mesh.checkFacePyramids(true, -small, &faces))
|
||||
{
|
||||
noFailedChecks++;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user