 | |
Re: [BUGS] wrong behavior using to_char()
mailing.database.pgsql-bugs mailing.database.pgsql-bugs
 |
| | Thread Tools | Display Modes |
#1
 
| |||
| |||
|
Re: [BUGS] wrong behavior using to_char() -
09-14-2006
, 05:56 PM
|
In the pt_BR locale, the thousand separator is "". So it should return 12345,670. Looking at the source, I saw that the test cases for locale properties are independent among them. I think that the correct form is to have all-or-nothing test case or didn't test *lconv->property ("" is evaluated to false). Attached is a patch that fixes it using the second option. |
http://archives.postgresql.org/pgsql...2/msg00172.php
regards, tom lane
---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?
http://www.postgresql.org/docs/faq
#2
| |||
| |||
|
|
"Euler Taveira de Oliveira" <euler (AT) timbira (DOT) com> writes: In the pt_BR locale, the thousand separator is "". So it should return 12345,670. Looking at the source, I saw that the test cases for locale properties are independent among them. I think that the correct form is to have all-or-nothing test case or didn't test *lconv->property ("" is evaluated to false). Attached is a patch that fixes it using the second option. Not unless you have a solution to the problem seen in this thread: http://archives.postgresql.org/pgsql...2/msg00172.php |
date: 2006/02/12 23:48:23; author: momjian; state: Exp; lines: +3 -4
Revert because C locale uses "" for thousands_sep, meaning "n/a", while
French uses "" for "don't want". Seems we have to keep the existing
behavior.
----------------------------
revision 1.105
date: 2006/02/12 19:52:06; author: momjian; state: Exp; lines: +5 -4
Support "" for thousands separator and plus sign in to_char(), per
report from French Debian user. psql already handles "" fine.
One idea would be to handle C locale behavior differently from non-C
locale.
--
Bruce Momjian bruce (AT) momjian (DOT) us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?
http://archives.postgresql.org
#3
| |||
| |||
|
|
One idea would be to handle C locale behavior differently from non-C locale. |
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster
#4
| |||
| |||
|
|
In the pt_BR locale, the thousand separator is "". So it should return The thousands separator in pt_BR is ".". Oh, good catch. There is so much hack in my glibc. :-) |
--
Euler Taveira de Oliveira
http://www.timbira.com/
---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend
#5
| |||
| |||
|
|
Bruce Momjian wrote: One idea would be to handle C locale behavior differently from non-C locale. Right. |
--
Bruce Momjian bruce (AT) momjian (DOT) us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to majordomo (AT) postgresql (DOT) org so that your
message can get through to the mailing list cleanly
#6
| |||
| |||
|
|
In the pt_BR locale, the thousand separator is "". So it should return |
Quote:
|
12345,670. Looking at the source, I saw that the test cases for locale |
--
Jorge Godoy <jgodoy (AT) gmail (DOT) com>
---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match
|
« Previous Thread
|
Next Thread »
| Thread Tools | |
| Display Modes | |
Linear Mode
| |
Powered by vBulletin Version 3.5.3
Copyright ©2000 - 2012, Jelsoft Enterprises Ltd.
Copyright ©2000 - 2012, Jelsoft Enterprises Ltd.