Tom Lane wrote:
> Bruce Momjian <> writes:
> > Tom Lane wrote:
> >> This patch broke the regression tests.  It seems to change the C-locale
> >> behavior on many platforms.
> > Oh, so C locale has "" for thousands_sep?  Hmm.  So there is no way to
> > disinguish "" as "don't have" from "" as "don't want".  Will revert.
> Possibly we could special-case C locale, and allow an empty string only
> when not C locale.  However, it could be that there are other locales
> where the change would surprise people, too :-(

Not sure.  Peter stated the French locale was wrong for the Debian they
were using, and he should know, so I am going to wait for  more feeback.

One thing that has me confused is that psql/print.c doesn't check for a
zero-length string for thousands_sep in setDecimalLocale, so I am not
thinking that is wrong and should skip "" like we do in formatting.c.

  Bruce Momjian                        |               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?


Reply via email to