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 129956 - For locales with non "." radix predefined constants not properly initialized
For locales with non "." radix predefined constants not properly initialized
Status: RESOLVED DUPLICATE of bug 130282
Product: gnome-calculator
Classification: Core
Component: general
unspecified
Other FreeBSD
: Normal normal
: ---
Assigned To: Rich Burridge
gnome-utils Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-12-24 15:42 UTC by Alexander Nedotsukov
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (1.03 KB, patch)
2003-12-24 15:43 UTC, Alexander Nedotsukov
none Details | Review

Description Alexander Nedotsukov 2003-12-24 15:42:13 UTC
Description of Problem:


Steps to reproduce the problem:
1. LANG=ru_RU.KOI8-R gcalctool
2. Check [Con] button
3. 

Actual Results:
2 for E, 3 for PI etc.

Expected Results:
2,71828.. for E, 3,14159.. etc

How often does this happen? 
Always.
Comment 1 Alexander Nedotsukov 2003-12-24 15:43:37 UTC
Created attachment 22688 [details] [review]
Proposed patch
Comment 2 Luis Villa 2003-12-30 02:29:11 UTC
Adding PATCH keyword.
Comment 3 Glynn Foster 2003-12-31 21:18:38 UTC
Changing the category to put on Rich's radar. Thanks for the patch.
Comment 4 Rich Burridge 2004-01-01 03:36:47 UTC
Actually, this looks like a variant of bug #130282.
The proposed patch isn't the correct fix as "." isn't
always the radix character. It's a locale dependent 
thing.
Comment 5 Alexander Nedotsukov 2004-01-02 14:19:37 UTC
This is not variant of bug #60683.
The proposed patch is correct fix since you have "." as radix in
hardcoded strings. It temporary swaps actual locale radix with "."
just to let those strings be parsed.
Comment 6 Rich Burridge 2004-01-02 16:23:58 UTC
Just substituting "." for the radix character
does not solve the problem that the constants should be
displayed in the Con menu with the correct radix character.

Bug #128603 is currently discussing the best way to fix
that.

Fix #128603 and I believe this problem will go away.
Your proposed solution is not a real fix for this
problem as the constant strings need to display the correct
radix character.
Comment 7 Alexander Nedotsukov 2004-01-03 00:41:39 UTC
If you apply this patch you probably understand how it works. And will
see Con menu filled with proper values. They will include *current
locale* radix.
Btw making this strings localizable is very bad idea. This looks for
me just like making yet anothere place where bugs may come.
Comment 8 Rich Burridge 2004-01-04 17:27:51 UTC
You're right. The patch will allow constants to
be displayed with the locale radix char. I'll
still need the patch from bug #128603 though,
for the read-only memory problem. I agree with
you about not localising the constant strings.

I'll check these fixes in tomorrow. Thanks.
Comment 9 Rich Burridge 2004-01-04 17:38:03 UTC
The last attachment to bug #130282 has a similar
fix plus more, so I'm closing this out as a dup.
of that bug.

*** This bug has been marked as a duplicate of 130282 ***