From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 10 Mar 2016 04:29:05 +0000 (-0500)
Subject: Remove a couple of useless pstrdup() calls.
X-Git-Tag: REL9_6_BETA1~538
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cc402116ca156babcd3ef941317f462a96277e3a;p=postgresql

Remove a couple of useless pstrdup() calls.

There's no point in pstrdup'ing the result of TextDatumGetCString,
since that's necessarily already a freshly-palloc'd C string.

These particular calls are unlikely to be of any consequence
performance-wise, but still they're a bad precedent that can confuse
future patch authors.

Noted by Chapman Flack.
---

diff --git a/src/backend/foreign/foreign.c b/src/backend/foreign/foreign.c
index 45180c7414..239849bb0b 100644
--- a/src/backend/foreign/foreign.c
+++ b/src/backend/foreign/foreign.c
@@ -121,14 +121,14 @@ GetForeignServer(Oid serverid)
 							tp,
 							Anum_pg_foreign_server_srvtype,
 							&isnull);
-	server->servertype = isnull ? NULL : pstrdup(TextDatumGetCString(datum));
+	server->servertype = isnull ? NULL : TextDatumGetCString(datum);
 
 	/* Extract server version */
 	datum = SysCacheGetAttr(FOREIGNSERVEROID,
 							tp,
 							Anum_pg_foreign_server_srvversion,
 							&isnull);
-	server->serverversion = isnull ? NULL : pstrdup(TextDatumGetCString(datum));
+	server->serverversion = isnull ? NULL : TextDatumGetCString(datum);
 
 	/* Extract the srvoptions */
 	datum = SysCacheGetAttr(FOREIGNSERVEROID,