graesslin added inline comments.

INLINE COMMENTS

> sebas wrote in xrandrconfig.cpp:556-558
> No, this part is a bit different from the above hunk, it uses the xOutput by 
> default, and if that is empty, falls back to kscreen's currentModeId or the 
> preferredModeId.

but then I would reorder it. Why call toInt() and then check afterwards whether 
that one can work at all? I would suggest to make it if-else then

REPOSITORY
  rLIBKSCREEN KScreen Library

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: sebas, dvratil, graesslin
Cc: graesslin, plasma-devel, #plasma, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas

Reply via email to