Re: [HACKERS] [Review] Tests citext casts by David Wheeler.

[David E. Wheeler]

On Sep 5, 2008, at 11:33, David E. Wheeler wrote:

On Sep 5, 2008, at 11:30, Tom Lane wrote:

Thanks for reviewing.  I've committed this with your suggestions and
one additional non-cosmetic change: schema-qualify names in the

bodies of the SQL functions so that they are not search_path dependent.

Thanks, I'll check that out.

Finally got to this; sorry for the delay.

Two things I noticed:





CREATE OR REPLACE FUNCTION int8(citext)
RETURNS int8
AS 'SELECT int8( $1::text )'
LANGUAGE SQL IMMUTABLE STRICT;

CREATE OR REPLACE FUNCTION citext(int8)
RETURNS citext
AS 'SELECT text( $1 )::citext'
LANGUAGE SQL IMMUTABLE STRICT;

CREATE OR REPLACE FUNCTION int4(citext)
RETURNS int4
AS 'SELECT int4( $1::text )'
LANGUAGE SQL IMMUTABLE STRICT;

CREATE OR REPLACE FUNCTION citext(int4)
RETURNS citext
AS 'SELECT text( $1 )::citext'
LANGUAGE SQL IMMUTABLE STRICT;



Thanks,

David

1. Did I neglect to include the documentation patch? I've attached it here. It's necessary because of the addition of the new functions. 2. Many thanks for switching to using the network_show function instead of the SQL-based casting I had. Can you tell me how to go about finding such functions? Because for my 8.3 version of citext, I have a whole bunch of functions that do casting like this: ...and so on. I'd love to be able to replace these (and many others) with internal C functions, if only I could figure out what those functions were. A pointer to making that determination (if they even exist in 8.3) would be greatly appreciated.

citext_doc.patch
Description: Binary data

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers