QList<LogChangePathEntry> -> QVector<LogChangePathEntry>
Review Request #126394 - Created Dec. 16, 2015 and submitted
- changed QList<LogChangePathEntry> to QVector<LogChangePathEntry> - the vector is only initialized once, sometimes appended but no insert/delete
- The class ByteToString was useless and can be replaced by a simple function
- added const to some member functions
- removed some useless 'break' statements
used it the last two days at work without any problems.
The commit message is confusing though. Even if there were insert/deletes, switching from QList to QVector would have been a good thing. Remember that QList is not a linked list, but a vector.... So the main reason for switching is that sizeof(LogChangePathEntry) > sizeof(void *), in which case QList is very inefficient.