D27955: KCM/Compositing: Use KConfig XT to store values

zzag added a comment.


  Overall, +1.

INLINE COMMENTS

> compositing.cpp:53
>      , m_compositingInterface(new OrgKdeKwinCompositingInterface(QStringLiteral("org.kde.KWin"), QStringLiteral("/Compositor"), QDBusConnection::sessionBus(), this))
> -    , m_config(KSharedConfig::openConfig("kwinrc"))
> -{
> -    reset();
> -    connect(this, &Compositing::animationSpeedChanged,       this, &Compositing::changed);
> -    connect(this, &Compositing::windowThumbnailChanged,      this, &Compositing::changed);
> -    connect(this, &Compositing::glScaleFilterChanged,        this, &Compositing::changed);
> -    connect(this, &Compositing::xrScaleFilterChanged,        this, &Compositing::changed);
> -    connect(this, &Compositing::glSwapStrategyChanged,       this, &Compositing::changed);
> -    connect(this, &Compositing::compositingTypeChanged,      this, &Compositing::changed);
> -    connect(this, &Compositing::compositingEnabledChanged,   this, &Compositing::changed);
> -    connect(this, &Compositing::openGLPlatformInterfaceChanged, this, &Compositing::changed);
> -    connect(this, &Compositing::windowsBlockCompositingChanged, this, &Compositing::changed);
> -
> -    connect(this, &Compositing::changed, [this]{
> -        m_changed = true;
> -    });
> +    , m_settings(new KWinCompositingSetting(parent))
> +{

Shouldn't the parent be `this`?

> compositing.h:104-107
> +    void updateSettings();
> +
>  private:
> +    void applyValues();

naming nit: could you please rename these methods to something else? It's unclear what values `applyValues` applies.

What about these?

  void updateSettingsFromUi();
  void updateUiFromSettings();

Feel free to ignore this comment because it looks like these methods will go away in D27988 <https://phabricator.kde.org/D27988>

REPOSITORY
  R108 KWin

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

To: meven, #kwin, ervin, bport, crossi, hchain, zzag
Cc: zzag, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart