D22952: Kirigami.Icon: Fix loading bigger images when source is a URL

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Aug 6 11:58:40 BST 2019


broulik added inline comments.

INLINE COMMENTS

> icon.cpp:412
> +{
> +    reply->deleteLater();
> +    if (!reply) {

Move this after the `!reply` check

> icon.cpp:424
> +    const QString filename = reply->url().fileName();
> +    if (!m_loadedImage.load(reply, filename.mid(filename.indexOf(QLatin1Char('.'))).toLatin1().constData())) {
> +        qWarning() << "received broken image" << reply->url();

What happens if you leave `format` `nullptr`, does it guess?

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/b526a96e/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list