View Revision
davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY

There's already a QString::toInt() that validates input. We don't need
to make our own.

Also there was a bug in the current point parsing code if the first was
invalid but the second valid. Both need to be checked.


TEST PLAN

Compiles


REPOSITORY
R110 KScreen Library

BRANCH
master

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

AFFECTED FILES
src/doctor/doctor.cpp
src/doctor/doctor.h

To: davidedmundson, Plasma
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas