Avoids unnecessary conversions from QDate to QString, since QDate does
have a qHash() function in Qt5. Also might save some bytes per each event
as QDate is more lightweight than storing a QString representation of a
date.
Details
Details
Diff Detail
Diff Detail
- Branch
- memorycalendar-qdate-hash
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 18528 Build 18546: arc lint + arc unit
src/memorycalendar.cpp | ||
---|---|---|
108 | Documentation needs to be updated |
Comment Actions
Ouch, good find! Might even make sense to turn this from QMap<IncidenceBase::IncidenceType, QMultiHash<QDate, IncidenceBase::Ptr>> into QMultiHash<QDate, Incidence::Ptr>[4] in a second step.