From: Bruce Momjian <bruce@momjian.us>
Date: Fri, 29 Nov 1996 15:56:18 +0000 (+0000)
Subject: Allow select oid,* from table.  Allow * anywhere in target list.
X-Git-Tag: REL2_0~28
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a3d773a69320a92812d5dc7a5489db3ec77e2845;p=postgresql

Allow select oid,* from table.  Allow * anywhere in target list.
---

diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c
index 9704b94965..05f9c091ad 100644
--- a/src/backend/parser/analyze.c
+++ b/src/backend/parser/analyze.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *    $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.16 1996/11/26 03:17:45 bryanh Exp $
+ *    $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.17 1996/11/29 15:56:16 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1127,10 +1127,14 @@ transformTargetList(ParseState *pstate, List *targetlist)
 	     * (eg. SELECT * FROM emp)
 	     */
 	    if (att->relname!=NULL && !strcmp(att->relname, "*")) {
-		if(lnext(targetlist)!=NULL)
-		    elog(WARN, "cannot expand target list *, ...");
-		p_target = expandAllTables(pstate);
+		if (tail_p_target == NIL)
+		    p_target = tail_p_target = expandAllTables(pstate);
+		else
+		    lnext(tail_p_target) = expandAllTables(pstate);
 
+		while(lnext(tail_p_target)!=NIL)
+			/* make sure we point to the last target entry */
+		    tail_p_target = lnext(tail_p_target);
 		/*
 		 * skip rest of while loop
 		 */
@@ -1147,14 +1151,13 @@ transformTargetList(ParseState *pstate, List *targetlist)
 		/* tail_p_target is the target list we're building in the while
 		 * loop. Make sure we fix it after appending more nodes.
 		 */
-		if (tail_p_target == NIL) {
+		if (tail_p_target == NIL)
 		    p_target = tail_p_target = expandAll(pstate, att->relname,
 					att->relname, &pstate->p_last_resno);
-		} else {
+		else
 		    lnext(tail_p_target) =
 			expandAll(pstate, att->relname, att->relname,
 							&pstate->p_last_resno);
-		}
 		while(lnext(tail_p_target)!=NIL)
 			/* make sure we point to the last target entry */
 		    tail_p_target = lnext(tail_p_target);
diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index da8f5244d1..677d08a035 100644
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *    $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 1.18 1996/11/28 05:46:08 momjian Exp $
+ *    $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 1.19 1996/11/29 15:56:18 momjian Exp $
  *
  * HISTORY
  *    AUTHOR		DATE		MAJOR EVENT
@@ -1935,19 +1935,6 @@ res_target_list2:
 		{   $$ = lappend($1, $3);  }
 	| res_target_el2 			
 		{   $$ = lcons($1, NIL);  }
-	| '*'
-		{
-		    ResTarget *rt = makeNode(ResTarget);
-		    Attr *att = makeNode(Attr);
-		    att->relname = "*";
-		    att->paramNo = NULL;
-		    att->attrs = NULL;
-		    att->indirection = NIL;
-		    rt->name = NULL;
-		    rt->indirection = NULL;
-		    rt->val = (Node *)att;
-		    $$ = lcons(rt, NIL);
-		}
 	;
 
 /* AS is not optional because shift/red conflict with unary ops */
@@ -1977,6 +1964,18 @@ res_target_el2: a_expr AS Id
 		    $$->indirection = NULL;
 		    $$->val = (Node *)att;
 		}
+	| '*'
+		{
+		    Attr *att = makeNode(Attr);
+		    att->relname = "*";
+		    att->paramNo = NULL;
+		    att->attrs = NULL;
+		    att->indirection = NIL;
+		    $$ = makeNode(ResTarget);
+		    $$->name = NULL;
+		    $$->indirection = NULL;
+		    $$->val = (Node *)att;
+		}
 	;
 
 opt_id:  Id					{ $$ = $1; }