refs #624 CLogPattern can use Mixin::HashByTuple because QSet is hashable since Qt 5.5.

This commit is contained in:
Mathew Sutcliffe
2016-03-20 00:42:46 +00:00
parent 4df0f150de
commit 23a7f9b719

View File

@@ -27,6 +27,7 @@ namespace BlackMisc
*/
class BLACKMISC_EXPORT CLogPattern :
public Mixin::MetaType<CLogPattern>,
public Mixin::HashByTuple<CLogPattern>,
public Mixin::EqualsByTuple<CLogPattern>,
public Mixin::DBusOperators<CLogPattern>,
public Mixin::Index<CLogPattern>,
@@ -92,15 +93,6 @@ namespace BlackMisc
//! \copydoc BlackMisc::Mixin::DBusByTuple::unmarshallFromDbus()
void unmarshallFromDbus(const QDBusArgument &argument);
//! \copydoc CValueObject::qHash
//! \todo Use Mixin::HashByTuple when Qt 5.5 is baseline (qHash(QSet<T>) was added in Qt 5.5).
friend uint qHash(const CLogPattern &p, uint seed = 0)
{
seed = std::accumulate(p.m_severities.begin(), p.m_severities.end(), seed, [](uint i, CStatusMessage::StatusSeverity s) { return i + (1 << static_cast<int>(s)); });
seed ^= std::accumulate(p.m_strings.begin(), p.m_strings.end(), 0, [](uint i, const QString &s) { return i + qHash(s); });
return qHash(static_cast<int>(p.m_strategy), seed);
}
private:
bool checkInvariants() const;