Review Request 129096: Check defaultActionsCfg is valid
David Edmundson
david at davidedmundson.co.uk
Tue Oct 4 15:16:04 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129096/
-----------------------------------------------------------
Review request for Plasma.
Repository: plasma-framework
Description
-------
Commit 752626ad550552232d73d8b009db62b0f8e86113 left a code path through
the switch statement where defaultActionsCfg would be an invalid group.
This is semantically correct as there is no relevant actions config for
this type of containment, however we then call methods on that config group which leads to an assert - and a failing unit test.
Diffs
-----
src/plasma/containment.cpp dde5dd366d56147ca3d24badb17a8639c833285b
Diff: https://git.reviewboard.kde.org/r/129096/diff/
Testing
-------
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161004/cc7a4c5a/attachment.html>
More information about the Plasma-devel
mailing list