Comment additions in parser.

diff --git a/src/backend/parser/parse_func.c b/src/backend/parser/parse_func.c
index 4c24d684991dd4e72321dc3834c3ef1cde78b0a8..ef2adde5cc4024ca0271abd0d754caa4bee8e6ce 100644 (file)
--- a/src/backend/parser/parse_func.c
+++ b/src/backend/parser/parse_func.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.107 2001/05/19 00:37:45 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.108 2001/05/19 01:57:11 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -106,6 +106,7 @@ ParseNestedFuncOrColumn(ParseState *pstate, Attr *attr, int precedence)
 
 /*
  *     parse function
+ *
  *     This code is confusing because the database can accept
  *  relation.column, column.function, or relation.column.function.
  * In these cases, funcname is the last parameter, and fargs are
@@ -115,6 +116,8 @@ ParseNestedFuncOrColumn(ParseState *pstate, Attr *attr, int precedence)
  *  In this case, Funcname is the part before parens, and fargs
  *  are the part in parens.
  *
+ * FYI, projection is choosing column from a table.
+ *
  */
 Node *
 ParseFuncOrColumn(ParseState *pstate, char *funcname, List *fargs,
@@ -144,7 +147,8 @@ ParseFuncOrColumn(ParseState *pstate, char *funcname, List *fargs,
    /*
     * Most of the rest of the parser just assumes that functions do
     * not have more than FUNC_MAX_ARGS parameters.  We have to test
-    * here to protect against array overruns, etc.
+    * here to protect against array overruns, etc.  Of course, this
+    * may not be a function, but the test doesn't hurt.
     */
    if (nargs > FUNC_MAX_ARGS)
        elog(ERROR, "Cannot pass more than %d arguments to a function",
@@ -158,6 +162,8 @@ ParseFuncOrColumn(ParseState *pstate, char *funcname, List *fargs,
    }
 
    /*
+    * test for relation.column
+    *
     * check for projection methods: if function takes one argument, and
     * that argument is a relation, param, or PQ function returning a
     * complex * type, then the function could be a projection.
@@ -170,9 +176,7 @@ ParseFuncOrColumn(ParseState *pstate, char *funcname, List *fargs,
        {
            Ident      *ident = (Ident *) first_arg;
 
-           /*
-            * first arg is a relation. This could be a projection.
-            */
+           /* First arg is a relation. This could be a projection. */
            refname = ident->name;
 
            retval = qualifiedNameToVar(pstate, refname, funcname, true);