Review Request 129102: Don't enforce metadata.desktop, cleanup constructor

[David Edmundson]

> On Oct. 10, 2016, 11:23 a.m., Marco Martin wrote:
> > src/plasma/pluginloader.cpp, line 561
> > <https://git.reviewboard.kde.org/r/129102/diff/4/?file=481803#file481803line561>
> >
> >     this note isn't valid anymore?

yes and no.

It was, even in Plasma 5.8 the plasmoids runner used it. 

After all of Aleix's patches it is not valid, but changing it would crash all previous releases.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129102/#review99902
-----------------------------------------------------------


On Oct. 7, 2016, 3:23 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129102/
> -----------------------------------------------------------
> 
> (Updated Oct. 7, 2016, 3:23 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Makes it possible to use plugins that offer a metadata.json file.
> Define the service type when falling back to the desktop file parser, so the type system is proper.
> Don't destroy a KPluginMetadata tuple to instanciate it right away.
> 
> 
> Diffs
> -----
> 
>   src/plasma/applet.cpp 5eb529c 
>   src/plasma/containment.h 0b7a3ef 
>   src/plasma/containment.cpp 1840f24 
>   src/plasma/pluginloader.h 566461d 
>   src/plasma/pluginloader.cpp db2b2c9 
>   src/plasma/private/applet_p.cpp fafb450 
> 
> Diff: https://git.reviewboard.kde.org/r/129102/diff/
> 
> 
> Testing
> -------
> 
> Tests still pass, plasma still loads, even with RR #129103.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161010/a1bedc71/attachment.html>