Hello Peter!
Thanks for updates. Here is a small fix for clang "variable set but not
used" warnings.

-- 
Regards,
Timur Magomedov

From 8c17b18dc0c15ae87945169a85c3ee944fc815b7 Mon Sep 17 00:00:00 2001
From: Timur Magomedov <[email protected]>
Date: Fri, 12 Sep 2025 17:09:50 +0300
Subject: [PATCH] Fixed "set but not used" clang warnings

---
 contrib/vci/executor/vci_vector_executor.c | 4 ----
 contrib/vci/storage/vci_fetch.c            | 3 ---
 2 files changed, 7 deletions(-)

diff --git a/contrib/vci/executor/vci_vector_executor.c b/contrib/vci/executor/vci_vector_executor.c
index daf475b964e..d89988b4e75 100644
--- a/contrib/vci/executor/vci_vector_executor.c
+++ b/contrib/vci/executor/vci_vector_executor.c
@@ -1792,7 +1792,6 @@ static vci_vp_item_id
 vci_add_control_nodes(VciVPExecOp_func head_func, VciVPExecOp_func next_func, List *args,
 					  Expr *expr, PlanState *parent, ExprContext *econtext, VciVPContext *vpcontext, uint16 *skip_list)
 {
-	int			i;
 	vci_vp_item_id ret;
 	ListCell   *l;
 	Datum	   *itemValue = palloc(sizeof(Datum) * VCI_MAX_FETCHING_ROWS);
@@ -1807,7 +1806,6 @@ vci_add_control_nodes(VciVPExecOp_func head_func, VciVPExecOp_func next_func, Li
 	head_node->itemIsNull = itemIsNull;
 	head_node->data.init.orig_skip_list = skip_list;
 
-	i = 0;
 	foreach(l, args)
 	{
 		Expr	   *arg = (Expr *) lfirst(l);
@@ -1820,8 +1818,6 @@ vci_add_control_nodes(VciVPExecOp_func head_func, VciVPExecOp_func next_func, Li
 		next_node = &vpcontext->itemNode[ret];
 		next_node->itemValue = itemValue;
 		next_node->itemIsNull = itemIsNull;
-
-		i++;
 	}
 
 	return ret;
diff --git a/contrib/vci/storage/vci_fetch.c b/contrib/vci/storage/vci_fetch.c
index 80e497c4628..588ae59fa43 100644
--- a/contrib/vci/storage/vci_fetch.c
+++ b/contrib/vci/storage/vci_fetch.c
@@ -1479,7 +1479,6 @@ FillFixedWidthCopyBody1(char *dstData,
 						int numRows)
 {
 	int			aId;
-	int			numWrite = 0;
 	BlockNumber bNumCur = startBN;
 	int64		offset = startOf;
 	Buffer		buffer = InvalidBuffer;
@@ -1516,7 +1515,6 @@ FillFixedWidthCopyBody1(char *dstData,
 #endif							/* #ifdef WORDS_BIGENDIAN */
 			dstData += stepDstData;
 			offset += dataWidth;
-			++numWrite;
 		}
 
 		aId = maxBId;
@@ -1547,7 +1545,6 @@ FillFixedWidthCopyBody1(char *dstData,
 #endif							/* #ifdef WORDS_BIGENDIAN */
 			dstData += stepDstData;
 			offset = nextOffset;
-			++numWrite;
 			++aId;
 		}
 		else
-- 
2.43.0

Reply via email to