After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 122373 - Decimal separator
Decimal separator
Status: RESOLVED FIXED
Product: gnome-calculator
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Rich Burridge
Rich Burridge
: 123341 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-09-15 20:11 UTC by Stefano Canepa
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix for the problem. (5.36 KB, text/plain)
2003-09-16 16:38 UTC, Rich Burridge 		Details

Description Stefano Canepa 2003-09-15 20:11:59 UTC 
Even if in the italian translation a "," (comma) is used instead of a "."
(dot) the display uses always a "." (dot). In italian we use "," (comma) as
decimal separator and "." (dot) to group thousands. So one tenth is 0,10,
while one thousand is 1.000.
Comment 1 Rich Burridge 2003-09-16 16:38:26 UTC 
Created attachment 19989 [details]
Fix for the problem.
Comment 2 Rich Burridge 2003-09-16 16:41:07 UTC 
I've attached a patch that hopefully fixes this problem.
I've also checked it the same patch to CVS HEAD. Fixed
in v4.3.5. I'm closing the bug but I'd appreciate it if
you could add a comment letting me know that this works
for you. Alternatively, if it doesn't work, please
reopen, and let me know what the remaining problem is.

Thanks.
Comment 3 Rich Burridge 2003-09-16 20:52:18 UTC 
Refixed this to make it easier to update.
Created a new routine, get_numeric_sep()
which now returns a localised character
which is the numeric separator. Fixed in
CVS HEAD, in v4.3.6. 

Stefano, could you try this fix in CVS
HEAD please, rather than will the bogus
attached patch below.

Thanks.
Comment 4 Rich Burridge 2003-10-02 14:31:57 UTC 
I'm reopening this bug so that I can close
bug #123341 as a duplicate against it. The
real fix for this bug should have been to
use nl_langinfo(). I will attempt to fix this,
this way, later on today.
Comment 5 Rich Burridge 2003-10-02 14:34:57 UTC 
*** Bug 123341 has been marked as a duplicate of this bug. ***
Comment 6 Rich Burridge 2003-10-02 19:18:08 UTC 
Code to use nl_langinfo() to get the radix
character has now been added to gcalctool
CVS HEAD. Fixed in v4.3.11. Stefano, could
you possibly try this latest version and make
sure it does the right thing for you in the
Italian locale please?

Thanks.