https://bugs.kde.org/show_bug.cgi?id=366135

Leslie Zhai <xiangzha...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |INVALID
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #3 from Leslie Zhai <xiangzha...@gmail.com> ---
Hi dcb314,

> QString::toInt() only writes to the variable pointed to, so the code for the 
> first issue is fine.

Please pay more attention to int QString::toInt(bool *ok, int base) const
If a conversion error occurs, *ok is set to false; otherwise *ok is set to
true.
http://code.qt.io/cgit/qt/qtbase.git/tree/src/corelib/tools/qstring.cpp#n6418

Much deeper please see
http://code.qt.io/cgit/qt/qtbase.git/tree/src/corelib/tools/qlocale.cpp#n3277

I know references are syntactic sugar, so code is easier to read and write, and
you can Google with C++ reference vs pointer ;-)

cover was set by setCover
https://github.com/KDE/audex/commit/bd2d7578aa9ee9cd38cb4a55df256200dbe71690

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to