Re: contrib/xml2 function name change and minor bitrot

John Gray wrote:
> I have changed the name of the new parse function to xml_valid and fixed
> a reference to SortMem which meant that the code as supplied would work
> against 7.3 and 7.4 but wouldn't work in CVS. Oops! Patch is attached.

Patch applied.

A few comments on the module:

- can you include some licensing information? "Distributed under the
same terms as PostgreSQL, Copyright ..." should be sufficient unless
you'd prefer another license

- some information on how to configure the location of the libxml2
headers / libs would be good, I think

- the readme for both 'contrib/xml' and 'contrib/xml2' is called
'README.pgxml', yet neither module is actually named 'pgxml'

-Neil