diff --git a/src/blackcore/blackcorefreefunctions.cpp b/src/blackcore/blackcorefreefunctions.cpp index cb5d0075e..6462a5efb 100644 --- a/src/blackcore/blackcorefreefunctions.cpp +++ b/src/blackcore/blackcorefreefunctions.cpp @@ -15,15 +15,10 @@ namespace BlackCore { void registerMetadata() { - int id; - id = qRegisterMetaType(); - Q_ASSERT_X(id >= 1024, Q_FUNC_INFO, "wrong id for metatype"); - id = qRegisterMetaType(); - Q_ASSERT_X(id >= 1024, Q_FUNC_INFO, "wrong id for metatype"); - id = qRegisterMetaType(); - Q_ASSERT_X(id >= 1024, Q_FUNC_INFO, "wrong id for metatype"); - id = qRegisterMetaType(); - Q_UNUSED(id); + qRegisterMetaType(); + qRegisterMetaType(); + qRegisterMetaType(); + qRegisterMetaType(); } bool isCurrentThreadObjectThread(QObject *toBeTested) diff --git a/src/blackmisc/statusmessagelist.cpp b/src/blackmisc/statusmessagelist.cpp index 70ae09163..995a0b052 100644 --- a/src/blackmisc/statusmessagelist.cpp +++ b/src/blackmisc/statusmessagelist.cpp @@ -28,17 +28,18 @@ namespace BlackMisc bool CStatusMessageList::hasErrorMessages() const { - return findBySeverity(CStatusMessage::SeverityError).size() > 0; + return this->contains(&CStatusMessage::getSeverity, CStatusMessage::SeverityError); } bool CStatusMessageList::hasWarningMessages() const { - return findBySeverity(CStatusMessage::SeverityWarning).size() > 0; + return this->contains(&CStatusMessage::getSeverity, CStatusMessage::SeverityWarning); } bool CStatusMessageList::hasWarningOrErrorMessages() const { - return hasErrorMessages() || hasWarningMessages(); + return this->containsBy + ([ = ](const CStatusMessage & m) { return m.getSeverity() == CStatusMessage::SeverityWarning || m.getSeverity() == CStatusMessage::SeverityError; }); } void CStatusMessageList::addCategory(const CLogCategory &category)