D10512: try to preload certain applets in a smart way

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> appletquickitem.cpp:789
> +        if (!d->applet->isContainment()) {
> +            applet()->config().writeEntry(QStringLiteral("PreloadWeight"), qMin(d->preloadWeight() + s_preloadWeigthIncrement, 100));
> +        }

So if you ever open the any applet slightly more than every alternate login (at any point during that session) over time it'll become blocking preloaded on boot?

Your login to desktop will just get slower and slower until it's ultimately loading everything.

---

I'm against doing any hacks without prior profiling.

If we find plasma-pa (for example) gains a lot at no cost, great! Add the entry.

If we find plasma-pa loaded in a nanosecond anyway, and takes up 10Mb of RAM to do so, then it's a very silly thing to do.

I'd like to see some numbers for every applet we add it to, I don't think the shell is in a position to do this automatically.

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D10512

To: mart, #plasma
Cc: davidedmundson, broulik, apol, ngraham, plasma-devel, #frameworks, michaelh, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180214/cb014a1b/attachment.html>