Re: question on bits32 wraparound check - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: question on bits32 wraparound check
Date
Msg-id 20090404004759.GR23023@alvh.no-ip.org
Whole thread Raw
In response to Re: question on bits32 wraparound check  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> Alvaro Herrera <alvherre@commandprompt.com> writes:
> > So there's a minor issue on Takahiro-san fillfactor-on-toast patch,
> > which is that it does not hand out the last possible "kind" value.  This
> > is a bits32 field, so at least theoretically on some platforms it will
> > be wider than 32 while on others it will be exactly 32.
> 
> So just disallow kind > 31 (hard wired, with a note cross-referencing
> the datatype definition).  Even if that leaves some unused bits on some
> particular platform, it's correct and ensures portable behavior.

So Takahiro-san got it right the first time.  Thanks, committed that
way.

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: psql \d commands and information_schema
Next
From: Bruce Momjian
Date:
Subject: Re: Documentation Update: Document pg_start_backup checkpoint behavior