Skip site navigation (1) Skip section navigation (2)

Re: [PATCH] Re: Adding xpath_exists function

From: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
To: Mike Fowler <mike(at)mlfowler(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Re: Adding xpath_exists function
Date: 2010-08-06 02:29:10
Message-ID: 1281061552-sup-8773@alvh.no-ip.org (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-rrreviewers
Excerpts from Mike Fowler's message of mar jun 29 06:37:28 -0400 2010:

> After seeing some other posts in the last couple of days, I realised I 
> hadn't documented the function in the SGML. I have now done so, and 
> added a couple of tests with XML literals. Please find the patch 
> attached. Now time to go correct the xmlexists patch too...

Hmm, is 0 a valid node number in a xmlNodeSet?  If it is, searching for
the zeroth node would make the code call PG_RETURN_BOOL(0), no?

-- 
Álvaro Herrera <alvherre(at)commandprompt(dot)com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

In response to

pgsql-hackers by date

Next:From: Greg StarkDate: 2010-08-06 02:44:30
Subject: Re: BUG #5599: Vacuum fails due to index corruption issues
Previous:From: Robert HaasDate: 2010-08-06 02:11:20
Subject: LockDatabaseObject vs. LockSharedObject

pgsql-rrreviewers by date

Next:From: Peter EisentrautDate: 2010-08-06 04:38:12
Subject: Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function
Previous:From: Kevin GrittnerDate: 2010-08-05 18:42:54
Subject: CommitFest 2010-07 week three progress report

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group