diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 08:52:28 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2025-12-03 08:52:28 +0100 |
| commit | 8c6bbd674ed810df9af5ec42f6b38c205e3ad365 (patch) | |
| tree | 2ef7f5fd4135168a70c7ba71815723cf3d3abc7e /src | |
| parent | 623801b3bdb4e16df39f945b6aa5f109744c7b6b (diff) | |
Use more appropriate DatumGet* function
Use DatumGetCString() instead of DatumGetPointer() for returning a C
string. Right now, they are the same, but that doesn't always have to
be so.
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Reviewed-by: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Discussion: https://www.postgresql.org/message-id/4154950a-47ae-4223-bd01-1235cc50e933%40eisentraut.org
Diffstat (limited to 'src')
| -rw-r--r-- | src/test/modules/test_resowner/test_resowner_basic.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/modules/test_resowner/test_resowner_basic.c b/src/test/modules/test_resowner/test_resowner_basic.c index 8f794996371..b84ec2cb299 100644 --- a/src/test/modules/test_resowner/test_resowner_basic.c +++ b/src/test/modules/test_resowner/test_resowner_basic.c @@ -35,13 +35,13 @@ static const ResourceOwnerDesc string_desc = { static void ReleaseString(Datum res) { - elog(NOTICE, "releasing string: %s", DatumGetPointer(res)); + elog(NOTICE, "releasing string: %s", DatumGetCString(res)); } static char * PrintString(Datum res) { - return psprintf("test string \"%s\"", DatumGetPointer(res)); + return psprintf("test string \"%s\"", DatumGetCString(res)); } /* demonstrates phases and priorities between a parent and child context */ |