fvMeshSubSet setLargeCellSubset is ambiguous

for symmetry with

void setLargeCellSubset
(
    const labelHashSet& globalCellMap,
    const label patchID = -1,
    const bool syncPar = true
);

introduced this last year to avoid the unnecessary overhead of creating a large labelHashSet from a labelList.

void setLargeCellSubset
(
    const labelList& globalCellMap,
    const label patchID = -1,
    const bool syncPar = true
);

However this causes an ambiguity with another setLargeCellSubset.

void setLargeCellSubset
(
    const labelList& region,
    const label currentRegion,
    const label patchID = -1,
    const bool syncPar = true
);

Another name for the first version?
swap parameter order for the second versions? - setLargeCellSubset(const label currentRegion, const labelList& region)

Thoughts/opinions? @Mattijs @andy

Get rid of void setLargeCellSubset ( const labelList& region,

Get rid of the setCellSubset since does not run in parallel.

mentioned in commit f2f25336801c194d6c8207bb0048c0d6f8eb73f2

mentioned in commit 239087f11209c384e5018486994167d755fb1ee2

mentioned in commit 092695d242c723340260fb99e1127d876df0dc5e

closed

mentioned in commit 7a80ddc6

mentioned in commit cf7dbf4d

Child items