Magnus Hagander <mag...@hagander.net> writes: > On Mon, Aug 15, 2011 at 16:20, Tom Lane <t...@sss.pgh.pa.us> wrote: >> However, for a narrow fix, I could see moving the data type definition >> to someplace with fewer dependencies. Perhaps split it into a separate >> file timestamp_type.h, or something like that.
> Yes, that seems to fix the problem of timestamptz. See the attached > patch - seems ok? Don't think you should expose fsec_t, nor most of those macros. The foo_per_bar values are just namespace clutter. > I also ran into a similar problem with some WAL macro definitions that > are in xlog_internal.h. I've moved them to xlogdefs.h in the attached > xlog.diff file. Does that seem ok as well, or should I move them > somewhere else? I don't like the idea of exposing those to frontends, either. What do you actually *need* out of that, and why? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers