diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-02-23 15:57:08 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-02-23 15:57:08 -0500 |
| commit | c29aff959dc64f7321062e7f33d8c6ec23db53d3 (patch) | |
| tree | 4c3efa167e01805b2e4fcd6eead2e1a320a2c420 /contrib/btree_gist/btree_ts.c | |
| parent | b9d092c962ea3262930e3c31a8c3d79b66ce9d43 (diff) | |
| download | postgresql-c29aff959dc64f7321062e7f33d8c6ec23db53d3.tar.gz postgresql-c29aff959dc64f7321062e7f33d8c6ec23db53d3.zip | |
Consistently declare timestamp variables as TimestampTz.
Twiddle the replication-related code so that its timestamp variables
are declared TimestampTz, rather than the uninformative "int64" that
was previously used for meant-to-be-always-integer timestamps.
This resolves the int64-vs-TimestampTz declaration inconsistencies
introduced by commit 7c030783a, though in the opposite direction to
what was originally suggested.
This required including datatype/timestamp.h in a couple more places
than before. I decided it would be a good idea to slim down that
header by not having it pull in <float.h> etc, as those headers are
no longer at all relevant to its purpose. Unsurprisingly, a small number
of .c files turn out to have been depending on those inclusions, so add
them back in the .c files as needed.
Discussion: https://postgr.es/m/26788.1487455319@sss.pgh.pa.us
Discussion: https://postgr.es/m/27694.1487456324@sss.pgh.pa.us
Diffstat (limited to 'contrib/btree_gist/btree_ts.c')
| -rw-r--r-- | contrib/btree_gist/btree_ts.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/contrib/btree_gist/btree_ts.c b/contrib/btree_gist/btree_ts.c index 68177d2784a..97408f54d2d 100644 --- a/contrib/btree_gist/btree_ts.c +++ b/contrib/btree_gist/btree_ts.c @@ -3,6 +3,8 @@ */ #include "postgres.h" +#include <limits.h> + #include "btree_gist.h" #include "btree_utils_num.h" #include "utils/builtins.h" |