Re: page macros cleanup (ver 04) - Mailing list pgsql-patches

From Tom Lane
Subject Re: page macros cleanup (ver 04)
Date
Msg-id 12344.1215972344@sss.pgh.pa.us
Whole thread Raw
In response to Re: page macros cleanup (ver 04)  (Zdenek Kotala <Zdenek.Kotala@Sun.COM>)
List pgsql-patches
Zdenek Kotala <Zdenek.Kotala@Sun.COM> writes:
> Good catch. if we have to bump catalog version then I have there small patch
> which introduce following macro and remove PageHeaderData stucture from
> HashMetaPageData:

Seems like a bad idea --- PageGetContents is not guaranteed to deliver
a maxaligned pointer, so at least in principle this could result in
alignment violations.  It would accidentally fail to fail as of CVS
HEAD, but any future rearrangement of PageHeaderData or HashMetaPageData
could create a problem.

(Possibly PageGetContents *should* guarantee a maxaligned result,
but the current coding does not.)

            regards, tom lane

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: variadic function support
Next
From: Tom Lane
Date:
Subject: Re: page macros cleanup (ver 04)