From 3200b15b20d9248be1b0f436ee787b2077d00298 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Thu, 5 Mar 2015 13:16:27 -0500 Subject: [PATCH] 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(). --- src/include/nodes/primnodes.h | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) 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 {