D6027: Use existing parseInt function
David Edmundson
noreply at phabricator.kde.org
Tue May 30 14:26:01 UTC 2017
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170530/30f93b0e/attachment.html>
More information about the Plasma-devel
mailing list