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