From 141e878bc3afd6cd271b273ed1e43f0c07666248 Mon Sep 17 00:00:00 2001
From: Neil Conway <neilc@samurai.com>
Date: Tue, 21 Jun 2005 00:35:05 +0000
Subject: [PATCH] Trivial dead code removal: in makeObjectName(), name1 must be
 non-NULL (due to the preceding strlen(), for example), so we needn't recheck
 this before invoking pg_mbcliplen().

Per Coverity static analysis performed by EnterpriseDB.
---
 src/backend/commands/indexcmds.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c
index f4faebd06cb..07adceb0228 100644
--- a/src/backend/commands/indexcmds.c
+++ b/src/backend/commands/indexcmds.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/commands/indexcmds.c,v 1.131 2005/05/06 17:24:53 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/commands/indexcmds.c,v 1.132 2005/06/21 00:35:05 neilc Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -760,8 +760,7 @@ makeObjectName(const char *name1, const char *name2, const char *label)
 			name2chars--;
 	}
 
-	if (name1)
-		name1chars = pg_mbcliplen(name1, name1chars, name1chars);
+	name1chars = pg_mbcliplen(name1, name1chars, name1chars);
 	if (name2)
 		name2chars = pg_mbcliplen(name2, name2chars, name2chars);
 
-- 
GitLab