Review Request: Properly handle deletion of stale sessions (using their lock status).

Ivan Shapovalov intelfx100 at gmail.com
Fri Oct 12 19:25:45 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106802/
-----------------------------------------------------------

Review request for KDevelop.


Description
-------

In one of the places (empty session cleanup), uncomment session deletion and use tryLockSession() to determine whether it is actually possible.
In another one (session chooser dialog), wrap the existing session deletion into a tryLockSession() conditional.


Diffs
-----

  shell/sessioncontroller.cpp c9fac67 

Diff: http://git.reviewboard.kde.org/r/106802/diff/


Testing
-------

Made sure that the sessions are cleaned correctly.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121012/0e2d2161/attachment.html>


More information about the KDevelop-devel mailing list