David Edmundson <> changed:

           What    |Removed                     |Added
      Latest Commit|                            |
                   |                            |plasma-addons/a51086867c0f0
                   |                            |f7e19b0dd2b9d59657542c46901
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #12 from David Edmundson <> ---
Git commit a51086867c0f0f7e19b0dd2b9d59657542c46901 by David Edmundson.
Committed on 15/01/2020 at 00:36.
Pushed by davidedmundson into branch 'master'.

[applets/calculator] Use Qt's in-built number formatting

Gets rid of a load of code.
Correctly handles floating point issues.
Results are correct.

Potentially this inadvertedly introduces a behaviour change
of including group separators depending on locale. This could be
considered a good thing, but if it's problematic we can always add a
.replace() at the end of this line.

Test Plan:
62 * 1.3 = 80.6

100000000 * 5000000 = 5e+17
(with whatever the right number of zeroes is)

42 * 1.33333 = 6.6666665

Reviewers: #plasma

Subscribers: plasma-devel

Tags: #plasma

Differential Revision:

M  +2    -18   applets/calculator/package/contents/ui/calculator.qml

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

Reply via email to