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

Albert Astals Cid <aa...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/kun
                   |                            |itconversion/bc7495095baa7a
                   |                            |70e756c4d2044de2197d1a4e3b
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED

--- Comment #6 from Albert Astals Cid <aa...@kde.org> ---
Git commit bc7495095baa7a70e756c4d2044de2197d1a4e3b by Albert Astals Cid, on
behalf of Michal MalĂ˝.
Committed on 14/04/2019 at 15:59.
Pushed by aacid into branch 'master'.

Fix l/100 km to MPG conversion

Summary:
Previous code (probably) worked only for l/100 km -> MPG but not the other way
around. Fix this by using reciprocal conversion when necessary.

Output:
38.7 MPG -> 6.077 l/100 km; 16.455 km/l; 46.482 MPGI
16.455 km/l -> 6.077 l/100 km; 38.7 MPG
46.48 MPGI -> 6.077 l/100 km

Seems good to me.

Reviewers: broulik, #frameworks, aacid

Subscribers: apol, aacid, meven, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D20284

M  +20   -0    autotests/convertertest.cpp
M  +23   -9    src/fuel_efficiency.cpp

https://commits.kde.org/kunitconversion/bc7495095baa7a70e756c4d2044de2197d1a4e3b

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

Reply via email to