From d1c7d9f09d6a5eb8405c3807f4a6b02b86019227 Mon Sep 17 00:00:00 2001
From: Hadi Moshayedi <hadi@moshayedi.net>
Date: Thu, 8 Oct 2020 11:29:27 -0700
Subject: [PATCH] address feedback

---
 Makefile        | 2 +-
 cstore_reader.c | 5 +----
 cstore_writer.c | 4 ++--
 3 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/Makefile b/Makefile
index 461c33280..6ef8431c8 100644
--- a/Makefile
+++ b/Makefile
@@ -25,7 +25,7 @@ else
 $(error version $(VER) is not supported)
 endif
 
-PG_CPPFLAGS = -std=c11 -Wshadow
+PG_CFLAGS = -std=c11 -Wshadow
 OBJS = cstore.o cstore_writer.o cstore_reader.o \
        cstore_compression.o mod.o cstore_metadata_tables.o
 
diff --git a/cstore_reader.c b/cstore_reader.c
index 69a115ed9..cf2d0b171 100644
--- a/cstore_reader.c
+++ b/cstore_reader.c
@@ -988,7 +988,6 @@ DeserializeBlockData(StripeBuffers *stripeBuffers, uint64 blockIndex,
 static Datum
 ColumnDefaultValue(TupleConstr *tupleConstraints, Form_pg_attribute attributeForm)
 {
-	Datum defaultValue = 0;
 	Node *defaultValueNode = NULL;
 	int defValIndex = 0;
 
@@ -1009,7 +1008,7 @@ ColumnDefaultValue(TupleConstr *tupleConstraints, Form_pg_attribute attributeFor
 	if (IsA(defaultValueNode, Const))
 	{
 		Const *constNode = (Const *) defaultValueNode;
-		defaultValue = constNode->constvalue;
+		return constNode->constvalue;
 	}
 	else
 	{
@@ -1018,8 +1017,6 @@ ColumnDefaultValue(TupleConstr *tupleConstraints, Form_pg_attribute attributeFor
 						errhint("Expression is either mutable or "
 								"does not evaluate to constant value")));
 	}
-
-	return defaultValue;
 }
 
 
diff --git a/cstore_writer.c b/cstore_writer.c
index 9d57ec275..2c0ca541e 100644
--- a/cstore_writer.c
+++ b/cstore_writer.c
@@ -386,8 +386,8 @@ WriteToSmgr(TableWriteState *writeState, char *data, uint32 dataLength)
 
 		while (addr.blockno >= nblocks)
 		{
-			buffer = ReadBuffer(rel, P_NEW);
-			ReleaseBuffer(buffer);
+			Buffer newBuffer = ReadBuffer(rel, P_NEW);
+			ReleaseBuffer(newBuffer);
 			nblocks = smgrnblocks(rel->rd_smgr, MAIN_FORKNUM);
 		}