mwolff accepted this revision.
mwolff added a comment.
This revision is now accepted and ready to land.
one minor nit, otherwise lgtm
INLINE COMMENTS
> kfileplacesmodel.cpp:431
> + QModelIndexList indexes;
> + for (int row = 0; row < rowCount(); ++row) {
> + const QModelIndex current = index(row, 0);
only call rowCount once:
for (int row = 0, c = rowCount(); row < c; ++row) {
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D8450
To: franckarrecot, ngraham, renatoo, ervin, mwolff, mlaurent
Cc: mwolff, #frameworks
