8284920: Incorrect Token type causes XPath expression to return empty result

Reviewed-by: andrew

Author: AntonKozlov

jaxp/src/com/sun/org/apache/xpath/internal/compiler/Lexer.java

@@ -358,7 +358,7 @@ else if ((Token.LPAREN != c) && (Token.RPAREN != c) && (Token.RBRACK != c)) {
 
         addToTokenQueue(pat.substring(i, i + 1));
         break;
-      case Token.COLON :
+      case Token.COLON_CHAR:
         if (i>0)
         {
           if (posOfNSSep == (i - 1))
@@ -613,7 +613,7 @@ private void recordTokenString(Vector targetStrings)
         resetTokenMark(tokPos + 1);
       }
 
-      if (m_processor.lookahead(Token.COLON, 1))
+      if (m_processor.lookahead(Token.COLON_CHAR, 1))
       {
         tokPos += 2;
       }

jaxp/src/com/sun/org/apache/xpath/internal/compiler/Token.java

@@ -45,10 +45,9 @@ public final class Token {
     static final char LPAREN = '(';
     static final char RPAREN = ')';
     static final char COMMA = ',';
-    static final char DOT = '.';
     static final char AT = '@';
     static final char US = '_';
-    static final char COLON = ':';
+    static final char COLON_CHAR = ':';
     static final char SQ = '\'';
     static final char DQ = '"';
     static final char DOLLAR = '$';
@@ -58,6 +57,7 @@ public final class Token {
     static final String DIV = "div";
     static final String MOD = "mod";
     static final String QUO = "quo";
+    static final String DOT = ".";
     static final String DDOT = "..";
     static final String DCOLON = "::";
     static final String ATTR = "attribute";

jaxp/src/com/sun/org/apache/xpath/internal/compiler/XPathParser.java

@@ -1461,7 +1461,7 @@ else if ((null != m_token) && ((('.' == m_tokenChar) && (m_token.length() > 1) &
 
       matchFound = true;
     }
-    else if (lookahead(Token.LPAREN, 1) || (lookahead(Token.COLON, 1) && lookahead(Token.LPAREN, 3)))
+    else if (lookahead(Token.LPAREN, 1) || (lookahead(Token.COLON_CHAR, 1) && lookahead(Token.LPAREN, 3)))
     {
       matchFound = FunctionCall();
     }
@@ -1505,7 +1505,7 @@ protected boolean FunctionCall() throws TransformerException
 
     int opPos = m_ops.getOp(OpMap.MAPINDEX_LENGTH);
 
-    if (lookahead(Token.COLON, 1))
+    if (lookahead(Token.COLON_CHAR, 1))
     {
       appendOp(4, OpCodes.OP_EXTFUNCTION);
 
@@ -1889,7 +1889,7 @@ protected void NodeTest(int axesType) throws TransformerException
       m_ops.setOp(m_ops.getOp(OpMap.MAPINDEX_LENGTH), OpCodes.NODENAME);
       m_ops.setOp(OpMap.MAPINDEX_LENGTH, m_ops.getOp(OpMap.MAPINDEX_LENGTH) + 1);
 
-      if (lookahead(Token.COLON, 1))
+      if (lookahead(Token.COLON_CHAR, 1))
       {
         if (tokenIs(Token.STAR))
         {
@@ -1992,7 +1992,7 @@ protected void PredicateExpr() throws TransformerException
   protected void QName() throws TransformerException
   {
     // Namespace
-    if(lookahead(Token.COLON, 1))
+    if(lookahead(Token.COLON_CHAR, 1))
     {
       m_ops.setOp(m_ops.getOp(OpMap.MAPINDEX_LENGTH), m_queueMark - 1);
       m_ops.setOp(OpMap.MAPINDEX_LENGTH, m_ops.getOp(OpMap.MAPINDEX_LENGTH) + 1);