STYLE: use HashSet xor instead of two operations (faceZoneSet)

STYLE: use global operator instead of HashSet -= operator
This commit is contained in:
Mark Olesen
2020-07-24 18:16:55 +02:00
parent 6e75cf2e7d
commit d5884c8d85
2 changed files with 6 additions and 13 deletions

View File

@ -166,8 +166,7 @@ int main(int argc, char *argv[])
if (args.found("writeFields")) if (args.found("writeFields"))
{ {
selectedFields = args.getList<word>("writeFields"); selectedFields = args.getList<word>("writeFields");
wordHashSet badFields(selectedFields); const wordHashSet badFields(selectedFields - allFields);
badFields -= allFields;
if (!badFields.empty()) if (!badFields.empty())
{ {
@ -229,7 +228,7 @@ int main(int argc, char *argv[])
IOobject::MUST_READ, IOobject::MUST_READ,
IOobject::NO_WRITE IOobject::NO_WRITE
) )
) )
); );
} }

View File

@ -320,17 +320,11 @@ void Foam::faceZoneSet::sync(const polyMesh& mesh)
{ {
const labelHashSet zoneSet(addressing_); const labelHashSet zoneSet(addressing_);
// Get elements that are in zone but not faceSet // Elements that are in zone but not faceSet, and
labelHashSet badSet(zoneSet); // elements that are in faceSet but not in zone
badSet -= *this; labelHashSet badSet(*this ^ zoneSet);
// Add elements that are in faceSet but not in zone const label nBad = returnReduce(badSet.size(), sumOp<label>());
labelHashSet fSet(*this);
fSet -= zoneSet;
badSet += fSet;
label nBad = returnReduce(badSet.size(), sumOp<label>());
if (nBad) if (nBad)
{ {