D6212: Don't crash when deleted during network operations

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> desktopicon.cpp:333
> +        bool isDestroyed = false;
> +        connect(this, &QObject::destroyed, this, [&isDestroyed](){ isDestroyed = true; });
>          QByteArray data;

Normal convention for handling nested event loops (which is effectively what this is) is making a QPointer of "this" and checking that

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D6212

To: leinir, #kirigami, mart, apol
Cc: davidedmundson, plasma-devel, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170613/e1ee7423/attachment-0001.html>