<div dir="ltr"><div>I opened an issue in line with the new kdereview process</div><div><br></div><div><a href="https://invent.kde.org/libraries/kcgroups/-/issues/1">https://invent.kde.org/libraries/kcgroups/-/issues/1</a></div><div><br></div><div>Jonathan</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 21 Nov 2020 at 00:38, Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">El divendres, 20 de novembre de 2020, a les 14:55:16 CET, Henri Chain va escriure:<br>
> Hello everyone,<br>
> <br>
> KCgroups has been moved to KDEReview !<br>
> What is that, you ask ? It's a library that wraps the systemd dbus API to <br>
> expose a higher-level concept of desktop application and allow control of its <br>
> system resource usage (CPU, RAM, IO, etc).<br>
> <br>
> It relies on the recent ability of plasma to launch applications in their own <br>
> systemd scopes, with correspond to cgroups and provides a more robust <br>
> definition for an application (more details at <a href="https://lwn.net/Articles/834329/" rel="noreferrer" target="_blank">https://lwn.net/Articles/834329/</a> <br>
> ) .<br>
> <br>
> The main use of the library is to expose related resource control settings for <br>
> those applications, at a user space level that other KDE applications and <br>
> frameworks can use, including consumption straight from QML as demonstrated in <br>
> the test application.<br>
> <br>
> KCgroups is intended to become a (Tier 1) framework. A first user of this <br>
> library might be the foreground window CPU booster daemon that is available <br>
> here: <a href="https://invent.kde.org/libraries/kcgroups/-/tree/work/foreground-booster" rel="noreferrer" target="_blank">https://invent.kde.org/libraries/kcgroups/-/tree/work/foreground-booster</a><br>
> <br>
> Packages are already available for both Neon and Arch Linux.<br>
> <br>
> Looking forward to your feedback and ideas for using this,<br>
<br>
I'm a bit scared about your optional class being there all in the main namespace. I'd suggest putting in some "namespace kcgroups{}" or name it kcgoptional or something.<br>
<br>
you have a few properties without NOTIFY, ideally you should either add it if they can change or mark them as CONSTANT if they can't.<br>
<br>
Cheers,<br>
  Albert<br>
<br>
<br>
> Henri<br>
> <br>
> <br>
> <br>
<br>
<br>
<br>
<br>
</blockquote></div>