From 589f5aa2412155a52acaf68c92d8ff2b8eb3ca3c Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Sun, 13 Dec 1998 04:16:51 +0000
Subject: [PATCH] I have a simple patch about the treatment of functions. But
 it may be self-satisfied. Please check my patch at the end of this posting.

Case 1. executor evaluates functions twice

Hiroshi Inoue
Inoue@tpf.co.jp
---
 src/backend/parser/parse_func.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/backend/parser/parse_func.c b/src/backend/parser/parse_func.c
index e1a625f9ef..a95c1b46b5 100644
--- a/src/backend/parser/parse_func.c
+++ b/src/backend/parser/parse_func.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.32 1998/12/08 06:18:56 thomas Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.33 1998/12/13 04:16:51 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1490,15 +1490,13 @@ ParseComplexProjection(ParseState *pstate,
 						newexpr->typeOid = funcnode->functype;
 						newexpr->opType = FUNC_EXPR;
 						newexpr->oper = (Node *) funcnode;
-						newexpr->args = lcons(first_arg, NIL);
+						newexpr->args = expr->args;
 
 						return (Node *) newexpr;
 					}
 
 				}
 
-				elog(ERROR, "Function '%s' has bad return type %d",
-					 funcname, argtype);
 				break;
 			}
 		case T_Param:
-- 
2.40.0