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

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 eec0040c4, so no need for back-patch.

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 5fad1fa44c1a1893440d87f2c491f67111806be2..f971f6019ca7c84c34ca224ef6b4c16bddcb475f 100644 (file)
--- 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,