D9123: reuse KPackage instance between PluginLoader and Applet
David Edmundson
noreply at phabricator.kde.org
Mon Dec 4 11:20:42 UTC 2017
davidedmundson added inline comments.
INLINE COMMENTS
> applet.cpp:98
> : QObject(0),
> - d(new AppletPrivate(KPluginMetaData(), args.count() > 1 ? args[1].toInt() : 0, this))
> + d(new AppletPrivate(KPluginMetaData(), args.count() > 2 ? args[2].toInt() : 0, this))
> {
If anyone uses the old argument list syntax (where the first argument was a string or a kpluginmetadata) this will break.
Can we be sure that the PluginLoader was the only user?
It was a somewhat undocumented thing right?
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D9123
To: mart, #plasma
Cc: davidedmundson, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171204/cf5c9f57/attachment.html>
More information about the Kde-frameworks-devel
mailing list