0002-Use-better-DatumGet-function.patch

text/plain

Filename: 0002-Use-better-DatumGet-function.patch
Type: text/plain
Part: 1
Message: get rid of Pointer type, mostly

Patch

Same data as JSON: GET /api/v1/attachments/:id/patch the parsed metadata as JSON — format, series position, per-file stats; never the diff bytes. API reference →
Format: format-patch
Series: patch 0002
Subject: Use better DatumGet* function
File+
src/test/modules/test_resowner/test_resowner_basic.c 2 2
From b58263c3df6dc2eb918bde92eb04f7fd5d2a061b Mon Sep 17 00:00:00 2001
From: Peter Eisentraut <peter@eisentraut.org>
Date: Mon, 24 Nov 2025 11:02:35 +0100
Subject: [PATCH 2/6] Use better 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.
---
 src/test/modules/test_resowner/test_resowner_basic.c | 4 ++--
 1 file 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 */
-- 
2.52.0