Commitb85cf68

committed

Add more comments about thousands separator handling.

1 parentf608f3b commitb85cf68Copy full SHA for b85cf68

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`/* -----------------------------------------------------------------------`
`2`	`2`	`* formatting.c`
`3`	`3`	`*`
`4`		`- * $PostgreSQL: pgsql/src/backend/utils/adt/formatting.c,v 1.135 2007/11/2215:10:05 momjian Exp $`
	`4`	`+ * $PostgreSQL: pgsql/src/backend/utils/adt/formatting.c,v 1.136 2007/11/2217:51:39 momjian Exp $`
`5`	`5`	`*`
`6`	`6`	`*`
`7`	`7`	`* Portions Copyright (c) 1999-2007, PostgreSQL Global Development Group`
`@@ -3926,6 +3926,7 @@ NUM_prepare_locale(NUMProc *Np)`
`3926`	`3926`	`*`
`3927`	`3927`	`* Some locales (e.g. broken glibc pt_BR), have a comma for decimal,`
`3928`	`3928`	`* but "" for thousands_sep, so we set the thousands_sep too.`
	`3929`	`+ * http://archives.postgresql.org/pgsql-hackers/2007-11/msg00772.php`
`3929`	`3930`	`*/`
`3930`	`3931`	`if (lconv->thousands_sep&&*lconv->thousands_sep)`
`3931`	`3932`	`Np->L_thousands_sep=lconv->thousands_sep;`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`*`
`4`	`4`	`* Copyright (c) 2000-2007, PostgreSQL Global Development Group`
`5`	`5`	`*`
`6`		`- * $PostgreSQL: pgsql/src/bin/psql/print.c,v 1.93 2007/11/2215:10:05 momjian Exp $`
	`6`	`+ * $PostgreSQL: pgsql/src/bin/psql/print.c,v 1.94 2007/11/2217:51:39 momjian Exp $`
`7`	`7`	`*`
`8`	`8`	`* Note: we include postgres.h not postgres_fe.h so that we can include`
`9`	`9`	`* catalog/pg_type.h, and thereby have access to INT4OID and similar macros.`
`@@ -2037,6 +2037,8 @@ setDecimalLocale(void)`
`2037`	`2037`	`grouping=strdup(extlconv->grouping);`
`2038`	`2038`	`else`
`2039`	`2039`	`grouping="3";/* most common */`
	`2040`	`+`
	`2041`	`+/* similar code exists in formatting.c */`
`2040`	`2042`	`if (*extlconv->thousands_sep)`
`2041`	`2043`	`thousands_sep=strdup(extlconv->thousands_sep);`
`2042`	`2044`	`/* Make sure thousands separator doesn't match decimal point symbol. */`

Comments

(0)