Revision: 6564
http://squirrel-sql.svn.sourceforge.net/squirrel-sql/?rev=6564&view=rev
Author: gerdwagner
Date: 2012-01-30 20:16:49 +0000 (Mon, 30 Jan 2012)
Log Message:
-----------
Parser mistook minus signs in lists as errors
Modified Paths:
--------------
trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Parser.java
trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/ParsingConstants.java
trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Scanner.java
trunk/sql12/doc/src/main/resources/changes.txt
Modified: trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Parser.java
===================================================================
--- trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Parser.java 2012-01-30 20:11:16 UTC (rev 6563)
+++ trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Parser.java 2012-01-30 20:16:49 UTC (rev 6564)
@@ -9,6 +9,7 @@
private static final int maxT = 103;
private static final boolean T = true;
+ private static final boolean G = true; // to see Gerds changes
private static final boolean x = false;
private static final int minErrDist = 2;
@@ -726,7 +727,7 @@
SelectStmt();
}
else
- Error(115);
+ Error(ParsingConstants.KW_INSET);
CloseParens();
}
@@ -788,7 +789,7 @@
{
Get();
}
- else if (t.kind == 52)
+ else if (t.kind == ParsingConstants.KW_MINUS_SIGN)
{
Get();
}
@@ -843,7 +844,7 @@
private final void Term()
{
- if (t.kind == 52)
+ if (t.kind == ParsingConstants.KW_MINUS_SIGN)
{
Get();
}
@@ -1008,6 +1009,12 @@
Get();
break;
}
+ case ParsingConstants.KW_MINUS_SIGN:
+ {
+ Get();
+ Get();
+ break;
+ }
case 53:
{
Param();
@@ -1879,7 +1886,7 @@
private static boolean[][] set = {
{T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, T, T, x, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
- {x, T, T, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, x, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
+ {x, T, T, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, G, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
{x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, x, x, T, x, x, x, x, x, x, x, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
{x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
{x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, T, x, x, x, x, T, x, T, x, x, x, x, x, T, T, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x, x},
Modified: trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/ParsingConstants.java
===================================================================
--- trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/ParsingConstants.java 2012-01-30 20:11:16 UTC (rev 6563)
+++ trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/ParsingConstants.java 2012-01-30 20:16:49 UTC (rev 6564)
@@ -111,4 +111,6 @@
static final int KIND_ASTERISK = 39;
static final int KIND_EQUALS = 14;
+ static final int KW_INSET = 115;
+ static final int KW_MINUS_SIGN = 52;
}
Modified: trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Scanner.java
===================================================================
--- trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Scanner.java 2012-01-30 20:11:16 UTC (rev 6563)
+++ trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/session/parser/kernel/Scanner.java 2012-01-30 20:16:49 UTC (rev 6564)
@@ -392,7 +392,7 @@
case ParsingConstants.KW_ALL:
{t.kind = ParsingConstants.KIND_ASTERISK; break loop;}
case ParsingConstants.KW_UPDATE:
- {t.kind = 52; break loop;}
+ {t.kind = ParsingConstants.KW_MINUS_SIGN; break loop;}
case ParsingConstants.KW_SET:
{t.kind = 53; break loop;}
case ParsingConstants.KIND_EQUALS:
Modified: trunk/sql12/doc/src/main/resources/changes.txt
===================================================================
--- trunk/sql12/doc/src/main/resources/changes.txt 2012-01-30 20:11:16 UTC (rev 6563)
+++ trunk/sql12/doc/src/main/resources/changes.txt 2012-01-30 20:16:49 UTC (rev 6564)
@@ -28,6 +28,8 @@
Bug fixes:
+SQL Editor: Parser mistook minus signs in SQL IN lists as errors.
+
Error dialog showed up twice when an error occurred during Alias connect.
3026995 Fixed problem in squirrel-sql.bat for 64-bit windows.
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
|