Fix obsolete comment. It's no longer the case that Param nodes don't

carry typmod.

diff --git a/src/backend/optimizer/plan/subselect.c b/src/backend/optimizer/plan/subselect.c
index b861728502bf9f456cd4fb3a13f213db71d088db..441dd55045c7dec76667c43d224731bc78a47296 100644 (file)
--- a/src/backend/optimizer/plan/subselect.c
+++ b/src/backend/optimizer/plan/subselect.c
@@ -94,14 +94,9 @@ replace_outer_var(PlannerInfo *root, Var *var)
         * NOTE: in sufficiently complex querytrees, it is possible for the same
         * varno/abslevel to refer to different RTEs in different parts of the
         * parsetree, so that different fields might end up sharing the same Param
-        * number.      As long as we check the vartype as well, I believe that this
-        * sort of aliasing will cause no trouble. The correct field should get
-        * stored into the Param slot at execution in each part of the tree.
-        *
-        * We also need to demand a match on vartypmod.  This does not matter for
-        * the Param itself, since those are not typmod-dependent, but it does
-        * matter when make_subplan() instantiates a modified copy of the Var for
-        * a subplan's args list.
+        * number.      As long as we check the vartype/typmod as well, I believe that
+        * this sort of aliasing will cause no trouble.  The correct field should
+        * get stored into the Param slot at execution in each part of the tree.
         */
        i = 0;
        foreach(ppl, root->glob->paramlist)