meven added a comment.
In D27965#625381 <https://phabricator.kde.org/D27965#625381>, @apol wrote: > Looks good to me, I wonder why you turned some to for+iterators. Those modified the list as they iterated through the list : `authList->removeOne(current);` REPOSITORY R241 KIO BRANCH l-kpasswdserver (branched from master) REVISION DETAIL https://phabricator.kde.org/D27965 To: ahmadsamir, #frameworks, dfaure, meven Cc: apol, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
