Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function

Mike Fowler <mike@mlfowler.com>

From: Mike Fowler <mike@mlfowler.com>

To: Peter Eisentraut <peter_e@gmx.net>

Cc: David Fetter <david@fetter.org>, Bruce Momjian <bruce@momjian.us>, Robert Haas <robertmhaas@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>, Round Robin Reviewers <pgsql-rrreviewers@postgresql.org>

Date: 2010-08-07T15:38:06Z

Lists: pgsql-hackers

Attachments

xpath_exists-4.patch (text/x-diff) patch

On 06/08/10 20:55, Peter Eisentraut wrote:
> On fre, 2010-08-06 at 09:04 +0100, Mike Fowler wrote:
>> If the patch is to be committed, does it make sense for me to refine
>> it such that it uses the new xpath internal function you extracted in
>> the xmlexists patch?
>
> Yes, you can probably shrink this patch down to about 20 lines.
>

Updated the patch so that it will apply to head and re-worked the 
function to use the new xpath internal function.

Regards,

-- 
Mike Fowler
Registered Linux user: 379787