diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-11-03 10:57:49 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-11-03 10:57:49 -0500 |
| commit | 741b1aaf61a3da10250555f827c0a7a2f9bc2822 (patch) | |
| tree | 54ee8a18a225d8508c8f94ab66c443a1885efab2 | |
| parent | 8af1624e3f0efd2d029217a91d07bd3795f080a6 (diff) | |
| download | postgresql-741b1aaf61a3da10250555f827c0a7a2f9bc2822.tar.gz postgresql-741b1aaf61a3da10250555f827c0a7a2f9bc2822.zip | |
Fix PG_GETARG_SEG_P() definition.
DatumGetPointer() takes a Datum argument, not a pointer.
This is cosmetic given the current definitions of the
underlying macros, but it's still formally a type violation.
Bug was introduced in commit 389bb2818, but there seems
no need to back-patch.
Dagfinn Ilmari Mannsåker
Discussion: https://postgr.es/m/d8jlfsxq3a0.fsf@dalvik.ping.uio.no
| -rw-r--r-- | contrib/seg/seg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/seg/seg.c b/contrib/seg/seg.c index 4e34fba7c7b..f87456405c3 100644 --- a/contrib/seg/seg.c +++ b/contrib/seg/seg.c @@ -19,7 +19,7 @@ #define DatumGetSegP(X) ((SEG *) DatumGetPointer(X)) -#define PG_GETARG_SEG_P(n) DatumGetSegP(PG_GETARG_POINTER(n)) +#define PG_GETARG_SEG_P(n) DatumGetSegP(PG_GETARG_DATUM(n)) /* |