<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 18, 2015 at 9:28 PM, Marco Martin <span dir="ltr"><<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi all,<br>
<br>
I think the branches for the single shared qqmlengine are pretty much ready<br>
(few cleanups upcoming days), seems pretty stable here.. did someone ran the<br>
branch for a while as well?<br>
<br>
In the end i managed to get a single engine for the whole session, views<br>
included (had to duplicate the View class in plasmaquick and kept the old one<br>
as deprecated for retrocompatibility unfortunately)<br>
<br>
the memory save seems pretty good, i *hope* stability improved as well :p<br>
<br>
what still uses separed engines are the applet configuration dialogs: this is<br>
necessary because they are supposed to use a different style for the<br>
qquickcontrols, and being the settings object that decides the style a qml<br>
singleton (qml singletons are unique by engine), the engine has to be<br>
different from the desktop/panel. The good thing is that config dialogs are<br>
instantiated relatively rarely, in most sessions not at all, so shouldn't<br>
touch too much a "normal run"<br>
<br>
For retrocompatibility the applets unfortunately have to specify explicitly<br>
they can share the engine in their metadata file (or eventual plasmapackage:/<br>
urls break)<br>
<br>
at the moment it's using<br>
X-Plasma-RequiredExtensions=SharedEngine<br></blockquote><div><br></div><div>What does "RequiredExtensions" mean anyway? Is that a new attribute where you intent to add more "extensions" or does it already exist?</div><div>It's name sounds slightly confusing to me. How can a SharedEngine be a "extension"?</div><div><br></div><div>Anyway, with that attribute you let the applet developer OPT-IN in for shared engine. Setting that attribute can be easily forgotten. If anything, they should OPT-OUT thus by default use the shared engine.<br></div><div>Perhaps a attribute like this is more appropriate?:</div><div>X-Plasma-RequiresOwnQmlEngine=true</div><div><br></div><div>or something alike.</div><div><br></div><div>I'd definitely go for OPT-OUT (defaults = use shared engine).</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
but I'm leaning more on the direction of something like<br>
X-Plasma-MinimumAPIVersion=5.11<br>
<br>
I would like to have all set before frameworks 5.11<br>
<br>
thoughts?<br>
<span class=""><font color="#888888"><br>
--<br>
Marco Martin<br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</font></span></blockquote></div><br></div></div>