Robert Haas wrote: > On Fri, Jul 29, 2011 at 11:18 AM, Johann 'Myrkraverk' Oskarsson > <joh...@2ndquadrant.com> wrote: > > Hi all, > > > > I just noticed that the USECS_* constants are not defined when the server > > is compiled without integer dates and timestamps. > > > > Explicitly, timestamp.h is > > > > #ifdef HAVE_INT64_TIMESTAMP > > #define USECS_PER_DAY INT64CONST(86400000000) > > #define USECS_PER_HOUR INT64CONST(3600000000) > > #define USECS_PER_MINUTE INT64CONST(60000000) > > #define USECS_PER_SEC INT64CONST(1000000) > > #endif > > > > Is there a particular reason for this? ?Even with float8 timestamps > > there are uses for these constants in extensions. > > I don't see any particular reason not define them unconditionally.
Well, they are only used by integer dates, so why expand their visibility? The define does make it clear how they are used. I suppose if someone wanted to use them outside that case, we could open them up. It is true that with integer dates now the default, we might find that someone introduces compile problems by using them outside the integer dates scope. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
