From: Jan Wieck <jwieck@debis.com>

scrappy · scrappy · commit a04dcc9137ba · 1998-02-13T13:23:33.000Z
Someone changed the parser to build a TypeName node on CREATE
    FUNCTION in any  case.  As  a  side  effect,  ALL!  functions
    created  got  the  proretset  attribute  to  true. Thus for a
    SELECT the parser wrapped an Iter node around  the  Expr  and
    since  singleton  functions  set  isDone  the Iter returns no
    tuple up.
diff --git a/src/backend/commands/define.c b/src/backend/commands/define.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/commands/define.c,v 1.21 1998/01/05 16:38:54 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/commands/define.c,v 1.22 1998/02/13 13:23:33 scrappy Exp $
  *
  * DESCRIPTION
  *	  The "DefineFoo" routines take the parse tree and pick out the
@@ -94,7 +94,7 @@ compute_return_type(const Node *returnType,
 		TypeName   *setType = (TypeName *) returnType;
 
 		*prorettype_p = setType->name;
-		*returnsSet_p = true;
+		*returnsSet_p = setType->setof;
 	}
 	else
 	{

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $Header: /cvsroot/pgsql/src/backend/commands/define.c,v 1.21 1998/01/05 16:38:54 momjian Exp $`
	`12`	`+ * $Header: /cvsroot/pgsql/src/backend/commands/define.c,v 1.22 1998/02/13 13:23:33 scrappy Exp $`
`13`	`13`	`*`
`14`	`14`	`* DESCRIPTION`
`15`	`15`	`* The "DefineFoo" routines take the parse tree and pick out the`
`@@ -94,7 +94,7 @@ compute_return_type(const Node *returnType,`
`94`	`94`	`TypeName setType = (TypeName ) returnType;`
`95`	`95`
`96`	`96`	`*prorettype_p = setType->name;`
`97`		`- *returnsSet_p = true;`
	`97`	`+ *returnsSet_p = setType->setof;`
`98`	`98`	`}`
`99`	`99`	`else`
`100`	`100`	`{`