Review Request 125277: Update recentApplicationCount in all places we modify applications list
David Edmundson
david at davidedmundson.co.uk
Wed Sep 16 23:29:27 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125277/
-----------------------------------------------------------
Review request for Plasma.
Repository: plasma-desktop
Description
-------
hopefully fixes linked crash
BUG: 352047
Diffs
-----
applets/kickoff/core/recentlyusedmodel.cpp 27af6a595bfed97292b43be6df46528b1e79f038
Diff: https://git.reviewboard.kde.org/r/125277/diff/
Testing
-------
Crash is in
while (d->recentApplicationCount != 0) {
QList<QStandardItem *>items = takeRow(0);
which implies at some point d->recentApplicationCount got messed up somewhere, so did a code review on that.
Seems to still work.
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150916/5119897a/attachment.html>
More information about the Plasma-devel
mailing list