Re: support for printing/exporting xml

Brian Moore <brianmooreca@yahoo.com> writes:
> please find attached my support for printing/exporting
> xml from postgresql.

A few quick comments:

- why is it necessary to include an entire hash table implementation?
  Why does this code require a hash table to begin with, considering
  the relatively small size of pg_type? ISTM something far simpler
  (for example, a dynamically allocated sorted array and a binary
  search) would be sufficient.

- testing code doesn't belong in libpq

- you'll need to update the libpq docs

- you should follow the libpq API naming convention: the function
  should be look like some variant of PQresultAsXML()

- context diffs are the preferred format

-Neil