public inbox for [email protected]
help / color / mirror / Atom feedFrom: Bruce Momjian <[email protected]>
To: Heikki Linnakangas <[email protected]>
Cc: Dmitriy Igrishin <[email protected]>
Cc: pgsql-docs <[email protected]>
Subject: Re: Clarification suggestion for 46.4 chapter.
Date: Sat, 1 Sep 2012 12:05:24 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <CAAfz9KN4PEf0QodA_F37M6RifK2sMXR8bjAgUqGuNKqOdVchXw@mail.gmail.com>
<[email protected]>
<[email protected]>
On Fri, Aug 17, 2012 at 11:54:54AM +0300, Heikki Linnakangas wrote:
> On 17.08.2012 05:07, Bruce Momjian wrote:
> >On Mon, Dec 5, 2011 at 02:53:22PM +0300, Dmitriy Igrishin wrote:
> >>Hey,
> >>
> >>The section 46.4 describes the base data types used in messages.
> >>http://www.postgresql.org/docs/9.1/static/protocol-message-types.html
> >>
> >>According to section 46.5
> >>http://www.postgresql.org/docs/9.1/static/protocol-message-formats.html
> >>there are cases when Int32 can be negative (e.g. see DataRow(B) message
> >>description.)
> >>
> >>Thus, I would like to suggest to change the description of Int(i)
> >>from
> >>"An n-bit integer in network byte order ..."
> >>to
> >>"An n-bit signed integer in network byte order ..."
> >
> >OK, documentation updated.
>
> Actually, in some cases the integers are signed, and in others
> unsigned. For example, in a Bind('F') message, the number of
> parameters is an Int16 according to the docs, but it is treated as
> unsigned. The maximum number of parameters is 65535.
>
> The sentence used to be factually correct, when it didn't mention
> whether they're signed or unsigned. If we want to do better than
> that, we'd need to go through all the mentions of IntN in the docs
> and explicitly say which ones are signed and which ones unsigned.
> Perhaps use Uint16 or Uint32 for the unsigned ones.
I have reverted the patch until someone is able to correctly designate
the values. Thanks.
--
Bruce Momjian <[email protected]> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
view thread (4+ messages)
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], [email protected]
Subject: Re: Clarification suggestion for 46.4 chapter.
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