Simplify code a bit

This commit is contained in:
Dennis Felsing 2018-07-27 07:51:16 +02:00
parent 4f18d7f9b8
commit 409abf1b90

View file

@ -353,23 +353,24 @@ void CAutoMapper::Proceed(CLayerTiles *pLayer, int ConfigID)
if(pRule->m_Value == CPosRule::INDEX) if(pRule->m_Value == CPosRule::INDEX)
{ {
bool PosRuleTest = false; RespectRules = false;
for(int i = 0; i < pRule->m_aIndexList.size() && !PosRuleTest; ++i) { for(int i = 0; i < pRule->m_aIndexList.size(); ++i) {
if(CheckIndex == pRule->m_aIndexList[i].m_ID && (pRule->m_aIndexList[i].m_Flag == -1 || CheckFlags == pRule->m_aIndexList[i].m_Flag)) if(CheckIndex == pRule->m_aIndexList[i].m_ID && (pRule->m_aIndexList[i].m_Flag == -1 || CheckFlags == pRule->m_aIndexList[i].m_Flag))
PosRuleTest = true; {
RespectRules = true;
break;
}
} }
if(!PosRuleTest)
RespectRules = false;
} }
else if(pRule->m_Value == CPosRule::NOTINDEX) else if(pRule->m_Value == CPosRule::NOTINDEX)
{ {
bool PosRuleTest = true; for(int i = 0; i < pRule->m_aIndexList.size(); ++i) {
for(int i = 0; i < pRule->m_aIndexList.size() && PosRuleTest; ++i) {
if(CheckIndex == pRule->m_aIndexList[i].m_ID && (pRule->m_aIndexList[i].m_Flag == -1 || CheckFlags == pRule->m_aIndexList[i].m_Flag)) if(CheckIndex == pRule->m_aIndexList[i].m_ID && (pRule->m_aIndexList[i].m_Flag == -1 || CheckFlags == pRule->m_aIndexList[i].m_Flag))
PosRuleTest = false; {
RespectRules = false;
break;
}
} }
if(!PosRuleTest)
RespectRules = false;
} }
} }