diff options
author | Peter Eisentraut <peter@eisentraut.org> | 2022-09-12 17:35:55 +0200 |
---|---|---|
committer | Peter Eisentraut <peter@eisentraut.org> | 2022-09-12 17:36:26 +0200 |
commit | 595836e99bf1ee6d43405b885fb69bb8c6d3ee23 (patch) | |
tree | 9af38bcdaab9e75913c66b29f30caf1f9f4e5409 /src/include/utils/expandedrecord.h | |
parent | 8cb2a22bbb2cf4212482ac15021ceaa2e9c52209 (diff) | |
download | postgresql-595836e99bf1ee6d43405b885fb69bb8c6d3ee23.tar.gz postgresql-595836e99bf1ee6d43405b885fb69bb8c6d3ee23.zip |
Convert *GetDatum() and DatumGet*() macros to inline functions
The previous macro implementations just cast the argument to a target
type but did not check whether the input type was appropriate. The
function implementation can do better type checking of the input type.
Reviewed-by: Aleksander Alekseev <aleksander@timescale.com>
Discussion: https://www.postgresql.org/message-id/flat/8528fb7e-0aa2-6b54-85fb-0c0886dbd6ed%40enterprisedb.com
Diffstat (limited to 'src/include/utils/expandedrecord.h')
-rw-r--r-- | src/include/utils/expandedrecord.h | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/src/include/utils/expandedrecord.h b/src/include/utils/expandedrecord.h index be60e2ce530..aaf5b6bfd1b 100644 --- a/src/include/utils/expandedrecord.h +++ b/src/include/utils/expandedrecord.h @@ -138,10 +138,20 @@ typedef struct ExpandedRecordHeader MemoryContextCallback er_mcb; } ExpandedRecordHeader; -/* fmgr macros for expanded record objects */ +/* fmgr functions and macros for expanded record objects */ +static inline Datum +ExpandedRecordGetDatum(const ExpandedRecordHeader *erh) +{ + return EOHPGetRWDatum(&erh->hdr); +} + +static inline Datum +ExpandedRecordGetRODatum(const ExpandedRecordHeader *erh) +{ + return EOHPGetRODatum(&erh->hdr); +} + #define PG_GETARG_EXPANDED_RECORD(n) DatumGetExpandedRecord(PG_GETARG_DATUM(n)) -#define ExpandedRecordGetDatum(erh) EOHPGetRWDatum(&(erh)->hdr) -#define ExpandedRecordGetRODatum(erh) EOHPGetRODatum(&(erh)->hdr) #define PG_RETURN_EXPANDED_RECORD(x) PG_RETURN_DATUM(ExpandedRecordGetDatum(x)) /* assorted other macros */ |