D22952: Kirigami.Icon: Fix loading bigger images when source is a URL
Kai Uwe Broulik
noreply at phabricator.kde.org
Tue Aug 6 07:38:11 BST 2019
broulik added inline comments.
INLINE COMMENTS
> icon.cpp:406
> + m_networkReply = qnam->get(request);
> + connect(m_networkReply.data(), &QNetworkReply::finished, this, [this](){handleFinished(m_networkReply); });
> }
Just noticed, doesn't the `m_networkReply` leak? It's not deleted in `handleFinished`
REPOSITORY
R169 Kirigami
REVISION DETAIL
https://phabricator.kde.org/D22952
To: apol, #frameworks, #kirigami
Cc: broulik, plasma-devel, fbampaloukas, domson, dkardarakos, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190806/0a1357d0/attachment.html>
More information about the Kde-frameworks-devel
mailing list