D10555: Accept both decimal separators (dot and comma) if the group separator is not dot neither comma

[Miklos Marton]

martonmiklos created this revision.
Restricted Application added a subscriber: kde-utils-devel.
martonmiklos requested review of this revision.

REVISION SUMMARY
  There are some software which does not tolerate the locale and producing float numbers with fixed decimal separator.
  
  Copy pasting data from these software can be painful because kcalc cannot interpret these numbers properly.
  
  To increase the user experience this patch will allow pasting both separators if the thousand-group separator is not a dot neither a comma. (Usually it is a space).

TEST PLAN
  - Set some locale where the thousand separator is a space. For e.g.: Hungarian can be set with export LANG=hu_HU.UTF-8
  - Paste a number with the proper decimal separator (in the Hungarian example this is comma so the test number example is: 42,42)
  - Check if the number pasted correctly
  - Paste a number with the another decimal separator (in the Hungarian example this is dot so the test number example is: 42.42)
  - Check if the number pasted correctly

REPOSITORY
  R353 KCalc

REVISION DETAIL
  https://phabricator.kde.org/D10555

AFFECTED FILES
  kcalcdisplay.cpp

To: martonmiklos
Cc: kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20180215/7bf0b997/attachment.html>