v1-0013-formatting.c-cleanup-Change-fill_str-return-type-.patch

text/plain

Filename: v1-0013-formatting.c-cleanup-Change-fill_str-return-type-.patch
Type: text/plain
Part: 12
Message: formatting.c cleanup

Patch

Format: format-patch
Series: patch v1-0013
Subject: formatting.c cleanup: Change fill_str() return type to void
File+
src/backend/utils/adt/formatting.c 2 3
From 9038f68c2877a10b6b3303e8a51fb48ec6ad6516 Mon Sep 17 00:00:00 2001
From: Peter Eisentraut <peter@eisentraut.org>
Date: Mon, 20 Oct 2025 08:16:03 +0200
Subject: [PATCH v1 13/13] formatting.c cleanup: Change fill_str() return type
 to void

The return value is not used anywhere.
---
 src/backend/utils/adt/formatting.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/backend/utils/adt/formatting.c b/src/backend/utils/adt/formatting.c
index c421625cf60..4c217d0e825 100644
--- a/src/backend/utils/adt/formatting.c
+++ b/src/backend/utils/adt/formatting.c
@@ -1113,7 +1113,7 @@ static bool from_char_seq_search(int *dest, const char **src,
 static bool do_to_timestamp(text *date_txt, text *fmt, Oid collid, bool std,
 							struct pg_tm *tm, fsec_t *fsec, struct fmt_tz *tz,
 							int *fprec, uint32 *flags, Node *escontext);
-static char *fill_str(char *str, int c, size_t maxlen);
+static void fill_str(char *str, int c, size_t maxlen);
 static FormatNode *NUM_cache(int len, NUMDesc *Num, text *pars_str, bool *shouldFree);
 static char *int_to_roman(int number);
 static int	roman_to_int(NUMProc *Np, size_t input_len);
@@ -4824,12 +4824,11 @@ do_to_timestamp(text *date_txt, text *fmt, Oid collid, bool std,
  *********************************************************************/
 
 
-static char *
+static void
 fill_str(char *str, int c, size_t maxlen)
 {
 	memset(str, c, maxlen);
 	str[maxlen] = '\0';
-	return str;
 }
 
 /* This works the same as DCH_prevent_counter_overflow */
-- 
2.51.0