broulik added inline comments. INLINE COMMENTS
> mimetypesmodel.cpp:109 > { > - for (int i = 0; i < m_mimeTypesList.size(); i++) { > + for (int i = 0, total = m_mimeTypesList.size(); i < total; ++i) { > if (m_mimeTypesList.at(i).name() == name) { Isn't the compiler smart ehough? > previewpluginsmodel.cpp:112 > QStringList list; > - for (int i =0; i < m_checkedRows.size(); ++i) { > + for (int i =0, total = m_checkedRows.size(); i < total; ++i) { > if (m_checkedRows.at(i)) { Space after `=` REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D8473 To: mlaurent, hein Cc: broulik, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart