From 2f131ca5880196d54ce0aac734bebe9c8393aa07 Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Fri, 30 Jun 2000 18:03:40 +0000
Subject: [PATCH] Fix select '1--2'; for PEter.

---
 src/bin/psql/mainloop.c | 58 +++++++++++++++++++++++------------------
 1 file changed, 33 insertions(+), 25 deletions(-)

diff --git a/src/bin/psql/mainloop.c b/src/bin/psql/mainloop.c
index d53efaeadc..748ef862e5 100644
--- a/src/bin/psql/mainloop.c
+++ b/src/bin/psql/mainloop.c
@@ -3,7 +3,7 @@
  *
  * Copyright 2000 by PostgreSQL Global Development Group
  *
- * $Header: /cvsroot/pgsql/src/bin/psql/mainloop.c,v 1.31 2000/06/29 16:27:57 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/bin/psql/mainloop.c,v 1.32 2000/06/30 18:03:40 momjian Exp $
  */
 #include "postgres.h"
 #include "mainloop.h"
@@ -44,7 +44,7 @@ MainLoop(FILE *source)
 
 	bool		success;
 	volatile char in_quote;		/* == 0 for no in_quote */
-	volatile bool xcomment;		/* in extended comment */
+	volatile bool in_xcomment;	/* in extended comment */
 	volatile int paren_level;
 	unsigned int query_start;
 	volatile int count_eof = 0;
@@ -80,7 +80,7 @@ MainLoop(FILE *source)
 		exit(EXIT_FAILURE);
 	}
 
-	xcomment = false;
+	in_xcomment = false;
 	in_quote = 0;
 	paren_level = 0;
 	slashCmdStatus = CMD_UNKNOWN;		/* set default */
@@ -123,7 +123,7 @@ MainLoop(FILE *source)
 				resetPQExpBuffer(query_buf);
 
 				/* reset parsing state */
-				xcomment = false;
+				in_xcomment = false;
 				in_quote = 0;
 				paren_level = 0;
 				count_eof = 0;
@@ -147,7 +147,7 @@ MainLoop(FILE *source)
 			line = xstrdup(query_buf->data);
 			resetPQExpBuffer(query_buf);
 			/* reset parsing state since we are rescanning whole line */
-			xcomment = false;
+			in_xcomment = false;
 			in_quote = 0;
 			paren_level = 0;
 			slashCmdStatus = CMD_UNKNOWN;
@@ -168,7 +168,7 @@ MainLoop(FILE *source)
 					prompt_status = PROMPT_SINGLEQUOTE;
 				else if (in_quote && in_quote == '"')
 					prompt_status = PROMPT_DOUBLEQUOTE;
-				else if (xcomment)
+				else if (in_xcomment)
 					prompt_status = PROMPT_COMMENT;
 				else if (paren_level)
 					prompt_status = PROMPT_PAREN;
@@ -296,23 +296,44 @@ MainLoop(FILE *source)
 				bslash_count = 0;
 
 		rescan:
-			/* start of extended comment? */
-			if (line[i] == '/' && line[i + thislen] == '*')
+			/*
+			 *	It is important to place the in_* test routines
+			 * 	before the in_* detection routines.
+			 *	i.e. we have to test if we are in a quote before
+			 *	testing for comments. bjm  2000-06-30
+			 */
+			
+			/* in quote? */
+			if (in_quote)
 			{
-				xcomment = true;
-				ADVANCE_1;
+				/* end of quote */
+				if (line[i] == in_quote && bslash_count % 2 == 0)
+					in_quote = '\0';
 			}
 
 			/* in extended comment? */
-			else if (xcomment)
+			else if (in_xcomment)
 			{
 				if (line[i] == '*' && line[i + thislen] == '/')
 				{
-					xcomment = false;
+					in_xcomment = false;
 					ADVANCE_1;
 				}
 			}
 
+			/* start of extended comment? */
+			else if (line[i] == '/' && line[i + thislen] == '*')
+			{
+				in_xcomment = true;
+				ADVANCE_1;
+			}
+
+			/* start of quote */
+			else if (!was_bslash &&
+					 (line[i] == '\'' || line[i] == '"'))
+				in_quote = line[i];
+
+
 			/* single-line comment? truncate line */
 			else if (line[i] == '-' && line[i + thislen] == '-')
 			{
@@ -320,19 +341,6 @@ MainLoop(FILE *source)
 				break;
 			}
 
-			/* in quote? */
-			else if (in_quote)
-			{
-				/* end of quote */
-				if (line[i] == in_quote && bslash_count % 2 == 0)
-					in_quote = '\0';
-			}
-
-			/* start of quote */
-			else if (!was_bslash &&
-					 (line[i] == '\'' || line[i] == '"'))
-				in_quote = line[i];
-
 			/* count nested parentheses */
 			else if (line[i] == '(')
 				paren_level++;