<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24966">View Revision</a></tr></table><br /><div><div><p>To leave KAboutData::programIconData as a deprecated property and instead turn to use KPluginMetaData where the iconName property is undisputed in its usefullnes, I have now uploaded 3 patches for view;</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><a href="https://phabricator.kde.org/D25063" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25063</a>: Add KAboutPluginDialog, to be used with KPluginMetaData</li>
<li class="remarkup-list-item"><a href="https://phabricator.kde.org/D25059" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25059</a>: KPluginSelector: use new KAboutPluginDialog</li>
<li class="remarkup-list-item"><a href="https://phabricator.kde.org/D25063" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25063</a>: Deprecate KAboutData::fromPluginMetaData, now there is KAboutPluginDialog</li>
</ul>

<p>Now, there is still the idea of using KAboutData with plugins in the KAboutData API, by the methods <tt style="background: #ebebeb; font-size: 13px;">void KAboutData::registerPluginData(const KAboutData &aboutData)</tt> & <tt style="background: #ebebeb; font-size: 13px;"> KAboutData *KAboutData::pluginData(const QString &componentName)</tt>, By what lxr.kde.org reports,  the register method is still in use, but nothing seems to actually query the registration data by the other method. So possibly we could discard that pair of methods as well. Would be curious though what the usecase for this once was, it surely had a piurpose.</p>

<p>So, IMHO the current patch as proposed here should not be done, instead we should just silence the warnings via push/pop of compiler settings stash. We still need to support any legacy code out there which still relies on setting the icon via KAboutData::setApplicationData()). Which actually might also get unnoticed with code only tested with Wayland, as QGuiApplication::setWindowIcon is without effect there and the window icon actually fetched via the desktop file metadata entry. And we actually have a problem if no icon is provided by either way... Hm... possibly we need to resurrect KAboutData::programIconName for this wayland-only future, so the program itself has the same set of metadata about itself as the shell has...  though this needs QCoreApplication to have a way to support us here, so any metaddata does not get out-of-sync...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24966">https://phabricator.kde.org/D24966</a></div></div><br /><div><strong>To: </strong>dfaure, kossebau, elvisangelaccio, vkrause, jriddell<br /><strong>Cc: </strong>jriddell, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>