From: Peter Eisentraut <peter_e@gmx.net>
Date: Thu, 6 Apr 2017 15:17:43 +0000 (-0400)
Subject: Fix compiler warning and add some more comments
X-Git-Tag: REL_10_BETA1~367
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dc0400cc501ebe839c15a387911945d1585e4787;p=postgresql

Fix compiler warning and add some more comments
---

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 49a73707bc..60f8b7f26c 100644
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -5924,6 +5924,11 @@ ATExecAddIdentity(Relation rel, const char *colName,
 	return address;
 }
 
+/*
+ * ALTER TABLE ALTER COLUMN SET { GENERATED or sequence options }
+ *
+ * Return the address of the affected column.
+ */
 static ObjectAddress
 ATExecSetIdentity(Relation rel, const char *colName, Node *def, LOCKMODE lockmode)
 {
@@ -5992,6 +5997,8 @@ ATExecSetIdentity(Relation rel, const char *colName, Node *def, LOCKMODE lockmod
 		ObjectAddressSubSet(address, RelationRelationId,
 							RelationGetRelid(rel), attnum);
 	}
+	else
+		address = InvalidObjectAddress;
 
 	heap_freetuple(tuple);
 	heap_close(attrelation, RowExclusiveLock);
@@ -5999,6 +6006,11 @@ ATExecSetIdentity(Relation rel, const char *colName, Node *def, LOCKMODE lockmod
 	return address;
 }
 
+/*
+ * ALTER TABLE ALTER COLUMN DROP IDENTITY
+ *
+ * Return the address of the affected column.
+ */
 static ObjectAddress
 ATExecDropIdentity(Relation rel, const char *colName, bool missing_ok, LOCKMODE lockmode)
 {