Re: [PATCHES] lc_time and localized dates

2008-05-19 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > I have fixed all these items and the updated patch is at: > ftp://momjian.us/pub/postgresql/mypatches/lc_time Applied with further fixes --- mostly, ensure that it doesn't leave a crash-inducing corrupted cache if strdup() returns NULL, and make th

Re: [PATCHES] lc_time and localized dates

2008-05-17 Thread Euler Taveira de Oliveira
Bruce Momjian wrote: I have reviewed this patch. I like the method you used, but I did find a few things I had to change. Good catch. I tested here and it seems ok. Thanks for your review. -- Euler Taveira de Oliveira http://www.timbira.com/ -- Sent via pgsql-patches mailing list (pgsq

Re: [PATCHES] lc_time and localized dates

2008-05-17 Thread Bruce Momjian
I have reviewed this patch. I like the method you used, but I did find a few things I had to change. First, I found the abbreviated variable names confusing so I used longer ones, like: extern char *localized_abbrev_days[7]; extern char *localized_full_days[7];

Re: [PATCHES] lc_time and localized dates

2008-04-24 Thread Alvaro Herrera
Euler Taveira de Oliveira wrote: > Here is an updated patch. It follows the Oracle behaviour and uses a > cache mechanism to avoid calling setlocale() all the time. I unified the > localized_* and str_* functions. I didn't test it on Windows. I would > appreciate some feedback. Added to Ma

Re: [PATCHES] lc_time and localized dates

2008-04-23 Thread Euler Taveira de Oliveira
Bruce Momjian wrote: Euler, have you updated this patch yet? Here is an updated patch. It follows the Oracle behaviour and uses a cache mechanism to avoid calling setlocale() all the time. I unified the localized_* and str_* functions. I didn't test it on Windows. I would appreciate some fe

Re: [PATCHES] lc_time and localized dates

2008-04-15 Thread Euler Taveira de Oliveira
Bruce Momjian wrote: Euler, have you updated this patch yet? I'm in a process to submit another patch but I want to test on Windows first. -- Euler Taveira de Oliveira http://www.timbira.com/ -- Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org) To make changes to your

Re: [PATCHES] lc_time and localized dates

2008-04-15 Thread Bruce Momjian
Euler Taveira de Oliveira wrote: > Tom Lane wrote: > > > But as Peter remarks nearby, this discussion is wasted anyway, because > > there is only one correct answer: whatever Oracle does with these > > cases is what to_char() should do. > > > My patch does exactly what Oracle does besides one thi

Re: [PATCHES] lc_time and localized dates

2008-02-28 Thread Zdeněk Kotala
Peter Eisentraut napsal(a): Am Dienstag, 26. Februar 2008 schrieb Zdenek Kotala: But how you handle situation when you have multi language application which needs correct output for all languages? You cannot use any of the pattern because it will be wrong for some group of languages. T

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Euler Taveira de Oliveira
Tom Lane wrote: But as Peter remarks nearby, this discussion is wasted anyway, because there is only one correct answer: whatever Oracle does with these cases is what to_char() should do. My patch does exactly what Oracle does besides one thing: my code does not contain a real capitalization f

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Gevik Babakhani
> But as Peter remarks nearby, this discussion is wasted > anyway, because there is only one correct answer: whatever > Oracle does with these cases is what to_char() should do. ++1 ---(end of broadcast)--- TIP 7: You can help support the Postgre

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Gevik Babakhani
> This is what you get when you copy a proprietary, poorly > specified interface. > The to_char functions are supposed to be Oracle-compatible, > so we need to check what Oracle does. Whether that is useful > in practice is a bit secondary. > > I'm beginning to think, if you want formatting

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Tom Lane
Euler Taveira de Oliveira <[EMAIL PROTECTED]> writes: > FYI, strftime() [1] doesn't say anything about capitalization. I don't > know if some translators are aware of it and even if they are, how would > they know that a day or month is the first word of a sentence? IMHO we > should provide Xxxx

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Euler Taveira de Oliveira
Zdenek Kotala wrote: But how you handle situation when you have multi language application which needs correct output for all languages? You cannot use any of the pattern because it will be wrong for some group of languages. FYI, strftime() [1] doesn't say anything about capitalization. I don

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Peter Eisentraut
Am Dienstag, 26. Februar 2008 schrieb Zdenek Kotala: > But how you handle situation when you have multi language application > which needs correct output for all languages? You cannot use any of the > pattern because it will be wrong for some group of languages. This is what you get when you copy

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Gevik Babakhani
> No. [Looking at the code...] I think it only affects the > LC_MESSAGES 'cause setlocale(LC_MESSAGES) don't work on Windows. In order to make setlocale(LC_MESSAGES) affect on windows some additional steps must be taken. I don't go deep in that now. I have a fix with description etc. etc. > Is

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Zdenek Kotala
Euler Taveira de Oliveira napsal(a): Zdenek Kotala wrote: Yes it is. Only if it is a first word in a sentence or name you should use "Ú". Also name of day is 'pondělí' (Monday) with small p But we're not talking about *sentence*, we need to consider just the *word*. So I think TMMonth should

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Euler Taveira de Oliveira
Zdenek Kotala wrote: Yes it is. Only if it is a first word in a sentence or name you should use "Ú". Also name of day is 'pondělí' (Monday) with small p But we're not talking about *sentence*, we need to consider just the *word*. So I think TMMonth should be Xxxx, TMMONTH should be X

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Euler Taveira de Oliveira
Gevik Babakhani wrote: Have you tested this patch on MSVC and MinGW (Windows) builds? changing LC_MESSAGES/LC_TIME will most probably break the Windows behavior. No. [Looking at the code...] I think it only affects the LC_MESSAGES 'cause setlocale(LC_MESSAGES) don't work on Windows. AFAIK,

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Gevik Babakhani
Magnus, Please look at this patch before you check my last patch about the locale. It seems that Euler's solution also fixes the windows locale bug that I was trying to fix. Replacing the lc_messages with lc_time when using to_char seems to be the correct direction. I have compiled and tested Eul

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Gevik Babakhani
> Attached is a patch that replaces the lc_messages with > lc_time when using to_char in translation mode (TM) [1]. It > doesn't change the output behaviour. Per discussion [2], it's > using some cache mechanism so we don't need to call > setlocale() all the time. Have you tested this patch on

Re: [PATCHES] lc_time and localized dates

2008-02-26 Thread Zdenek Kotala
Euler Taveira de Oliveira napsal(a): Zdenek Kotala wrote: Please, Day names does not have capitalized first letter in Czech language. We have "pondeli" as a Monday. If locale does not do that it is probably intention :-). Hmmm... I don't know about that. I do it that way 'cause I'm concerned

Re: [PATCHES] lc_time and localized dates

2008-02-25 Thread Euler Taveira de Oliveira
Zdenek Kotala wrote: Please, Day names does not have capitalized first letter in Czech language. We have "pondeli" as a Monday. If locale does not do that it is probably intention :-). Hmmm... I don't know about that. I do it that way 'cause I'm concerned about some locales that don't capita

Re: [PATCHES] lc_time and localized dates

2008-02-25 Thread Zdenek Kotala
Euler Taveira de Oliveira napsal(a): Hi, Hi Some issues: (i) some locales don't capitalize the first letter. I'm using pg_toupper() to do the job but I'm afraid it's not appropriated. I'm using it 'cause i'm too lazy to move localized_str_toupper(). Any suggestions? Please, Day names doe