Re: 8.0.3 regexp_replace()... - Mailing list pgsql-general

From Tom Lane
Subject Re: 8.0.3 regexp_replace()...
Date
Msg-id 4033.1138720931@sss.pgh.pa.us
Whole thread Raw
In response to 8.0.3 regexp_replace()...  ("rlee0001" <robeddielee@hotmail.com>)
List pgsql-general
"rlee0001" <robeddielee@hotmail.com> writes:
> CREATE OR REPLACE FUNCTION "regexp_replacex" (source varchar, pattern
> varchar, replacement varchar) RETURNS varchar AS
> $body$
> DECLARE
>  retvalue VARCHAR;
> BEGIN
>  retvalue = "source";
>  LOOP
>   retvalue = REPLACE(retvalue, SUBSTRING(retvalue FROM "pattern"),
> "replacement");
>   EXIT WHEN retvalue = REPLACE(retvalue, SUBSTRING(retvalue FROM
> "pattern"), "replacement");
>  END LOOP;
>  RETURN retvalue;
> END;
> $body$
> LANGUAGE 'plpgsql' VOLATILE CALLED ON NULL INPUT SECURITY INVOKER;

You probably don't want to be double-quoting the variable names.  Also,
I think this should be marked STRICT IMMUTABLE rather than the options
you have chosen.

> if you test it and do expect to have to kill the server when it enters
> the infinate loop.

Control-C cancels the query just fine for me ...

            regards, tom lane

pgsql-general by date:

Previous
From: "P. Scott DeVos"
Date:
Subject: plpython error logs
Next
From: Richard Huxton
Date:
Subject: Re: Can't get the field = ANY(array) clause to work...