Re: [HACKERS] Make length(char(n)) return 'true' length

2004-02-12 Thread Bruce Momjian
Looks good to me but I will get some other eyse on it before I apply it. Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches I will try to apply it within the next 48 hours.

Re: [HACKERS] Make length(char(n)) return 'true' length

2004-02-12 Thread Tom Lane
Bruce Momjian [EMAIL PROTECTED] writes: Looks good to me but I will get some other eyse on it before I apply it. It's in already ... regards, tom lane ---(end of broadcast)--- TIP 6: Have you searched our list archives?

Re: [HACKERS] Make length(char(n)) return 'true' length

2004-02-12 Thread Bruce Momjian
Already applied. Thanks. --- Gavin Sherry wrote: The attached patch changes the existing behaviour of length(char(n)). Currently, this is what happens: template1=# select length('blah'::char(10)); length

Re: [HACKERS] Make length(char(n)) return 'true' length

2004-02-12 Thread Bruce Momjian
Gavin Sherry wrote: I believe Tom applied this while you were away. Oh, sorry, I see it now: test= select length('blah'::char(10)); length 4 (1 row) I did test this before placing it the queue, but I now realize I have been testing

Re: [HACKERS] Make length(char(n)) return 'true' length

2004-02-12 Thread Gavin Sherry
I believe Tom applied this while you were away. Gavin On Thu, 12 Feb 2004, Bruce Momjian wrote: Looks good to me but I will get some other eyse on it before I apply it. Your patch has been added to the PostgreSQL unapplied patches list at:

[HACKERS] Make length(char(n)) return 'true' length

2004-01-27 Thread Gavin Sherry
The attached patch changes the existing behaviour of length(char(n)). Currently, this is what happens: template1=# select length('blah'::char(10)); length 10 (1 row) With this patch: template1=# select length('blah'::char(10)); length 4 (1 row) This behaviour was