D6026: Add setScale option to kscreendoctor
Marco Martin
noreply at phabricator.kde.org
Mon Jun 5 12:45:48 UTC 2017
mart accepted this revision.
mart added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> doctor.cpp:227
> + } else if (ops.count() == 4 && ops[2] == QStringLiteral("scale")) {
> + int scale = ops[3].toInt(&ok);
> + if (!ok) {
would make a sense to guard from here against values < 1 or very big?
REPOSITORY
R110 KScreen Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D6026
To: davidedmundson, #plasma, mart
Cc: mart, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170605/7a8f242d/attachment-0003.html>
More information about the Plasma-devel
mailing list