Fix decrement index and not iterator as discussed with David
ClosedPublic

Authored by mlaurent on Dec 11 2019, 6:47 PM.

Details

Diff Detail

Repository
R320 KIO Extras
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mlaurent created this revision.Dec 11 2019, 6:47 PM
Restricted Application added projects: Dolphin, Frameworks. · View Herald TranscriptDec 11 2019, 6:47 PM
Restricted Application added subscribers: kfm-devel, kde-frameworks-devel. · View Herald Transcript
mlaurent requested review of this revision.Dec 11 2019, 6:47 PM
anthonyfieroni added inline comments.
network/kded/kioslavenotifier.cpp
114

Same here

mlaurent updated this revision to Diff 71322.Dec 11 2019, 8:45 PM

Fix other code

anthonyfieroni accepted this revision.Dec 12 2019, 6:55 PM
This revision is now accepted and ready to land.Dec 12 2019, 6:55 PM
dfaure accepted this revision.Dec 12 2019, 7:07 PM

We can thank Qt for deprecating QHash::iterator::operator-- which allowed Laurent to catch this :-)

This revision was automatically updated to reflect the committed changes.