subdiff added inline comments. INLINE COMMENTS
> davidedmundson wrote in drm_output.cpp:104 > It's good design to not have member variables that can ever be in a corrupt > state, it reduces chance for bugs being introduced later, which is why I'm > against changing it. > > Dividing two integers is not going to make any performance difference > whatsoever and an int divided by an int will result in the same every single > time. Ok, fine with me. Can you now expand the testing parameter from the other Diff, so it's possible to use it in DRM mode? I tried to set a hard coded value in DrmOutput::setScale in order to test different scale factors, but the results were garbage, so I think (hope) that this hard coding doesn't set the scale factors elsewhere correctly and the testing parameter is necessary. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D3498 To: davidedmundson, graesslin, subdiff, #plasma Cc: subdiff, #kwin, plasma-devel, kwin, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol