broulik added a comment.
> why don't we just change the check to QIcon::isNull? We still want to serialize custom pixmap data icons which won't be null. What we want to do is tell between "icon on disk" and "icon from pixmap data" which we can't really at the moment. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D25762 To: broulik, #plasma, hein Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart