[Differential] [Commented On] D2330: [xrandr backend] fall back to preferred mode, fix crash
graesslin (Martin Gräßlin)
noreply at phabricator.kde.org
Wed Aug 10 11:27:39 UTC 2016
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160810/044e5f79/attachment-0001.html>
More information about the Plasma-devel
mailing list