diff options
| author | Tom Lane | 2025-12-07 16:33:35 +0000 |
|---|---|---|
| committer | Tom Lane | 2025-12-07 16:33:35 +0000 |
| commit | 3628af42107d588af73280d7e96d1c6188aadad7 (patch) | |
| tree | 4b232defb5ef96dc4d1b2dc4bb3269492e163a8a | |
| parent | 6498287696dafc1ebd380ea4eb249124989294d3 (diff) | |
Add a macro for the declared typlen of type timetz.
pg_type.typlen says 12 for the size of timetz, but sizeof(TimeTzADT)
will be 16 on most platforms due to alignment padding. Using the
sizeof number is no problem for usages such as palloc'ing a result
datum, but in usages such as datumCopy we really ought to match
what pg_type says. Add a macro TIMETZ_TYPLEN so that we have a
symbolic way to write that rather than hard-coding "12".
I cannot find any place where we've needed this so far, but an
upcoming patch requires it.
Author: Tom Lane <tgl@sss.pgh.pa.us>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Discussion: https://postgr.es/m/2329959.1765047648@sss.pgh.pa.us
| -rw-r--r-- | src/include/utils/date.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/include/utils/date.h b/src/include/utils/date.h index 7316ac0ff17..2aca785b65d 100644 --- a/src/include/utils/date.h +++ b/src/include/utils/date.h @@ -31,6 +31,14 @@ typedef struct } TimeTzADT; /* + * sizeof(TimeTzADT) will be 16 on most platforms due to alignment padding. + * However, timetz's typlen is 12 according to pg_type. In most places + * we can get away with using sizeof(TimeTzADT), but where it's important + * to match the declared typlen, use TIMETZ_TYPLEN. + */ +#define TIMETZ_TYPLEN 12 + +/* * Infinity and minus infinity must be the max and min values of DateADT. */ #define DATEVAL_NOBEGIN ((DateADT) PG_INT32_MIN) |