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