<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 18, 2015 at 8: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:1px solid rgb(204,204,204);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?</blockquote><div><br></div><div>Thanks for doing all that.<br></div><div><br></div><div>Now to make you hate me.<br></div><div>I have a crash, <a href="https://paste.kde.org/ppeqjl1c1">https://paste.kde.org/ppeqjl1c1</a><br><br></div><div>That's running:<br>kdeclarative - your branch<br>plasma-framework - your branch<br></div><div>plsama-workspace - master<br></div><div>(which is pretty close to running latest frameworks with Plasma 5.3)<br></div><div><br></div><div>It's possibly unrelated, but I switched the top two back to master and it went away.<br></div><br><br><br><div>Other than that, I think code-wise from me it's nearly good to go.<br></div><br><div>Can I check it's these<br><a href="https://git.reviewboard.kde.org/r/123736/">https://git.reviewboard.kde.org/r/123736/</a><br></div><div>and<br><a href="https://git.reviewboard.kde.org/r/123735/">https://git.reviewboard.kde.org/r/123735/</a><br><br></div><div>and the branches in p-w, p-d, are just changing the metadata?<br></div><div><br></div><div>in plasma-workspace TaskDelegate.qml has an import line removed, is that intended?<br></div><div><br><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
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></blockquote><div>Lets not worry about that for now, we've got from super many -> just a few.  <br></div><div>Better to get this stuff merged, than worry about getting it down to 1.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
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>
<br>
but I'm leaning more on the direction of something like<br>
X-Plasma-MinimumAPIVersion=5.11<br></blockquote><br><br><br></div><div class="gmail_quote">Out of curiosity, which plasmoids actually need their own engine?<br></div><div class="gmail_quote"></div><div class="gmail_quote"></div><div class="gmail_quote"><br></div><div class="gmail_quote"></div><div class="gmail_quote">Were they all changed as a bulk find & replace?</div><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I would like to have all set before frameworks 5.11<br></blockquote><div><br></div>So that gives us roughly 2 weeks of testing.<br></div><div class="gmail_quote"><br><div></div><div>Without the plasma-workspace changes coming in 5.4 it doesn't really have any benefit to the user.<br></div>Being a massively miserable grumpy git, I'd rather merge just after 5.11, giving us 4 weeks of testing.<br><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<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>