OilfieldToolsNet/opm-common
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use const reference instead of copying map.

Browse Source 
Also use at() instead of operator[], the at() is
guaranteed to succeed by the checks of the lines above it.
This commit is contained in:
Atgeirr Flø Rasmussen
2018-06-29 15:51:57 +02:00
parent ca79b16b26
commit 9d5a43d4e6
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/opm/parser/eclipse/EclipseState/Grid/MULTREGTScanner.cpp
View File

@@ -235,19 +235,18 @@ namespace Opm {
for (auto iter = m_searchMap.begin(); iter != m_searchMap.end(); iter++) {
const Opm::GridProperty<int>& region = m_e3DProps.getIntGridProperty( (*iter).first );
MULTREGTSearchMap map = (*iter).second;
const MULTREGTSearchMap& map = (*iter).second;
int regionId1 = region.iget(globalIndex1);
int regionId2 = region.iget(globalIndex2);
std::pair<int,int> pair{regionId1 , regionId2};
std::pair<int,int> pair{ regionId1, regionId2 };
if (map.count(pair) != 1 || !(map.at(pair)->m_directions & faceDir)) {
pair = std::pair<int,int>{regionId2 , regionId1};
if (map.count(pair) != 1 || !(map.at(pair)->m_directions & faceDir))
continue;
}
const MULTREGTRecord * record = map[pair];
const MULTREGTRecord* record = map.at(pair);
bool applyMultiplier = true;
int i1 = globalIndex1 % region.getNX();