D6718: Fix occasional abort in the krunner plugin
Fabian Vogt
noreply at phabricator.kde.org
Sun Jul 16 09:22:25 UTC 2017
fvogt added a comment.
In https://phabricator.kde.org/D6718#125807, @davidedmundson wrote:
> This shouldn't be needed.
> Even an empty QList<QVariant> will have an argument of that empty QList<QVariant> and errors will have a different type.
I thought so too, until I got crashes without the test. IMO it can't hurt to prevent crashes due to misbehaviour of other components anyway.
REPOSITORY
R856 Plasma Browser Integration
REVISION DETAIL
https://phabricator.kde.org/D6718
To: fvogt, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170716/7fd7d00e/attachment.html>
More information about the Plasma-devel
mailing list