Re: [PATCHES] Case Conversion Fix for MB Chars

2005-12-02 Thread Volkan YAZICI
On 12/1/05, Bruce Momjian pgman@candle.pha.pa.us wrote: Where are we on this patch? Is it to be applied? After Tom's advice (he was doubtful about the patch), while I was thinking about how to improve the spectrum of tests, decided to use src/test/mb. In the tests, patch just succeded for

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-12-02 Thread Bruce Momjian
Volkan YAZICI wrote: On 12/1/05, Bruce Momjian pgman@candle.pha.pa.us wrote: Where are we on this patch? Is it to be applied? After Tom's advice (he was doubtful about the patch), while I was thinking about how to improve the spectrum of tests, decided to use src/test/mb. In the tests,

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-12-02 Thread Volkan YAZICI
Last minute edit: src/test/mb seems a little bit old. I've tested SQL files in src/test/mb/sql with the expected results in src/test/mb/expected manually and it worked. (Output files need a little bit editing, like removing lines similar to CREATE TABLE.) But it'll be better if any EUC users will

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-12-01 Thread Bruce Momjian
Where are we on this patch? Is it to be applied? --- Volkan YAZICI wrote: On 11/27/05, Volkan YAZICI [EMAIL PROTECTED] wrote: Tests made on an i686 with a 2.6.12.5 kernel. Here's a short list of cases I tried with

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-28 Thread Volkan YAZICI
On 11/27/05, Volkan YAZICI [EMAIL PROTECTED] wrote: Tests made on an i686 with a 2.6.12.5 kernel. Here's a short list of cases I tried with both latin5 and unicode charsets: - lower/upper functions with Turkish characters. - ILIKE matches with both lower and upper case Turkish characters.

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-27 Thread Volkan YAZICI
[Sorry if the post is duplicated. But I don't receive and ACK from majordamo.] On 11/27/05, Tom Lane [EMAIL PROTECTED] wrote: Doesn't this just move the failure cases around? I don't think so. I've tried to fix every tolower/toupper related problem (depending on the MB characters) I found in

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-27 Thread Volkan YAZICI
On 11/27/05, Tom Lane [EMAIL PROTECTED] wrote: Doesn't this just move the failure cases around? I don't think so. I've tried to fix every tolower/toupper related problem (depending on the MB characters) I found in the code. If there's still a problem with them, it should be related with the

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-27 Thread Tom Lane
Volkan YAZICI [EMAIL PROTECTED] writes: On 11/27/05, Tom Lane [EMAIL PROTECTED] wrote: The really fundamental problem is that tolower/toupper don't work on wchar_t. Yes, indeed. I agree with you to find a proper solution for character set handling. But, IMHO, it's better to have a-patchy

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-27 Thread Volkan YAZICI
On 11/27/05, Tom Lane [EMAIL PROTECTED] wrote: Yes, indeed. I agree with you to find a proper solution for character set handling. But, IMHO, it's better to have a-patchy working system instead of a not working one. But you just agreed that it doesn't work. You get me wrong. I tried to to

[PATCHES] Case Conversion Fix for MB Chars

2005-11-26 Thread Volkan YAZICI
Here's small patch to fix case conversion problems for ILIKE and (Oracle Compat.) lower/upper functions. (Related bug report is here: http://archives.postgresql.org/pgsql-bugs/2005-10/msg1.php) In tests it succeeded for Turkish characters while using LATIN5 encoding. But when encoding is

Re: [PATCHES] Case Conversion Fix for MB Chars

2005-11-26 Thread Volkan YAZICI
Last minute edit: On 11/26/05, Volkan YAZICI [EMAIL PROTECTED] wrote: - In tests it succeeded for Turkish characters while using LATIN5 - encoding. But when encoding is UNICODE it still doesn't work. (IMHO, - for latin-N encodings there will be no compatibility problems; for - Unicode, I've no