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