D26040: [KCM/Effects] Allow the user to know when the settings are set to default
Kevin Ottens
noreply at phabricator.kde.org
Mon Dec 23 11:09:46 GMT 2019
ervin added inline comments.
INLINE COMMENTS
> zzag wrote in effectsmodel.cpp:572
> Agreed, using it in modern C++ code is not good but on the other hand we have `(bool)effect.status != effect.enabledByDefault` just right above this method. If my memory serves me well, the c-style cast is leftover from the monolithic compositing kcm. This minor issue can be addressed in another patch though...
Sure, can only be in another patch anyway, that one got merged anyway. ;-)
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D26040
To: meven, #kwin, crossi, ervin, zzag
Cc: zzag, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20191223/6322e20a/attachment-0001.html>
More information about the kwin
mailing list