From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 5 Mar 2015 18:16:27 +0000 (-0500)
Subject: Remove comment claiming that PARAM_EXTERN Params always have typmod -1.
X-Git-Tag: REL9_5_ALPHA1~671
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3200b15b20d9248be1b0f436ee787b2077d00298;p=postgresql

Remove comment claiming that PARAM_EXTERN Params always have typmod -1.

This hasn't been true in quite some time, cf plpgsql's make_datum_param().
---

diff --git a/src/include/nodes/primnodes.h b/src/include/nodes/primnodes.h
index dbc5a35cc8..4f1d234d30 100644
--- a/src/include/nodes/primnodes.h
+++ b/src/include/nodes/primnodes.h
@@ -179,9 +179,10 @@ typedef struct Const
 	int			location;		/* token location, or -1 if unknown */
 } Const;
 
-/* ----------------
+/*
  * Param
- *		paramkind - specifies the kind of parameter. The possible values
+ *
+ *		paramkind specifies the kind of parameter. The possible values
  *		for this field are:
  *
  *		PARAM_EXTERN:  The parameter value is supplied from outside the plan.
@@ -204,12 +205,6 @@ typedef struct Const
  *				of the `paramid' field contain the SubLink's subLinkId, and
  *				the low-order 16 bits contain the column number.  (This type
  *				of Param is also converted to PARAM_EXEC during planning.)
- *
- * Note: currently, paramtypmod is always -1 for PARAM_EXTERN params, since
- * the APIs that supply values for such parameters don't carry any typmod
- * info.  It is valid in other types of Params, if they represent expressions
- * with determinable typmod.
- * ----------------
  */
 typedef enum ParamKind
 {