public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: [email protected]
Cc: [email protected]
Subject: Re: Documentation update for PQexecParams
Date: Fri, 10 Nov 2006 17:24:10 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <1163009239.2384.2.camel@josh>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

Theo Kramer <[email protected]> writes:
> On Fri, 2006-11-10 at 10:50 +0100, Peter Eisentraut wrote:
>> The patch is backwards.

> Thanks - re-done and re-attached

I applied the part of this that reformats the PQexecParams argument
descriptions --- I agree that makes it more readable.  I did not like
the proposed table of datatypes though.  It strikes me as incomplete,
unhelpful, and in places downright misleading.  For instance, I don't
see the point of listing ODBC type codes; they are irrelevant to libpq,
and the only likely outcome of listing them is that someone might think
he could use them for paramTypes[] entries.  The description of the
binary formats was not helpful either, as it avoided any of the useful
details (endianness etc), as well as portability issues like whether
timestamps are int or float.

			regards, tom lane

view thread (8+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Documentation update for PQexecParams
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox