Sync declarations and definitions of two new tablecmds.c functions.

tglsfdc · tglsfdc · commit ab3d8afc7f42 · 2025-04-16T17:59:08.000-04:00
Buildfarm member drongo complained because the definitions of these functions used "const Oid foo" where the forward declarations just had "Oid foo". (I'm a bit surprised that drongo seems to be the only complainant.) I chose to fix this by removing the "consts" because (a) I'm generally not a fan of using const that way, and (b) it was a minority usage even within these two functions, let alone compared to the rest of our code base. Oversight in commit eec0040, so no need for back-patch.
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
@@ -12384,7 +12384,7 @@ ATExecAlterConstraintInternal(List **wqueue, ATAlterConstraint *cmdcon,
 static bool
 ATExecAlterConstrEnforceability(List **wqueue, ATAlterConstraint *cmdcon,
 								Relation conrel, Relation tgrel,
-								const Oid fkrelid, const Oid pkrelid,
+								Oid fkrelid, Oid pkrelid,
 								HeapTuple contuple, LOCKMODE lockmode,
 								Oid ReferencedParentDelTrigger,
 								Oid ReferencedParentUpdTrigger,
@@ -12732,7 +12732,7 @@ AlterConstrTriggerDeferrability(Oid conoid, Relation tgrel, Relation rel,
 static void
 AlterConstrEnforceabilityRecurse(List **wqueue, ATAlterConstraint *cmdcon,
 								 Relation conrel, Relation tgrel,
-								 const Oid fkrelid, const Oid pkrelid,
+								 Oid fkrelid, Oid pkrelid,
 								 HeapTuple contuple, LOCKMODE lockmode,
 								 Oid ReferencedParentDelTrigger,
 								 Oid ReferencedParentUpdTrigger,
