minor changes
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpeclipse / phpeditor / PHPParser.java
index 3bbd368..7da9027 100644 (file)
@@ -1,22 +1,43 @@
-package net.sourceforge.phpeclipse.phpeditor;
-
-import java.util.HashMap;
-
-import net.sourceforge.phpeclipse.phpeditor.php.PHPKeywords;
-
 /**********************************************************************
-Copyright (c) 2000, 2002 IBM Corp. and others.
+Copyright (c) 2002 Klaus Hartlage - www.eclipseproject.de
 All rights reserved. This program and the accompanying materials
 are made available under the terms of the Common Public License v1.0
 which accompanies this distribution, and is available at
 http://www.eclipse.org/legal/cpl-v10.html
 
 Contributors:
-    IBM Corporation - Initial implementation
     Klaus Hartlage - www.eclipseproject.de
 **********************************************************************/
+package net.sourceforge.phpeclipse.phpeditor;
+
+import java.text.MessageFormat;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.Hashtable;
+
+import net.sourceforge.phpeclipse.PHPeclipsePlugin;
+import net.sourceforge.phpeclipse.actions.PHPStartApacheAction;
+import net.sourceforge.phpeclipse.phpeditor.php.PHPKeywords;
+import org.eclipse.core.resources.IFile;
+import org.eclipse.core.resources.IMarker;
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IPath;
+import org.eclipse.jface.preference.IPreferenceStore;
+import org.eclipse.ui.texteditor.MarkerUtilities;
 
 public class PHPParser extends PHPKeywords {
+  // strings for external parser call
+  private static final String PARSE_ERROR_STRING = "Parse error"; //$NON-NLS-1$
+  private static final String PARSE_WARNING_STRING = "Warning"; //$NON-NLS-1$
+
+  public static final int ERROR = 2;
+  public static final int WARNING = 1;
+  public static final int INFO = 0;
+  private IFile fileToParse;
+  private ArrayList phpList;
+
+  private int currentPHPString;
+  private boolean phpEnd;
 
   private static HashMap keywordMap = null;
   private String str;
@@ -68,7 +89,18 @@ public class PHPParser extends PHPKeywords {
   final static int TT_AMPERSAND = 52;
   final static int TT_DOLLARLISTOPEN = 53;
   final static int TT_TILDE = 54;
-
+  final static int TT_TILDEASSIGN = 55;
+  final static int TT_MODASSIGN = 56;
+  final static int TT_POWASSIGN = 57;
+  final static int TT_RSHIFTASSIGN = 58;
+  final static int TT_LSHIFTASSIGN = 59;
+  final static int TT_ANDASSIGN = 60;
+  final static int TT_QUESTIONMARK = 61;
+  final static int TT_DDOT2 = 62;
+  final static int TT_AT = 63;
+  // final static int TT_HEREDOC = 64;
+
+  final static int TT_DOLLAROPEN = 127;
   final static int TT_ARGOPEN = 128;
   final static int TT_ARGCLOSE = 129;
   final static int TT_LISTOPEN = 130;
@@ -108,20 +140,50 @@ public class PHPParser extends PHPKeywords {
    *@param  sess  Description of Parameter
    *@see
    */
-  public PHPParser() {
+  public PHPParser(IFile fileToParse) {
     if (keywordMap == null) {
       keywordMap = new HashMap();
       for (int i = 0; i < PHP_KEYWORS.length; i++) {
         keywordMap.put(PHP_KEYWORS[i], new Integer(PHP_KEYWORD_TOKEN[i]));
       }
     }
+    this.currentPHPString = 0;
+    this.fileToParse = fileToParse;
+    this.phpList = null;
     this.str = "";
     this.token = TT_EOF;
     this.chIndx = 0;
     this.rowCount = 1;
     this.columnCount = 0;
+    this.phpEnd = false;
+
+    //   getNextToken();
+  }
+
+  /**
+   * Create marker for the parse error
+   */
+  private void setMarker(String message, int lineNumber, int errorLevel) throws CoreException {
+    setMarker(fileToParse, message, lineNumber, errorLevel);
+  }
+
+  public static void setMarker(IFile file, String message, int lineNumber, int errorLevel) throws CoreException {
 
-    getNextToken();
+    Hashtable attributes = new Hashtable();
+    MarkerUtilities.setMessage(attributes, message);
+    switch (errorLevel) {
+      case ERROR :
+        attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_ERROR));
+        break;
+      case WARNING :
+        attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_WARNING));
+        break;
+      case INFO :
+        attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_INFO));
+        break;
+    }
+    MarkerUtilities.setLineNumber(attributes, lineNumber);
+    MarkerUtilities.createMarker(file, attributes, IMarker.PROBLEM);
   }
 
   private void throwSyntaxError(String error) {
@@ -141,12 +203,17 @@ public class PHPParser extends PHPKeywords {
     throw new SyntaxError(rowCount, chIndx - columnCount + 1, str.substring(columnCount, eol), error);
   }
 
+  private void throwSyntaxError(String error, int startRow) {
+
+    throw new SyntaxError(startRow, 0, " ", error);
+  }
+
   /**
    *  Method Declaration.
    *
    *@see
    */
-  void getChar() {
+  private void getChar() {
     if (str.length() > chIndx) {
       ch = str.charAt(chIndx++);
 
@@ -155,13 +222,16 @@ public class PHPParser extends PHPKeywords {
 
     chIndx = str.length() + 1;
     ch = ' ';
-    token = TT_EOF;
+    //  token = TT_EOF;
+    phpEnd = true;
   }
 
   /**
    * gets the next token from input
    */
-  void getNextToken() {
+  private void getNextToken() throws CoreException {
+    phpEnd = false;
+
     while (str.length() > chIndx) {
       ch = str.charAt(chIndx++);
       token = TT_UNDEFINED;
@@ -170,9 +240,22 @@ public class PHPParser extends PHPKeywords {
         columnCount = chIndx;
         continue; // while loop
       }
-
+      if (str.length() == chIndx) {
+        phpEnd = true;
+      }
       if (!Character.isWhitespace(ch)) {
-        if ((ch >= 'a' && ch <= 'z') || (ch >= 'A' && ch <= 'Z') || (ch == '_') || (ch == '$') || (ch == '@')) {
+        if (ch == '$') {
+          if (str.length() > chIndx) {
+            if (str.charAt(chIndx) == '{') {
+              chIndx++;
+              token = TT_DOLLAROPEN;
+              return;
+            }
+          }
+          getIdentifier();
+          return;
+        }
+        if ((ch >= 'a' && ch <= 'z') || (ch >= 'A' && ch <= 'Z') || (ch == '_') || (ch == '$')) {
           getIdentifier();
           return;
         }
@@ -237,6 +320,7 @@ public class PHPParser extends PHPKeywords {
         } else if (ch == '\'') {
           // read string until end
           boolean openString = true;
+          int startRow = rowCount;
           while (str.length() > chIndx) {
             ch = str.charAt(chIndx++);
             if (ch == '\\') {
@@ -252,8 +336,32 @@ public class PHPParser extends PHPKeywords {
             }
           }
           if (openString) {
-            throwSyntaxError("Open string character \"'\" at end of file.");
+            throwSyntaxError("Open string character \"'\" at end of file.", startRow);
+          }
+          token = TT_STRING_CONSTANT;
+          return;
+        } else if (ch == '`') {
+          // read string until end
+          boolean openString = true;
+          int startRow = rowCount;
+          while (str.length() > chIndx) {
+            ch = str.charAt(chIndx++);
+            if (ch == '\\') {
+              if (str.length() > chIndx) {
+                ch = str.charAt(chIndx++);
+              }
+            } else if (ch == '`') {
+              openString = false;
+              break;
+            } else if (ch == '\n') {
+              rowCount++;
+              columnCount = chIndx;
+            }
+          }
+          if (openString) {
+            throwSyntaxError("Open string character \"`\" at end of file.", startRow);
           }
+          setMarker("Other string delimiters prefered (found \"`\").", rowCount, PHPParser.INFO);
           token = TT_STRING_CONSTANT;
           return;
         }
@@ -288,9 +396,22 @@ public class PHPParser extends PHPKeywords {
             token = TT_COMMA;
 
             break;
+          case '?' :
+            token = TT_QUESTIONMARK;
+            break;
+          case '@' :
+            token = TT_AT;
+            break;
           case '~' :
             token = TT_TILDE;
+            if (str.length() > chIndx) {
+              if (str.charAt(chIndx) == '=') {
+                chIndx++;
+                token = TT_TILDEASSIGN;
 
+                break;
+              }
+            }
             break;
           case '.' :
             token = TT_DOT;
@@ -310,7 +431,14 @@ public class PHPParser extends PHPKeywords {
             break;
           case '%' :
             token = TT_MOD;
+            if (str.length() > chIndx) {
+              if (str.charAt(chIndx) == '=') {
+                chIndx++;
+                token = TT_MODASSIGN;
 
+                break;
+              }
+            }
             break;
           case ';' :
             token = TT_SEMICOLON;
@@ -318,7 +446,14 @@ public class PHPParser extends PHPKeywords {
             break;
           case '^' :
             token = TT_POW;
+            if (str.length() > chIndx) {
+              if (str.charAt(chIndx) == '=') {
+                chIndx++;
+                token = TT_POWASSIGN;
 
+                break;
+              }
+            }
             break;
           case '/' :
             token = TT_DIV;
@@ -447,13 +582,18 @@ public class PHPParser extends PHPKeywords {
               if (str.charAt(chIndx) == '=') {
                 chIndx++;
                 token = TT_GREATEREQUAL;
-
                 break;
               }
               if (str.charAt(chIndx) == '>') {
                 chIndx++;
                 token = TT_RSHIFT;
-
+                if (str.length() > chIndx) {
+                  if (str.charAt(chIndx) == '=') {
+                    chIndx++;
+                    token = TT_RSHIFTASSIGN;
+                    break;
+                  }
+                }
                 break;
               }
             }
@@ -472,7 +612,35 @@ public class PHPParser extends PHPKeywords {
               if (str.charAt(chIndx) == '<') {
                 chIndx++;
                 token = TT_LSHIFT;
-
+                if (str.charAt(chIndx) == '<') {
+                  // heredoc
+                  int startRow = rowCount;
+                  if (str.length() > chIndx) {
+
+                    ch = str.charAt(++chIndx);
+                    if ((ch >= 'a' && ch <= 'z') || (ch >= 'A' && ch <= 'Z') || (ch == '_')) {
+                      chIndx++;
+                      getIdentifier();
+                      token = TT_STRING_CONSTANT;
+                      while (str.length() > chIndx) {
+                        ch = str.charAt(chIndx++);
+                        if (ch == '\n') {
+                          if (str.length() >= chIndx + identifier.length()) {
+                            if (str.substring(chIndx, chIndx + identifier.length()).equals(identifier)) {
+                              chIndx += identifier.length();
+                              return;
+                            }
+                          }
+                        }
+                      }
+                    }
+                  }
+                  throwSyntaxError("Open heredoc syntax after operator '<<<'.", startRow);
+                } else if (str.charAt(chIndx) == '=') {
+                  chIndx++;
+                  token = TT_LSHIFTASSIGN;
+                  break;
+                }
                 break;
               }
             }
@@ -493,23 +661,30 @@ public class PHPParser extends PHPKeywords {
 
             break;
           case '&' :
+            token = TT_AMPERSAND;
             if (str.length() > chIndx) {
               if (str.charAt(chIndx) == '&') {
                 chIndx++;
                 token = TT_AND;
-
                 break;
-              } else {
-                token = TT_AMPERSAND;
-
+              }
+              if (str.charAt(chIndx) == '=') {
+                chIndx++;
+                token = TT_ANDASSIGN;
                 break;
               }
+              break;
             }
 
             break;
           case ':' :
             token = TT_DDOT;
-
+            if (str.length() > chIndx) {
+              if (str.charAt(chIndx) == ':') {
+                chIndx++;
+                token = TT_DDOT2;
+              }
+            }
             break;
           case '#' :
             token = TT_HASH;
@@ -534,9 +709,27 @@ public class PHPParser extends PHPKeywords {
     chIndx = str.length() + 1;
     ch = ' ';
     token = TT_EOF;
+    phpEnd = true;
+    PHPString temp;
+    if (phpList != null) {
+      if (currentPHPString < phpList.size()) {
+        token = TT_UNDEFINED;
+        temp = (PHPString) phpList.get(currentPHPString++);
+        this.str = temp.getPHPString();
+        this.token = TT_EOF;
+        this.chIndx = 0;
+        this.rowCount = temp.getLineNumber();
+        this.columnCount = 0;
+        getNextToken();
+        phpEnd = true;
+      } else {
+        token = TT_UNDEFINED;
+        return;
+      }
+    }
   }
 
-  void getIdentifier() {
+  private void getIdentifier() {
     StringBuffer ident = new StringBuffer();
 
     ident.append(ch);
@@ -559,7 +752,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  void getNumber() {
+  private void getNumber() {
     StringBuffer inum = new StringBuffer();
     char dFlag = ' ';
     int numFormat = 10;
@@ -644,44 +837,245 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void start(String s, int rowCount) throws SyntaxError {
-    // start up
-    this.str = s;
-    this.token = TT_EOF;
-    this.chIndx = 0;
-    this.rowCount = rowCount;
-    this.columnCount = 0;
-    getNextToken();
-    if (token != TT_EOF) {
-      statementList();
-    }
-    if (token != TT_EOF) {
-      if (token == TT_ARGCLOSE) {
-        throwSyntaxError("too many closing ')'; end-of-file not reached");
+  public void htmlParse(String input) {
+    int lineNumber = 1;
+    int startLineNumber = 1;
+    int startIndex = 0;
+    char ch;
+    char ch2;
+    boolean phpMode = false;
+    boolean phpFound = false;
+
+    phpList = new ArrayList();
+    currentPHPString = 0;
+
+    try {
+      int i = 0;
+      while (i < input.length()) {
+        ch = input.charAt(i++);
+        if (ch == '\n') {
+          lineNumber++;
+        }
+        if ((!phpMode) && ch == '<') {
+          ch2 = input.charAt(i++);
+          if (ch2 == '?') {
+            ch2 = input.charAt(i++);
+            if (Character.isWhitespace(ch2)) {
+              // php start 
+              phpMode = true;
+              phpFound = true;
+              startIndex = i;
+              startLineNumber = lineNumber;
+              continue;
+            } else if (ch2 == 'p') {
+              ch2 = input.charAt(i++);
+              if (ch2 == 'h') {
+                ch2 = input.charAt(i++);
+                if (ch2 == 'p') {
+                  phpMode = true;
+                  phpFound = true;
+                  startIndex = i;
+                  startLineNumber = lineNumber;
+                  continue;
+                }
+                i--;
+              }
+              i--;
+            } else if (ch2 == 'P') {
+              ch2 = input.charAt(i++);
+              if (ch2 == 'H') {
+                ch2 = input.charAt(i++);
+                if (ch2 == 'P') {
+                  phpMode = true;
+                  phpFound = true;
+                  startIndex = i;
+                  startLineNumber = lineNumber;
+                  continue;
+                }
+                i--;
+              }
+              i--;
+            }
+            i--;
+          }
+          i--;
+        }
+
+        if (phpMode) {
+          if (ch == '/' && i < input.length()) {
+            ch2 = input.charAt(i++);
+            if (ch2 == '/') {
+              while (i < input.length()) {
+                ch = input.charAt(i++);
+                if (ch == '?' && i < input.length()) {
+                  ch2 = input.charAt(i++);
+                  if (ch2 == '>') {
+                    // php end
+                    phpMode = false;
+                    phpList.add(new PHPString(input.substring(startIndex, i - 2), startLineNumber));
+                    continue;
+                  }
+                  i--;
+                } else if (ch == '\n') {
+                  lineNumber++;
+                  break;
+                }
+              }
+              continue;
+            } else if (ch2 == '*') {
+              // multi-line comment
+              while (i < input.length()) {
+                ch = input.charAt(i++);
+                if (ch == '\n') {
+                  lineNumber++;
+                } else if (ch == '*' && i < input.length()) {
+                  ch2 = input.charAt(i++);
+                  if (ch2 == '/') {
+                    break;
+                  }
+                  i--;
+                }
+              }
+              continue;
+            } else {
+              i--;
+            }
+          } else if (ch == '#') {
+            while (i < input.length()) {
+              ch = input.charAt(i++);
+              if (ch == '?' && i < input.length()) {
+                ch2 = input.charAt(i++);
+                if (ch2 == '>') {
+                  // php end
+                  phpMode = false;
+                  phpList.add(new PHPString(input.substring(startIndex, i - 2), startLineNumber));
+                  continue;
+                }
+                i--;
+              } else if (ch == '\n') {
+                lineNumber++;
+                break;
+              }
+            }
+            continue;
+          } else if (ch == '"') {
+            ch = ' ';
+            while (i < input.length()) {
+              ch = input.charAt(i++);
+              if (ch == '\n') {
+                lineNumber++;
+              } else if (ch == '\\' && i < input.length()) { // escape
+                i++;
+              } else if (ch == '"') {
+                break;
+              }
+            }
+            continue;
+          } else if (ch == '\'') {
+            ch = ' ';
+            while (i < input.length()) {
+              ch = input.charAt(i++);
+              if (ch == '\n') {
+                lineNumber++;
+              } else if (ch == '\\' && i < input.length()) { // escape
+                i++;
+              } else if (ch == '\'') {
+                break;
+              }
+            }
+            continue;
+          }
+
+          if (ch == '?' && i < input.length()) {
+            ch2 = input.charAt(i++);
+            if (ch2 == '>') {
+              // php end
+              phpMode = false;
+              phpList.add(new PHPString(input.substring(startIndex, i - 2), startLineNumber));
+              continue;
+            }
+            i--;
+          }
+        }
       }
-      if (token == TT_LISTCLOSE) {
-        throwSyntaxError("too many closing '}'; end-of-file not reached");
+
+      if (!phpFound) {
+        setMarker("No PHP source code found.", lineNumber, PHPParser.INFO);
+      } else {
+        if (phpMode) {
+          setMarker("Open PHP tag at end of file.", lineNumber, PHPParser.INFO);
+          phpList.add(new PHPString(input.substring(startIndex, i - 2), startLineNumber));
+        }
+        //        for (int j=0;j<phpList.size();j++) {
+        //          String temp = ((PHPString)phpList.get(j)).getPHPString();
+        //          int startIndx = temp.length()-10;
+        //          if (startIndx<0) {
+        //            startIndx = 0; 
+        //          }
+        //          System.out.println(temp.substring(startIndx)+"?>");
+        //        }
+        phpParse(null, 1);
+        //        PHPString temp;
+        //        for(int j=0;j<phpList.size();j++) {
+        //          temp = (PHPString) phpList.get(j);
+        //          parser.start(temp.getPHPString(), temp.getLineNumber());
+        //        } 
+      }
+    } catch (CoreException e) {
+    }
+  }
+
+  public void phpParse(String s, int rowCount) throws CoreException {
+    // start up
+    try {
+      this.str = s;
+      if (s == null) {
+        if (phpList.size() != 0) {
+          this.str = ((PHPString) phpList.get(currentPHPString++)).getPHPString();
+        }
       }
-      if (token == TT_PARTCLOSE) {
-        throwSyntaxError("too many closing ']'; end-of-file not reached");
+      this.token = TT_EOF;
+      this.chIndx = 0;
+      this.rowCount = rowCount;
+      this.columnCount = 0;
+      this.phpEnd = false;
+      getNextToken();
+      if (token != TT_EOF && token != TT_UNDEFINED) {
+        statementList();
       }
+      if (token != TT_EOF && token != TT_UNDEFINED) {
+        if (token == TT_ARGCLOSE) {
+          throwSyntaxError("too many closing ')'; end-of-file not reached");
+        }
+        if (token == TT_LISTCLOSE) {
+          throwSyntaxError("too many closing '}'; end-of-file not reached");
+        }
+        if (token == TT_PARTCLOSE) {
+          throwSyntaxError("too many closing ']'; end-of-file not reached");
+        }
 
-      if (token == TT_ARGOPEN) {
-        throwSyntaxError("read character '('; end-of-file not reached");
-      }
-      if (token == TT_LISTOPEN) {
-        throwSyntaxError("read character '{';  end-of-file not reached");
+        if (token == TT_ARGOPEN) {
+          throwSyntaxError("read character '('; end-of-file not reached");
+        }
+        if (token == TT_LISTOPEN) {
+          throwSyntaxError("read character '{';  end-of-file not reached");
+        }
+        if (token == TT_PARTOPEN) {
+          throwSyntaxError("read character '[';  end-of-file not reached");
+        }
+
+        throwSyntaxError("end-of-file not reached");
       }
-      if (token == TT_PARTOPEN) {
-        throwSyntaxError("read character '[';  end-of-file not reached");
+    } catch (SyntaxError err) {
+      if (s != null) {
+        throw err;
+      } else {
+        setMarker(err.getMessage(), err.getLine(), ERROR);
       }
-
-      throwSyntaxError("end-of-file not reached");
     }
-
   }
 
-  public void statementList() {
+  private void statementList() throws CoreException {
     do {
       statement();
       if ((token == TT_LISTCLOSE)
@@ -693,13 +1087,14 @@ public class PHPParser extends PHPKeywords {
         || (token == TT_endforeach)
         || (token == TT_endwhile)
         || (token == TT_endswitch)
-        || (token == TT_EOF)) {
+        || (token == TT_EOF)
+        || (token == TT_UNDEFINED)) {
         return;
       }
     } while (true);
   }
 
-  public void compoundStatement() {
+  private void compoundStatement() throws CoreException {
     // '{' [statement-list] '}'
     if (token == TT_LISTOPEN) {
       getNextToken();
@@ -716,103 +1111,133 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void statement() {
-    if (token > TT_KEYWORD && token != TT_list) {
-      String keyword = identifier;
-      if (token == TT_include || token == TT_include_once) {
+  private void statement() throws CoreException {
+    //   if (token > TT_KEYWORD && token != TT_list && token != TT_new) {
+    String keyword = identifier;
+    if (token == TT_include || token == TT_include_once) {
+      getNextToken();
+      expression();
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        expression();
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' character after 'include' or 'include_once' expected.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' character after 'include' or 'include_once' expected.");
         }
-        return;
-      } else if (token == TT_require || token == TT_require_once) {
+      }
+      return;
+    } else if (token == TT_require || token == TT_require_once) {
+      getNextToken();
+      //constant();
+      expression();
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        //constant();
-        expression();
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' character after 'require' or 'require_once' expected.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' character after 'require' or 'require_once' expected.");
         }
-        return;
-      } else if (token == TT_if) {
+      }
+      return;
+    } else if (token == TT_if) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'if' keyword.");
-        }
-        expression();
-        if (token == TT_ARGCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("')' expected after 'if' condition.");
-        }
-        ifStatement();
-        return;
+      } else {
+        throwSyntaxError("'(' expected after 'if' keyword.");
+      }
+      expression();
+      if (token == TT_ARGCLOSE) {
+        getNextToken();
+      } else {
+        throwSyntaxError("')' expected after 'if' condition.");
+      }
+      ifStatement();
+      return;
 
-      } else if (token == TT_switch) {
+    } else if (token == TT_switch) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'switch' keyword.");
-        }
-        expression();
-        if (token == TT_ARGCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("')' expected after 'switch' condition.");
-        }
-        switchStatement();
-        return;
-      } else if (token == TT_for) {
+      } else {
+        throwSyntaxError("'(' expected after 'switch' keyword.");
+      }
+      expression();
+      if (token == TT_ARGCLOSE) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'for' keyword.");
-        }
+      } else {
+        throwSyntaxError("')' expected after 'switch' condition.");
+      }
+      switchStatement();
+      return;
+    } else if (token == TT_for) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'(' expected after 'for' keyword.");
+      }
+      if (token == TT_SEMICOLON) {
+        getNextToken();
+      } else {
+        expressionList();
         if (token == TT_SEMICOLON) {
           getNextToken();
         } else {
-          expression();
-          if (token == TT_SEMICOLON) {
-            getNextToken();
-          } else {
-            throwSyntaxError("';' character after 'for' expected.");
-          }
+          throwSyntaxError("';' expected after 'for'.");
         }
+      }
+      if (token == TT_SEMICOLON) {
+        getNextToken();
+      } else {
+        expressionList();
         if (token == TT_SEMICOLON) {
           getNextToken();
         } else {
-          expression();
-          if (token == TT_SEMICOLON) {
-            getNextToken();
-          } else {
-            throwSyntaxError("';' character after 'for' expected.");
-          }
+          throwSyntaxError("';' expected after 'for'.");
         }
+      }
+      if (token == TT_ARGCLOSE) {
+        getNextToken();
+      } else {
+        expressionList();
         if (token == TT_ARGCLOSE) {
           getNextToken();
         } else {
-          expression();
-          if (token == TT_ARGCLOSE) {
-            getNextToken();
-          } else {
-            throwSyntaxError("')' expected after 'for' condition.");
-          }
+          throwSyntaxError("')' expected after 'for'.");
         }
-        forStatement();
-        return;
-      } else if (token == TT_while) {
+      }
+      forStatement();
+      return;
+    } else if (token == TT_while) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'(' expected after 'while' keyword.");
+      }
+      expression();
+      if (token == TT_ARGCLOSE) {
+        getNextToken();
+      } else {
+        throwSyntaxError("')' expected after 'while' condition.");
+      }
+      whileStatement();
+      return;
+    } else if (token == TT_do) {
+      getNextToken();
+      if (token == TT_LISTOPEN) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'{' expected after 'do' keyword.");
+      }
+      if (token != TT_LISTCLOSE) {
+        statementList();
+      }
+      if (token == TT_LISTCLOSE) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'}' expected after 'do' keyword.");
+      }
+      if (token == TT_while) {
         getNextToken();
         if (token == TT_ARGOPEN) {
           getNextToken();
@@ -825,199 +1250,170 @@ public class PHPParser extends PHPKeywords {
         } else {
           throwSyntaxError("')' expected after 'while' condition.");
         }
-        whileStatement();
-        return;
-      } else if (token == TT_do) {
+      } else {
+        throwSyntaxError("'while' expected after 'do' keyword.");
+      }
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        if (token == TT_LISTOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'{' expected after 'do' keyword.");
-        }
-        if (token != TT_LISTCLOSE) {
-          statementList();
-        }
-        if (token == TT_LISTCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'}' expected after 'do' keyword.");
-        }
-        if (token == TT_while) {
-          getNextToken();
-          if (token == TT_ARGOPEN) {
-            getNextToken();
-          } else {
-            throwSyntaxError("'(' expected after 'while' keyword.");
-          }
-          expression();
-          if (token == TT_ARGCLOSE) {
-            getNextToken();
-          } else {
-            throwSyntaxError("')' expected after 'while' condition.");
-          }
-        } else {
-          throwSyntaxError("'while' expected after 'do' keyword.");
-        }
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after do-while statement.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' expected after do-while statement.");
         }
-        return;
-      } else if (token == TT_foreach) {
+      }
+      return;
+    } else if (token == TT_foreach) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'(' expected after 'foreach' keyword.");
+      }
+      expression();
+      if (token == TT_as) {
+        getNextToken();
+      } else {
+        throwSyntaxError("'as' expected after 'foreach' exxpression.");
+      }
+      variable();
+      if (token == TT_FOREACH) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'foreach' keyword.");
-        }
-        expression();
-        if (token == TT_as) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'as' expected after 'foreach' exxpression.");
-        }
         variable();
-        if (token == TT_FOREACH) {
-          getNextToken();
-          variable();
-        }
-        if (token == TT_ARGCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("')' expected after 'foreach' expression.");
-        }
-        foreachStatement();
-        return;
-
-      } else if (token == TT_continue || token == TT_break || token == TT_return) {
+      }
+      if (token == TT_ARGCLOSE) {
         getNextToken();
-        if (token != TT_SEMICOLON) {
-          expression();
-        }
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'continue', 'break' or 'return'.");
-          }
-        }
-        return;
+      } else {
+        throwSyntaxError("')' expected after 'foreach' expression.");
+      }
+      foreachStatement();
+      return;
 
-      } else if (token == TT_echo) {
+    } else if (token == TT_continue || token == TT_break || token == TT_return) {
+      getNextToken();
+      if (token != TT_SEMICOLON) {
+        expression();
+      }
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        expressionList();
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'echo' statement.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' expected after 'continue', 'break' or 'return'.");
         }
-        return;
+      }
+      return;
 
-      } else if (token == TT_print) {
+    } else if (token == TT_echo) {
+      getNextToken();
+      expressionList();
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        expression();
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'print' statement.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' expected after 'echo' statement.");
         }
-        return;
-
-      } else if (token == TT_global || token == TT_static) {
+      }
+      return;
+      //    } else if (token == TT_print) {
+      //      getNextToken();
+      //      expression();
+      //      if (token == TT_SEMICOLON) {
+      //        getNextToken();
+      //      } else {
+      //        if (!phpEnd) {
+      //          throwSyntaxError("';' expected after 'print' statement.");
+      //        }
+      //      }
+      //      return;
+
+    } else if (token == TT_global || token == TT_static) {
+      getNextToken();
+      variableList();
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        variableList();
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'global' or 'static' statement.");
-          }
+      } else {
+        if (!phpEnd) {
+          throwSyntaxError("';' expected after 'global' or 'static' statement.");
         }
-        return;
+      }
+      return;
 
-      } else if (token == TT_unset) {
+      //      } else if (token == TT_unset) {
+      //        getNextToken();
+      //        if (token == TT_ARGOPEN) {
+      //          getNextToken();
+      //        } else {
+      //          throwSyntaxError("'(' expected after 'unset' keyword.");
+      //        }
+      //        variableList();
+      //        if (token == TT_ARGCLOSE) {
+      //          getNextToken();
+      //        } else {
+      //          throwSyntaxError("')' expected after 'unset' statement.");
+      //        }
+      //        if (token == TT_SEMICOLON) {
+      //          getNextToken();
+      //        } else {
+      //          if (!phpEnd) {
+      //            throwSyntaxError("';' expected after 'unset' statement.");
+      //          }
+      //        }
+      //        return;
+
+      //      } else if (token == TT_exit || token == TT_die) {
+      //        getNextToken();
+      //        if (token != TT_SEMICOLON) {
+      //          exitStatus();
+      //        }
+      //        if (token == TT_SEMICOLON) {
+      //          getNextToken();
+      //        } else {
+      //          if (!phpEnd) {
+      //            throwSyntaxError("';' expected after 'exit' or 'die' statement.");
+      //          }
+      //        }
+      //        return;
+
+    } else if (token == TT_define) {
+      getNextToken();
+      if (token == TT_ARGOPEN) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'unset' keyword.");
-        }
-        variableList();
-        if (token == TT_ARGCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("')' expected after 'unset' statement.");
-        }
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'unset' statement.");
-          }
-        }
-        return;
-
-      } else if (token == TT_exit || token == TT_die) {
+      } else {
+        throwSyntaxError("'(' expected after 'define' keyword.");
+      }
+      expression();
+      if (token == TT_COMMA) {
         getNextToken();
-        if (token != TT_SEMICOLON) {
-          exitStatus();
-        }
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'exit' or 'die' statement.");
-          }
-        }
-        return;
-
-      } else if (token == TT_define) {
+      } else {
+        throwSyntaxError("',' expected after first 'define' constant.");
+      }
+      expression();
+      if (token == TT_COMMA) {
         getNextToken();
-        if (token == TT_ARGOPEN) {
-          getNextToken();
-        } else {
-          throwSyntaxError("'(' expected after 'define' keyword.");
-        }
-        constant();
-        if (token == TT_COMMA) {
-          getNextToken();
-        } else {
-          throwSyntaxError("',' expected after first 'define' constant.");
-        }
-        constant();
-        if (token == TT_ARGCLOSE) {
-          getNextToken();
-        } else {
-          throwSyntaxError("')' expected after 'define' statement.");
-        }
-        if (token == TT_SEMICOLON) {
-          getNextToken();
-        } else {
-          if (token != TT_EOF) {
-            throwSyntaxError("';' expected after 'define' statement.");
-          }
-        }
-        return;
-      } else if (token == TT_function) {
+        expression();
+      }
+      if (token == TT_ARGCLOSE) {
         getNextToken();
-        functionDefinition();
-        return;
-      } else if (token == TT_class) {
+      } else {
+        throwSyntaxError("')' expected after 'define' statement.");
+      }
+      if (token == TT_SEMICOLON) {
         getNextToken();
-        classDeclarator();
-        classBody();
-        return;
       } else {
-        throwSyntaxError("Unexpected keyword '" + keyword + "'");
+        if (!phpEnd) {
+          throwSyntaxError("';' expected after 'define' statement.");
+        }
       }
-
+      return;
+    } else if (token == TT_function) {
+      getNextToken();
+      functionDefinition();
+      return;
+    } else if (token == TT_class) {
+      getNextToken();
+      classDeclarator();
+      classBody();
+      return;
+      //      } else {
+      //        throwSyntaxError("Unexpected keyword '" + keyword + "'");
     } else if (token == TT_LISTOPEN) {
       // compoundStatement
       getNextToken();
@@ -1038,15 +1434,14 @@ public class PHPParser extends PHPKeywords {
         getNextToken();
         return;
       } else {
-        if (token != TT_EOF) {
+        if (!phpEnd) {
           throwSyntaxError("';' expected after expression.");
         }
       }
     }
-
   }
 
-  public void classDeclarator() {
+  private void classDeclarator() throws CoreException {
     //identifier
     //identifier 'extends' identifier
     if (token == TT_IDENTIFIER) {
@@ -1064,7 +1459,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void classBody() {
+  private void classBody() throws CoreException {
     //'{' [class-element-list] '}'
     if (token == TT_LISTOPEN) {
       getNextToken();
@@ -1081,13 +1476,13 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void classElementList() {
+  private void classElementList() throws CoreException {
     do {
       classElement();
-    } while (token != TT_function || token != TT_var);
+    } while (token == TT_function || token == TT_var);
   }
 
-  public void classElement() {
+  private void classElement() throws CoreException {
     //class-property
     //function-definition
     if (token == TT_function) {
@@ -1101,36 +1496,41 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void classProperty() {
+  private void classProperty() throws CoreException {
     //'var' variable ';'
     //'var' variable '=' constant ';'
-    if (token == TT_VARIABLE) {
-      getNextToken();
-      if (token == TT_ASSIGN) {
+    do {
+      if (token == TT_VARIABLE) {
         getNextToken();
-        constant();
-        if (token == TT_SEMICOLON) {
+        if (token == TT_ASSIGN) {
           getNextToken();
-        } else {
-          throwSyntaxError("';' expected after variable declaration.");
+          constant();
         }
-      } else if (token == TT_SEMICOLON) {
-        getNextToken();
       } else {
-        throwSyntaxError("';' or '=' expected after variable declaration.");
+        throwSyntaxError("Variable expected after keyword 'var'.");
+      }
+      if (token != TT_COMMA) {
+        break;
       }
+      getNextToken();
+    } while (true);
+    if (token == TT_SEMICOLON) {
+      getNextToken();
     } else {
-      throwSyntaxError("Variable expected after keyword 'var'.");
+      throwSyntaxError("';' expected after variable declaration.");
     }
   }
 
-  public void functionDefinition() {
+  private void functionDefinition() throws CoreException {
     functionDeclarator();
     compoundStatement();
   }
 
-  public void functionDeclarator() {
+  private void functionDeclarator() throws CoreException {
     //identifier '(' [parameter-list] ')'
+    if (token == TT_AMPERSAND) {
+      getNextToken();
+    }
     if (token == TT_IDENTIFIER) {
       getNextToken();
       if (token == TT_ARGOPEN) {
@@ -1149,7 +1549,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
   //
-  public void parameterList() {
+  private void parameterList() throws CoreException {
     //parameter-declaration
     //parameter-list ',' parameter-declaration
     do {
@@ -1161,9 +1561,17 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void parameterDeclaration() {
+  private void parameterDeclaration() throws CoreException {
     //variable
     //variable-reference
+    if (token == TT_AMPERSAND) {
+      getNextToken();
+      if (token == TT_VARIABLE) {
+        getNextToken();
+      } else {
+        throwSyntaxError("Variable expected after reference operator '&'.");
+      }
+    }
     //variable '=' constant
     if (token == TT_VARIABLE) {
       getNextToken();
@@ -1175,7 +1583,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void labeledStatementList() {
+  private void labeledStatementList() throws CoreException {
     if (token != TT_case && token != TT_default) {
       throwSyntaxError("'case' or 'default' expected.");
     }
@@ -1185,6 +1593,16 @@ public class PHPParser extends PHPKeywords {
         constant();
         if (token == TT_DDOT) {
           getNextToken();
+          if (token == TT_case || token == TT_default) { // empty case statement ?
+            continue;
+          }
+          statementList();
+        } else if (token == TT_SEMICOLON) {
+          setMarker("':' expected after 'case' keyword found ';'.", rowCount, PHPParser.INFO);
+          getNextToken();
+          if (token == TT_case) { // empty case statement ?
+            continue;
+          }
           statementList();
         } else {
           throwSyntaxError("':' character after 'case' constant expected.");
@@ -1224,11 +1642,11 @@ public class PHPParser extends PHPKeywords {
   //    }
   //  }
 
-  public void expressionStatement() {
-  }
+  //  public void expressionStatement() {
+  //  }
 
-  public void inclusionStatement() {
-  }
+  //  private void inclusionStatement() {
+  //  }
 
   //  public void compoundStatement() {
   //  }
@@ -1254,7 +1672,7 @@ public class PHPParser extends PHPKeywords {
   //  public void definitionStatement() {
   //  }
 
-  public void ifStatement() {
+  private void ifStatement() throws CoreException {
     // ':' statement-list [elseif-list] [else-colon-statement] 'endif' ';'
     if (token == TT_DDOT) {
       getNextToken();
@@ -1311,7 +1729,8 @@ public class PHPParser extends PHPKeywords {
       }
     }
   }
-  public void elseifStatementList() {
+
+  private void elseifStatementList() throws CoreException {
     do {
       elseifStatement();
       switch (token) {
@@ -1338,7 +1757,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void elseifStatement() {
+  private void elseifStatement() throws CoreException {
     if (token == TT_ARGOPEN) {
       getNextToken();
       expression();
@@ -1354,7 +1773,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void switchStatement() {
+  private void switchStatement() throws CoreException {
     if (token == TT_DDOT) {
       // ':' [labeled-statement-list] 'endswitch' ';'
       getNextToken();
@@ -1384,7 +1803,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void forStatement() {
+  private void forStatement() throws CoreException {
     if (token == TT_DDOT) {
       getNextToken();
       statementList();
@@ -1401,7 +1820,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void whileStatement() {
+  private void whileStatement() throws CoreException {
     // ':' statement-list 'endwhile' ';'
     if (token == TT_DDOT) {
       getNextToken();
@@ -1419,7 +1838,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void foreachStatement() {
+  private void foreachStatement() throws CoreException {
     if (token == TT_DDOT) {
       getNextToken();
       statementList();
@@ -1436,7 +1855,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void exitStatus() {
+  private void exitStatus() throws CoreException {
     if (token == TT_ARGOPEN) {
       getNextToken();
     } else {
@@ -1452,7 +1871,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void expressionList() {
+  private void expressionList() throws CoreException {
     do {
       expression();
       if (token == TT_COMMA) {
@@ -1463,7 +1882,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void expression() {
+  private void expression() throws CoreException {
     //    if (token == TT_STRING_CONSTANT || token == TT_INTERPOLATED_STRING) {
     //      getNextToken();
     //    } else {
@@ -1474,7 +1893,7 @@ public class PHPParser extends PHPKeywords {
     //    }
   }
 
-  public void postfixExpression() {
+  private void postfixExpression() throws CoreException {
     String ident;
     boolean castFlag = false;
     switch (token) {
@@ -1533,6 +1952,14 @@ public class PHPParser extends PHPKeywords {
       case TT_INT_NUMBER :
         getNextToken();
         break;
+      case TT_DOLLAROPEN :
+        getNextToken();
+        expression();
+        if (token != TT_LISTCLOSE) {
+          throwSyntaxError("'}' expected after indirect variable token '${'.");
+        }
+        getNextToken();
+        break;
       case TT_VARIABLE :
         ident = identifier;
         getNextToken();
@@ -1543,6 +1970,15 @@ public class PHPParser extends PHPKeywords {
             throwSyntaxError("'}' expected after variable '" + ident + "' in variable-expression.");
           }
           getNextToken();
+        } else if (token == TT_ARGOPEN) {
+          getNextToken();
+          if (token != TT_ARGCLOSE) {
+            expressionList();
+            if (token != TT_ARGCLOSE) {
+              throwSyntaxError("')' expected after variable '" + ident + "' in postfix-expression.");
+            }
+          }
+          getNextToken();
         }
         break;
       case TT_IDENTIFIER :
@@ -1559,6 +1995,17 @@ public class PHPParser extends PHPKeywords {
           getNextToken();
         }
         break;
+      case TT_print :
+        getNextToken();
+        expression();
+        //        if (token == TT_SEMICOLON) {
+        //          getNextToken();
+        //        } else {
+        //          if (!phpEnd) {
+        //            throwSyntaxError("';' expected after 'print' statement.");
+        //          }
+        //        }
+        break;
       case TT_list :
         getNextToken();
         if (token == TT_ARGOPEN) {
@@ -1579,6 +2026,33 @@ public class PHPParser extends PHPKeywords {
           throwSyntaxError("'(' expected after 'list' keyword.");
         }
         break;
+        //      case TT_exit :
+        //        getNextToken();
+        //        if (token != TT_SEMICOLON) {
+        //          exitStatus();
+        //        }
+        //        if (token == TT_SEMICOLON) {
+        //          getNextToken();
+        //        } else {
+        //          if (!phpEnd) {
+        //            throwSyntaxError("';' expected after 'exit' expression.");
+        //          }
+        //        }
+        //        break;
+        //      case TT_die :
+        //        getNextToken();
+        //        if (token != TT_SEMICOLON) {
+        //          exitStatus();
+        //        }
+        //        if (token == TT_SEMICOLON) {
+        //          getNextToken();
+        //        } else {
+        //          if (!phpEnd) {
+        //            throwSyntaxError("';' expected after 'die' expression.");
+        //          }
+        //        }
+        //        break;
+
         //      case TT_array :
         //        getNextToken();
         //        if (token == TT_ARGOPEN) {
@@ -1611,43 +2085,59 @@ public class PHPParser extends PHPKeywords {
           }
           getNextToken();
           break;
+        case TT_DDOT2 : // ::
         case TT_REF : // ->
           getNextToken();
-          switch (token) {
-            case TT_VARIABLE :
-              ident = identifier;
+          if (token > TT_KEYWORD) {
+            ident = identifier;
+            setMarker("Avoid using keyword '" + ident + "' as variable name.", rowCount, PHPParser.INFO);
+            getNextToken();
+            if (token == TT_ARGOPEN) {
               getNextToken();
-              //              if (token == TT_ARGOPEN) {
-              //                getNextToken();
-              //                expressionList();
-              //                if (token != TT_ARGCLOSE) {
-              //                  throwSyntaxError(") expected after variable '" + ident + "'.");
-              //                }
-              //                getNextToken();
-              //              }
-              break;
-            case TT_IDENTIFIER :
-              ident = identifier;
+              expressionList();
+              if (token != TT_ARGCLOSE) {
+                throwSyntaxError(") expected after identifier '" + ident + "'.");
+              }
               getNextToken();
-              if (token == TT_ARGOPEN) {
+            }
+            break;
+          } else {
+            switch (token) {
+              case TT_VARIABLE :
+                ident = identifier;
+                getNextToken();
+                //              if (token == TT_ARGOPEN) {
+                //                getNextToken();
+                //                expressionList();
+                //                if (token != TT_ARGCLOSE) {
+                //                  throwSyntaxError(") expected after variable '" + ident + "'.");
+                //                }
+                //                getNextToken();
+                //              }
+                break;
+              case TT_IDENTIFIER :
+                ident = identifier;
                 getNextToken();
-                expressionList();
-                if (token != TT_ARGCLOSE) {
-                  throwSyntaxError(") expected after identifier '" + ident + "'.");
+                if (token == TT_ARGOPEN) {
+                  getNextToken();
+                  expressionList();
+                  if (token != TT_ARGCLOSE) {
+                    throwSyntaxError(") expected after identifier '" + ident + "'.");
+                  }
+                  getNextToken();
                 }
+                break;
+              case TT_LISTOPEN :
                 getNextToken();
-              }
-              break;
-            case TT_LISTOPEN :
-              getNextToken();
-              expression();
-              if (token != TT_LISTCLOSE) {
-                throwSyntaxError("} expected in postfix-expression.");
-              }
-              getNextToken();
-              break;
-            default :
-              throwSyntaxError("Syntax error after '->' token.");
+                expression();
+                if (token != TT_LISTCLOSE) {
+                  throwSyntaxError("} expected in postfix-expression.");
+                }
+                getNextToken();
+                break;
+              default :
+                throwSyntaxError("Syntax error after '->' token.");
+            }
           }
           break;
         case TT_INCREMENT :
@@ -1659,10 +2149,11 @@ public class PHPParser extends PHPKeywords {
         default :
           while_flag = false;
       }
+
     } while (while_flag);
   }
 
-  public void unaryExpression() {
+  private void unaryExpression() throws CoreException {
     switch (token) {
       case TT_INCREMENT :
         getNextToken();
@@ -1672,7 +2163,11 @@ public class PHPParser extends PHPKeywords {
         getNextToken();
         unaryExpression();
         break;
-        //'&' '*' '+' '-' '~' '!' 
+        // '@' '&' '*' '+' '-' '~' '!' 
+      case TT_AT :
+        getNextToken();
+        castExpression();
+        break;
       case TT_AMPERSAND :
         getNextToken();
         castExpression();
@@ -1702,7 +2197,7 @@ public class PHPParser extends PHPKeywords {
     }
   }
 
-  public void castExpression() {
+  private void castExpression() throws CoreException {
     //    if (token == TT_ARGOPEN) {
     //      getNextToken();
     //      typeName();
@@ -1714,7 +2209,7 @@ public class PHPParser extends PHPKeywords {
     unaryExpression();
   }
 
-  public void typeName() {
+  private void typeName() throws CoreException {
     //'string' 'unset' 'array' 'object'
     //'bool' 'boolean'
     //'real' 'double' 'float'
@@ -1733,7 +2228,7 @@ public class PHPParser extends PHPKeywords {
     throwSyntaxError("Expected type cast '( <type-name> )'; Got '" + ident + "'.");
   }
 
-  public void assignExpression() {
+  private void assignExpression() throws CoreException {
     castExpression();
     if (token == TT_ASSIGN) { // =
       getNextToken();
@@ -1756,10 +2251,28 @@ public class PHPParser extends PHPKeywords {
     } else if (token == TT_DIVIDEBY) { // *=
       getNextToken();
       logicalinclusiveorExpression();
+    } else if (token == TT_MODASSIGN) { // %=
+      getNextToken();
+      logicalinclusiveorExpression();
+    } else if (token == TT_ANDASSIGN) { // &=
+      getNextToken();
+      logicalinclusiveorExpression();
+    } else if (token == TT_POWASSIGN) { // ^=
+      getNextToken();
+      logicalinclusiveorExpression();
+    } else if (token == TT_LSHIFTASSIGN) { // <<=
+      getNextToken();
+      logicalinclusiveorExpression();
+    } else if (token == TT_RSHIFTASSIGN) { // >>=
+      getNextToken();
+      logicalinclusiveorExpression();
+    } else if (token == TT_TILDEASSIGN) { // ~=
+      getNextToken();
+      logicalinclusiveorExpression();
     }
   }
 
-  public void multiplicativeExpression() {
+  private void multiplicativeExpression() throws CoreException {
     do {
       assignExpression();
       if (token != TT_MULTIPLY && token != TT_DIV && token != TT_MOD) {
@@ -1769,7 +2282,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void concatenationExpression() {
+  private void concatenationExpression() throws CoreException {
     do {
       multiplicativeExpression();
       if (token != TT_DOT) {
@@ -1779,7 +2292,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void additiveExpression() {
+  private void additiveExpression() throws CoreException {
     do {
       concatenationExpression();
       if (token != TT_ADD && token != TT_SUBTRACT) {
@@ -1789,7 +2302,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void shiftExpression() {
+  private void shiftExpression() throws CoreException {
     do {
       additiveExpression();
       if (token != TT_LSHIFT && token != TT_RSHIFT) {
@@ -1799,7 +2312,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void relationalExpression() {
+  private void relationalExpression() throws CoreException {
     do {
       shiftExpression();
       if (token != TT_LESS && token != TT_GREATER && token != TT_LESSEQUAL && token != TT_GREATEREQUAL) {
@@ -1809,7 +2322,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void identicalExpression() {
+  private void identicalExpression() throws CoreException {
     do {
       relationalExpression();
       if (token != TT_EX_EQUAL && token != TT_EX_UNEQUAL) {
@@ -1819,7 +2332,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void equalityExpression() {
+  private void equalityExpression() throws CoreException {
     do {
       identicalExpression();
       if (token != TT_EQUAL && token != TT_UNEQUAL) {
@@ -1829,9 +2342,23 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void andExpression() {
+  private void ternaryExpression() throws CoreException {
+    equalityExpression();
+    if (token == TT_QUESTIONMARK) {
+      getNextToken();
+      expression();
+      if (token == TT_DDOT) {
+        getNextToken();
+        expression();
+      } else {
+        throwSyntaxError("':' expected in ternary operator '? :'.");
+      }
+    }
+  }
+
+  private void andExpression() throws CoreException {
     do {
-      equalityExpression();
+      ternaryExpression();
       if (token != TT_AMPERSAND) {
         return;
       }
@@ -1839,7 +2366,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void exclusiveorExpression() {
+  private void exclusiveorExpression() throws CoreException {
     do {
       andExpression();
       if (token != TT_POW) {
@@ -1849,7 +2376,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void inclusiveorExpression() {
+  private void inclusiveorExpression() throws CoreException {
     do {
       exclusiveorExpression();
       if (token != TT_LINE) {
@@ -1859,7 +2386,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void booleanandExpression() {
+  private void booleanandExpression() throws CoreException {
     do {
       inclusiveorExpression();
       if (token != TT_AND) {
@@ -1869,7 +2396,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void booleanorExpression() {
+  private void booleanorExpression() throws CoreException {
     do {
       booleanandExpression();
       if (token != TT_OR) {
@@ -1879,7 +2406,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void logicalandExpression() {
+  private void logicalandExpression() throws CoreException {
     do {
       booleanorExpression();
       if (token != TT_and) {
@@ -1889,7 +2416,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void logicalexclusiveorExpression() {
+  private void logicalexclusiveorExpression() throws CoreException {
     do {
       logicalandExpression();
       if (token != TT_xor) {
@@ -1899,7 +2426,7 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void logicalinclusiveorExpression() {
+  private void logicalinclusiveorExpression() throws CoreException {
     do {
       logicalexclusiveorExpression();
       if (token != TT_or) {
@@ -1921,7 +2448,7 @@ public class PHPParser extends PHPKeywords {
   //    }
   //  }
 
-  public void variableList() {
+  private void variableList() throws CoreException {
     do {
       variable();
       if (token == TT_COMMA) {
@@ -1932,16 +2459,64 @@ public class PHPParser extends PHPKeywords {
     } while (true);
   }
 
-  public void variable() {
-    if (token == TT_VARIABLE) {
+  private void variable() throws CoreException {
+    if (token == TT_DOLLAROPEN) {
+      getNextToken();
+      expression();
+      ;
+      if (token != TT_LISTCLOSE) {
+        throwSyntaxError("'}' expected after indirect variable token '${'.");
+      }
       getNextToken();
     } else {
-      throwSyntaxError("$-variable expected in variable-list.");
+      if (token == TT_VARIABLE) {
+        getNextToken();
+        if (token == TT_PARTOPEN) {
+          getNextToken();
+          expression();
+          if (token != TT_PARTCLOSE) {
+            throwSyntaxError("']' expected in variable-list.");
+          }
+          getNextToken();
+        } else if (token == TT_ASSIGN) {
+          getNextToken();
+          constant();
+        }
+      } else {
+        throwSyntaxError("$-variable expected in variable-list.");
+      }
     }
   }
 
-  public void constant() {
+  private void constant() throws CoreException {
+    String ident;
     switch (token) {
+      case TT_ADD :
+        getNextToken();
+        switch (token) {
+          case TT_DOUBLE_NUMBER :
+            getNextToken();
+            break;
+          case TT_INT_NUMBER :
+            getNextToken();
+            break;
+          default :
+            throwSyntaxError("Constant expected after '+' presign.");
+        }
+        break;
+      case TT_SUBTRACT :
+        getNextToken();
+        switch (token) {
+          case TT_DOUBLE_NUMBER :
+            getNextToken();
+            break;
+          case TT_INT_NUMBER :
+            getNextToken();
+            break;
+          default :
+            throwSyntaxError("Constant expected after '-' presign.");
+        }
+        break;
       case TT_null :
         getNextToken();
         break;
@@ -1951,6 +2526,20 @@ public class PHPParser extends PHPKeywords {
       case TT_true :
         getNextToken();
         break;
+      case TT_IDENTIFIER :
+        ident = identifier;
+        getNextToken();
+        if (token == TT_ARGOPEN) {
+          getNextToken();
+          if (token != TT_ARGCLOSE) {
+            expressionList();
+            if (token != TT_ARGCLOSE) {
+              throwSyntaxError("')' expected after identifier '" + ident + "' in postfix-expression.");
+            }
+          }
+          getNextToken();
+        }
+        break;
       case TT_STRING_CONSTANT :
         getNextToken();
         break;
@@ -1967,5 +2556,91 @@ public class PHPParser extends PHPKeywords {
         throwSyntaxError("Constant expected.");
     }
   }
+  
+  /**
+   * Call the php parse command ( php -l -f &lt;filename&gt; )
+   * and create markers according to the external parser output
+   */
+  public static void phpExternalParse(IFile file) {
+    //IFile file = (IFile) resource;
+    IPath path = file.getFullPath();
+    IPreferenceStore store = PHPeclipsePlugin.getDefault().getPreferenceStore();
+    String filename = file.getLocation().toString();
 
+    String[] arguments = { filename };
+    MessageFormat form = new MessageFormat(store.getString(PHPeclipsePlugin.EXTERNAL_PARSER_PREF));
+    String command = form.format(arguments);
+
+    String parserResult = PHPStartApacheAction.execute(command, "External parser: ");
+
+    try {
+      // parse the buffer to find the errors and warnings
+      createMarkers(parserResult, file);
+    } catch (CoreException e) {
+    }
+  }
+  
+  /**
+   * Create markers according to the external parser output
+   */
+  private static void createMarkers(String output, IFile file) throws CoreException {
+    // delete all markers
+    file.deleteMarkers(IMarker.PROBLEM, false, 0);
+
+    int indx = 0;
+    int brIndx = 0;
+    boolean flag = true;
+    while ((brIndx = output.indexOf("<br />", indx)) != -1) {
+      // newer php error output (tested with 4.2.3)
+      scanLine(output, file, indx, brIndx);
+      indx = brIndx + 6;
+      flag = false;
+    }
+    if (flag) {
+      while ((brIndx = output.indexOf("<br>", indx)) != -1) {
+        // older php error output (tested with 4.2.3)
+        scanLine(output, file, indx, brIndx);
+        indx = brIndx + 4;
+      }
+    }
+  }
+  
+  private static void scanLine(String output, IFile file, int indx, int brIndx) throws CoreException {
+    String current;
+    String outLineNumberString;
+    StringBuffer lineNumberBuffer = new StringBuffer(10);
+    char ch;
+    current = output.substring(indx, brIndx);
+
+    if (current.indexOf(PARSE_WARNING_STRING) != -1 || current.indexOf(PARSE_ERROR_STRING) != -1) {
+      int onLine = current.indexOf("on line <b>");
+      if (onLine != -1) {
+        lineNumberBuffer.delete(0, lineNumberBuffer.length());
+        for (int i = onLine; i < current.length(); i++) {
+          ch = current.charAt(i);
+          if ('0' <= ch && '9' >= ch) {
+            lineNumberBuffer.append(ch);
+          }
+        }
+
+        int lineNumber = Integer.parseInt(lineNumberBuffer.toString());
+
+        Hashtable attributes = new Hashtable();
+
+        current = current.replaceAll("\n", "");
+        current = current.replaceAll("<b>", "");
+        current = current.replaceAll("</b>", "");
+        MarkerUtilities.setMessage(attributes, current);
+
+        if (current.indexOf(PARSE_ERROR_STRING) != -1)
+          attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_ERROR));
+        else if (current.indexOf(PARSE_WARNING_STRING) != -1)
+          attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_WARNING));
+        else
+          attributes.put(IMarker.SEVERITY, new Integer(IMarker.SEVERITY_INFO));
+        MarkerUtilities.setLineNumber(attributes, lineNumber);
+        MarkerUtilities.createMarker(file, attributes, IMarker.PROBLEM);
+      }
+    }
+  }
 }
\ No newline at end of file