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