From: | "Hiroshi Saito" <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp> |
---|---|
To: | "Teodor Sigaev" <teodor(at)sigaev(dot)ru>, "Hiroshi Inoue" <inoue(at)tpf(dot)co(dot)jp> |
Cc: | "PG Hackers" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: regression test crashes at tsearch |
Date: | 2009-03-01 07:09:23 |
Message-ID: | EA432AD35F4941409FD5F9DEF59DB8A2@HIRO57887DE653 |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Teodor-san.
Sorry late reaction.
----- Original Message -----
From: "Teodor Sigaev" <teodor(at)sigaev(dot)ru>
>> If there's an effective function like pg_wchar2mb_with_len() which
>> converts wchar_t strings to server encoded strings, we had better
>> simply call it for char2wchar().
> I don't see a way to produce correct result of char2wchar with C-locale and
> sizeof(wchar_t) = 2.
>
> In summary, I suggest to remove support of C-locale from char2wchar function and
> tsearch's parser should directly use pg_mb2wchar_with_len() in case of
> C-locale and multibyte encoding. In all other places char2wchar is called only
> for non-C locale.
>
> Please, test attached patch.
Um, I think your patch like the overkill reaction of C-locale...
However, I tried your patch.
make check MULTIBYTE=euc_jp NO_LOCALE=true
...
=======================
All 120 tests passed.
=======================
Anyway, either should be applied.
Thanks.
Regards,
Hiroshi Saito
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2009-03-01 07:21:24 | Re: Synchronous replication & Hot standby patches |
Previous Message | Robert Haas | 2009-03-01 04:22:45 | Re: add_path optimization |