[Differential] [Accepted] D3906: [Icon Applet] Fully re-populate when user changes Link URL

dfaure (David Faure) noreply at phabricator.kde.org
Mon Jan 2 09:21:24 UTC 2017


dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> iconapplet.cpp:369
> +            QFile::remove(m_localPath);
> +            setUrl(QUrl(desktopFile.readUrl())); // calls populate() itself
> +        } else {

It looks like you're reading from a deleted file here ;) But that's not a problem, it works since KDesktopFile (KConfig) caches everything in memory. So it's fine, don't change it, it's just funny to read.

REPOSITORY
  R120 Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, dfaure
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170102/48ba1450/attachment.html>


More information about the Plasma-devel mailing list