pgsql: Code review for magic-block patch.

Lists: pgsql-committers
From: tgl(at)postgresql(dot)org (Tom Lane)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Code review for magic-block patch.
Date: 2006-05-30 21:21:30
Message-ID: 20060530212130.5F70C9FADD1@postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers

Log Message:
-----------
Code review for magic-block patch. Remove separate header file pgmagic.h,
as this seems only likely to create headaches for module developers. Put
the macro in the pre-existing fmgr.h file instead. Avoid being too cute
about how many fields we can cram into a word, and avoid trying to fetch
from a library we've already unlinked.
Along the way, it occurred to me that the magic block really ought to be
'const' so it can be stored in the program text area. Do the same for
the existing data blocks for PG_FUNCTION_INFO_V1 functions.

Modified Files:
--------------
pgsql/doc/src/sgml:
xfunc.sgml (r1.113 -> r1.114)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/xfunc.sgml.diff?r1=1.113&r2=1.114)
pgsql/src/backend/utils/fmgr:
dfmgr.c (r1.83 -> r1.84)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/fmgr/dfmgr.c.diff?r1=1.83&r2=1.84)
fmgr.c (r1.100 -> r1.101)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/fmgr/fmgr.c.diff?r1=1.100&r2=1.101)
pgsql/src/include:
fmgr.h (r1.43 -> r1.44)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/fmgr.h.diff?r1=1.43&r2=1.44)
pgsql/src/test/regress:
regress.c (r1.66 -> r1.67)
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/test/regress/regress.c.diff?r1=1.66&r2=1.67)

Removed Files:
-------------
pgsql/src/include:
pgmagic.h
(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/pgmagic.h)