From 0767409988623e232b83ff72f9022a7d16460028 Mon Sep 17 00:00:00 2001 From: Mark Olesen Date: Wed, 6 Feb 2019 12:01:29 +0100 Subject: [PATCH] ENH: support objectRegistry::checkOut(const word& name) - similar to what erase() does, but as a mutable operation (#1180) - replace basicThermo lookupAndCheckout (commit 880c98757d91) with the new objectRegistry::checkOut() method. --- .../db/objectRegistry/objectRegistry.C | 6 +++++ .../db/objectRegistry/objectRegistry.H | 4 ++++ .../fieldAverageItem/fieldAverageItem.C | 17 ++++++------- src/functionObjects/field/momentum/momentum.C | 8 +++---- .../surfMeshSamplers/surfMeshSamplers.C | 24 ++++--------------- .../surfMeshSamplers/surfMeshSamplers.H | 8 ------- .../basic/basicThermo/basicThermo.C | 11 +-------- .../basic/basicThermo/basicThermo.H | 5 +--- 8 files changed, 27 insertions(+), 56 deletions(-) diff --git a/src/OpenFOAM/db/objectRegistry/objectRegistry.C b/src/OpenFOAM/db/objectRegistry/objectRegistry.C index e5ba7cf781..cb5965df22 100644 --- a/src/OpenFOAM/db/objectRegistry/objectRegistry.C +++ b/src/OpenFOAM/db/objectRegistry/objectRegistry.C @@ -285,6 +285,12 @@ bool Foam::objectRegistry::checkOut(regIOobject& io) const } +bool Foam::objectRegistry::checkOut(const word& key) const +{ + return const_cast(*this).erase(key); +} + + void Foam::objectRegistry::clear() { // Free anything owned by the registry diff --git a/src/OpenFOAM/db/objectRegistry/objectRegistry.H b/src/OpenFOAM/db/objectRegistry/objectRegistry.H index af6c0a3450..1bb7248212 100644 --- a/src/OpenFOAM/db/objectRegistry/objectRegistry.H +++ b/src/OpenFOAM/db/objectRegistry/objectRegistry.H @@ -438,6 +438,10 @@ public: //- object is ownedByRegistry bool checkOut(regIOobject& io) const; + //- Remove a regIOobject by name from registry and frees memory if the + //- object is ownedByRegistry + bool checkOut(const word& key) const; + //- Clear all entries from the registry // Performs a checkOut() for all objects that are ownedByRegistry void clear(); diff --git a/src/functionObjects/field/fieldAverage/fieldAverageItem/fieldAverageItem.C b/src/functionObjects/field/fieldAverage/fieldAverageItem/fieldAverageItem.C index 8ac5a9dafe..7864b12bbb 100644 --- a/src/functionObjects/field/fieldAverage/fieldAverageItem/fieldAverageItem.C +++ b/src/functionObjects/field/fieldAverage/fieldAverageItem/fieldAverageItem.C @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2009-2011, 2017 OpenCFD Ltd. + \\ / A nd | Copyright (C) 2009-2011, 2017-2019 OpenCFD Ltd. \\/ M anipulation | ------------------------------------------------------------------------------- | Copyright (C) 2011-2016 OpenFOAM Foundation @@ -152,7 +152,7 @@ void Foam::functionObjects::fieldAverageItem::evolve(const objectRegistry& obr) const word fieldName = windowFieldNames_.pop(); //Info<< "evolve: removing field: " << fieldName << endl; - obr.checkOut(*obr[fieldName]); + obr.checkOut(fieldName); } } } @@ -164,22 +164,19 @@ void Foam::functionObjects::fieldAverageItem::clear bool fullClean ) { - if (mean_ && obr.found(meanFieldName_)) + if (mean_) { - obr.checkOut(*obr[meanFieldName_]); + obr.checkOut(meanFieldName_); } - if (prime2Mean_ && obr.found(prime2MeanFieldName_)) + if (prime2Mean_) { - obr.checkOut(*obr[prime2MeanFieldName_]); + obr.checkOut(prime2MeanFieldName_); } for (const word& fieldName : windowFieldNames_) { - if (obr.found(fieldName)) - { - obr.checkOut(*obr[fieldName]); - } + obr.checkOut(fieldName); } if (totalTime_ < 0 || fullClean) diff --git a/src/functionObjects/field/momentum/momentum.C b/src/functionObjects/field/momentum/momentum.C index 342b46eb97..260253bf00 100644 --- a/src/functionObjects/field/momentum/momentum.C +++ b/src/functionObjects/field/momentum/momentum.C @@ -46,11 +46,9 @@ namespace functionObjects void Foam::functionObjects::momentum::purgeFields() { - objectRegistry& obr = const_cast(obr_); - - obr.erase(scopedName("momentum")); - obr.erase(scopedName("angularMomentum")); - obr.erase(scopedName("angularVelocity")); + obr_.checkOut(scopedName("momentum")); + obr_.checkOut(scopedName("angularMomentum")); + obr_.checkOut(scopedName("angularVelocity")); } diff --git a/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.C b/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.C index 5cf3391d5d..42cc4509b6 100644 --- a/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.C +++ b/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.C @@ -49,24 +49,6 @@ namespace Foam bool Foam::surfMeshSamplers::verbose_ = false; -void Foam::surfMeshSamplers::checkOutNames -( - const objectRegistry& registry, - const UList& names -) -{ - objectRegistry& reg = const_cast(registry); - - for (const word& fldName : names) - { - objectRegistry::iterator iter = reg.find(fldName); - if (iter.found()) - { - registry.checkOut(*iter()); - } - } -} - // * * * * * * * * * * * * * Private Member Functions * * * * * * * * * * * // @@ -297,7 +279,11 @@ bool Foam::surfMeshSamplers::execute() } } - checkOutNames(db, cleanup); + // Cleanup any locally introduced names + for (const word& fieldName : cleanup) + { + db.checkOut(fieldName); + } return true; } diff --git a/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.H b/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.H index 19cd1bb8fa..ecd729dabe 100644 --- a/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.H +++ b/src/sampling/surfMeshSample/surfMeshSamplers/surfMeshSamplers.H @@ -157,14 +157,6 @@ class surfMeshSamplers // Private Member Functions - //- Remove items by name from objectRegistry - static void checkOutNames - ( - const objectRegistry& registry, - const UList& names - ); - - //- Hard-coded derived field (rho * U) // \return true if field did not previously exist bool add_rhoU(const word& derivedName); diff --git a/src/thermophysicalModels/basic/basicThermo/basicThermo.C b/src/thermophysicalModels/basic/basicThermo/basicThermo.C index dd4909dfd4..88341dc0bb 100644 --- a/src/thermophysicalModels/basic/basicThermo/basicThermo.C +++ b/src/thermophysicalModels/basic/basicThermo/basicThermo.C @@ -160,15 +160,6 @@ Foam::volScalarField& Foam::basicThermo::lookupOrConstruct } -void Foam::basicThermo::lookupAndCheckout(const char* name) const -{ - if (db().foundObject(name)) - { - db().checkOut(*db()[name]); - } -} - - Foam::basicThermo::basicThermo ( const fvMesh& mesh, @@ -326,7 +317,7 @@ Foam::autoPtr Foam::basicThermo::New Foam::basicThermo::~basicThermo() { - lookupAndCheckout("p"); + db().checkOut("p"); } diff --git a/src/thermophysicalModels/basic/basicThermo/basicThermo.H b/src/thermophysicalModels/basic/basicThermo/basicThermo.H index bb98338380..337373c685 100644 --- a/src/thermophysicalModels/basic/basicThermo/basicThermo.H +++ b/src/thermophysicalModels/basic/basicThermo/basicThermo.H @@ -2,7 +2,7 @@ ========= | \\ / F ield | OpenFOAM: The Open Source CFD Toolbox \\ / O peration | - \\ / A nd | Copyright (C) 2004-2010, 2017 OpenCFD Ltd. + \\ / A nd | Copyright (C) 2004-2010, 2017-2019 OpenCFD Ltd. \\/ M anipulation | ------------------------------------------------------------------------------- | Copyright (C) 2011-2017 OpenFOAM Foundation @@ -105,9 +105,6 @@ protected: bool& isOwner ); - //- Lookup and check out field - void lookupAndCheckout(const char* name) const; - //- Return the enthalpy/internal energy field boundary types // by interrogating the temperature field boundary types wordList heBoundaryTypes();