Review Request 120357: [kcalc] KF5 port: replace KLocale::formatNumber() with custom code

Christoph Feck christoph at maxiom.de
Tue Sep 30 22:05:36 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120357/
-----------------------------------------------------------

(Updated Sept. 30, 2014, 10:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, KDE Utils, John Layt, and Evan Teran.


Repository: kcalc


Description
-------

To remove the last bits of KDELibs4Support in kcalc port, I had to replace KLocale::formatNumber() with a custom function, because I found no way to format arbitrary precision numbers with QLocale.

The groupSize is hardcoded to 3, I did not find  a simple way to query QLocale.

The code also replaces all digits with locale digits; not sure if the original KLocale code did the same and if this should be applied to kcalc.


Diffs
-----

  kcalcdisplay.h 807cb009d9ab3a91646779e2489a35ed635dedb7 
  kcalcdisplay.cpp b4e2b3bec4b11f3106765dbe5cb2aa0f7c8ceae2 

Diff: https://git.reviewboard.kde.org/r/120357/diff/


Testing
-------

Tested with german locale, which has '.' as groupSeperator, and ',' as decimalPoint.


Thanks,

Christoph Feck

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140930/4c7c0124/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list