No log message if already admitted

This commit is contained in:
Klaus Basan
2019-06-01 02:35:02 +02:00
committed by Mat Sutcliffe
parent 0250e5a7ad
commit f6e27bdf88
2 changed files with 16 additions and 10 deletions

View File

@@ -185,8 +185,10 @@ namespace BlackMisc
} }
else else
{ {
this->admitCacheImpl(sim); if (this->admitCacheImpl(sim))
CLogMessage(this).info(u"Admit model (%1) caches for %2") << this->getDescription() << simStr; {
CLogMessage(this).info(u"Admit model (%1) caches for %2") << this->getDescription() << simStr;
}
} }
} }
@@ -371,11 +373,11 @@ namespace BlackMisc
this->emitCacheChanged(simulator); // sync this->emitCacheChanged(simulator); // sync
} }
void CModelCaches::admitCacheImpl(const CSimulatorInfo &simulator) bool CModelCaches::admitCacheImpl(const CSimulatorInfo &simulator)
{ {
Q_ASSERT_X(simulator.isSingleSimulator(), Q_FUNC_INFO, "No single simulator"); Q_ASSERT_X(simulator.isSingleSimulator(), Q_FUNC_INFO, "No single simulator");
if (this->isCacheAlreadySynchronized(simulator)) { return; } if (this->isCacheAlreadySynchronized(simulator)) { return false; }
switch (simulator.getSimulator()) switch (simulator.getSimulator())
{ {
case CSimulatorInfo::FS9: m_modelCacheFs9.admit(); break; case CSimulatorInfo::FS9: m_modelCacheFs9.admit(); break;
@@ -387,6 +389,7 @@ namespace BlackMisc
Q_ASSERT_X(false, Q_FUNC_INFO, "wrong simulator"); Q_ASSERT_X(false, Q_FUNC_INFO, "wrong simulator");
break; break;
} }
return true;
} }
CModelSetCaches::CModelSetCaches(bool synchronizeCache, QObject *parent) : IMultiSimulatorModelCaches(parent) CModelSetCaches::CModelSetCaches(bool synchronizeCache, QObject *parent) : IMultiSimulatorModelCaches(parent)
@@ -401,8 +404,10 @@ namespace BlackMisc
} }
else else
{ {
this->admitCacheImpl(simulator); if (this->admitCacheImpl(simulator))
CLogMessage(this).info(u"Admit model (%1) caches for %2") << this->getDescription() << simStr; {
CLogMessage(this).info(u"Admit model (%1) caches for %2") << this->getDescription() << simStr;
}
} }
} }
@@ -556,11 +561,11 @@ namespace BlackMisc
this->emitCacheChanged(simulator); // sync this->emitCacheChanged(simulator); // sync
} }
void CModelSetCaches::admitCacheImpl(const CSimulatorInfo &simulator) bool CModelSetCaches::admitCacheImpl(const CSimulatorInfo &simulator)
{ {
Q_ASSERT_X(simulator.isSingleSimulator(), Q_FUNC_INFO, "No single simulator"); Q_ASSERT_X(simulator.isSingleSimulator(), Q_FUNC_INFO, "No single simulator");
if (this->isCacheAlreadySynchronized(simulator)) { return; } if (this->isCacheAlreadySynchronized(simulator)) { return false; }
switch (simulator.getSimulator()) switch (simulator.getSimulator())
{ {
case CSimulatorInfo::FS9: m_modelCacheFs9.admit(); break; case CSimulatorInfo::FS9: m_modelCacheFs9.admit(); break;
@@ -572,6 +577,7 @@ namespace BlackMisc
Q_ASSERT_X(false, Q_FUNC_INFO, "Wrong simulator"); Q_ASSERT_X(false, Q_FUNC_INFO, "Wrong simulator");
break; break;
} }
return true;
} }
} // ns } // ns
} // ns } // ns

View File

@@ -345,7 +345,7 @@ namespace BlackMisc
//! Non virtual version (can be used in ctor) //! Non virtual version (can be used in ctor)
//! \threadsafe //! \threadsafe
void admitCacheImpl(const CSimulatorInfo &simulator); bool admitCacheImpl(const CSimulatorInfo &simulator);
}; };
//! Bundle of caches for model sets of all simulators //! Bundle of caches for model sets of all simulators
@@ -386,7 +386,7 @@ namespace BlackMisc
//! Non virtual version (can be used in ctor) //! Non virtual version (can be used in ctor)
//! \threadsafe //! \threadsafe
void admitCacheImpl(const CSimulatorInfo &simulator); bool admitCacheImpl(const CSimulatorInfo &simulator);
}; };
//! One central instance of the caches base class //! One central instance of the caches base class