Re: regression test crashes at tsearch

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

In response to

Responses

Browse pgsql-hackers by date

  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