Update obsolete comment.

Neil Conway · Neil Conway · commit 9423383748b4 · 2005-03-13T05:19:26.000Z
diff --git a/src/backend/commands/functioncmds.c b/src/backend/commands/functioncmds.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/commands/functioncmds.c,v 1.54 2005/01/27 23:23:55 neilc Exp $
+ *	  $PostgreSQL: pgsql/src/backend/commands/functioncmds.c,v 1.55 2005/03/13 05:19:26 neilc Exp $
  *
  * DESCRIPTION
  *	  These routines take the parse tree and pick out the
@@ -300,23 +300,17 @@ compute_attributes_sql_style(List *options,
 
 
 /*-------------
- *	 Interpret the parameters *parameters and return their contents as
- *	 *byte_pct_p, etc.
+ *	 Interpret the parameters *parameters and return their contents via
+ *	 out parameters *isStrict_p and *volatility_p.
  *
  *	These parameters supply optional information about a function.
- *	All have defaults if not specified.
- *
- *	Note: currently, only two of these parameters actually do anything:
+ *	All have defaults if not specified. Parameters:
  *
  *	 * isStrict means the function should not be called when any NULL
  *	   inputs are present; instead a NULL result value should be assumed.
  *
  *	 * volatility tells the optimizer whether the function's result can
  *	   be assumed to be repeatable over multiple evaluations.
- *
- *	The other four parameters are not used anywhere.	They used to be
- *	used in the "expensive functions" optimizer, but that's been dead code
- *	for a long time.
  *------------
  */
 static void

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`*`
`11`	`11`	`*`
`12`	`12`	`* IDENTIFICATION`
`13`		`- * $PostgreSQL: pgsql/src/backend/commands/functioncmds.c,v 1.54 2005/01/27 23:23:55 neilc Exp $`
	`13`	`+ * $PostgreSQL: pgsql/src/backend/commands/functioncmds.c,v 1.55 2005/03/13 05:19:26 neilc Exp $`
`14`	`14`	`*`
`15`	`15`	`* DESCRIPTION`
`16`	`16`	`* These routines take the parse tree and pick out the`
`@@ -300,23 +300,17 @@ compute_attributes_sql_style(List *options,`
`300`	`300`
`301`	`301`
`302`	`302`	`/*-------------`
`303`		`- * Interpret the parameters *parameters and return their contents as`
`304`		`- * *byte_pct_p, etc.`
	`303`	`+ * Interpret the parameters *parameters and return their contents via`
	`304`	`+ * out parameters isStrict_p and volatility_p.`
`305`	`305`	`*`
`306`	`306`	`* These parameters supply optional information about a function.`
`307`		`- * All have defaults if not specified.`
`308`		`- *`
`309`		`- * Note: currently, only two of these parameters actually do anything:`
	`307`	`+ * All have defaults if not specified. Parameters:`
`310`	`308`	`*`
`311`	`309`	`* * isStrict means the function should not be called when any NULL`
`312`	`310`	`* inputs are present; instead a NULL result value should be assumed.`
`313`	`311`	`*`
`314`	`312`	`* * volatility tells the optimizer whether the function's result can`
`315`	`313`	`* be assumed to be repeatable over multiple evaluations.`
`316`		`- *`
`317`		`- * The other four parameters are not used anywhere. They used to be`
`318`		`- * used in the "expensive functions" optimizer, but that's been dead code`
`319`		`- * for a long time.`
`320`	`314`	`*------------`
`321`	`315`	`*/`
`322`	`316`	`static void`