subdiff marked 4 inline comments as done. subdiff added inline comments. INLINE COMMENTS
> subdiff wrote in nightcolor.cpp:111 > Well, in my tests logind always changed the value of this property way later > then the notifier was triggered (like 1 second), so I assumed that this is > always the case. But you're right, that I don't know for sure. > > > Why do we have this check anyway? > > It's only a small difference in look and it's working anyway, that's why I > probably found the possible race condition not that important. Take a look at > line 117. If it's true, we change the color temperature value without fade, > otherwise we do. The else case is for example triggered on timezone change > while running, so it fades nicely in. But this is a corner case, and we might > just ignore it. @davidedmundson and I talked about it and he said he is content with it. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D5928 To: subdiff, #kwin Cc: ngraham, leezu, behrmann, cfeck, graesslin, davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, alexeymin, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein