X-Git-Url: http://secure.phpeclipse.com

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/parser/Parser.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/parser/Parser.java
index c34ca1d..7035759 100644
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/parser/Parser.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/parser/Parser.java
@@ -422,6 +422,9 @@ public class Parser implements ITerminalSymbols, CompilerModifiers, ParserBasicI
 			try {
 				statement = statement();
 				blockStatements.add(statement);
+				if (token == TokenNameEOF) {
+					return null;
+				}
 				if (branchStatement && statement != null) {
 					// reportSyntaxError("Unreachable code", statement.sourceStart,
 					// statement.sourceEnd);
@@ -2502,37 +2505,33 @@ public class Parser implements ITerminalSymbols, CompilerModifiers, ParserBasicI
 			// | rw_variable T_INC
 			// | rw_variable T_DEC
 			case TokenNameIdentifier:
-				char[] ident = scanner.getCurrentTokenSource();
-				Expression lhsIdentifier = identifier(true, true);
-				if (lhsIdentifier != null) {
-					expression = lhsIdentifier;
-				}
-				if (token == TokenNameEQUAL || token == TokenNamePLUS_EQUAL
-						|| token == TokenNameMINUS_EQUAL || token == TokenNameMULTIPLY_EQUAL || token == TokenNameDIVIDE_EQUAL
-						|| token == TokenNameDOT_EQUAL || token == TokenNameREMAINDER_EQUAL || token == TokenNameAND_EQUAL
-						|| token == TokenNameOR_EQUAL || token == TokenNameXOR_EQUAL || token == TokenNameRIGHT_SHIFT_EQUAL
-						|| token == TokenNameLEFT_SHIFT_EQUAL) {
-					String error = "Assignment operator '"+scanner.toStringAction(token)+"' not allowed after identifier '"+new String(ident)+"' (use 'define(...)' to define constants).";
-					throwSyntaxError(error);
-				}
-				break;
 			case TokenNameVariable:
 			case TokenNameDOLLAR:
+				Expression lhs = null;
 				boolean rememberedVar = false;
-				Expression lhs = variable(true, true);
-				if (lhs != null && lhs instanceof FieldReference && token != TokenNameEQUAL && token != TokenNamePLUS_EQUAL
-						&& token != TokenNameMINUS_EQUAL && token != TokenNameMULTIPLY_EQUAL && token != TokenNameDIVIDE_EQUAL
-						&& token != TokenNameDOT_EQUAL && token != TokenNameREMAINDER_EQUAL && token != TokenNameAND_EQUAL
-						&& token != TokenNameOR_EQUAL && token != TokenNameXOR_EQUAL && token != TokenNameRIGHT_SHIFT_EQUAL
-						&& token != TokenNameLEFT_SHIFT_EQUAL) {
-					FieldReference ref = (FieldReference) lhs;
-					if (!containsVariableSet(ref.token)) {
-						problemReporter.uninitializedLocalVariable(new String(ref.token), ref.sourceStart(), ref.sourceEnd(), referenceContext,
-								compilationUnit.compilationResult);
-						addVariableSet(ref.token);
+				if (token == TokenNameIdentifier) {
+					lhs = identifier(true, true);
+					if (lhs != null) {
+						expression = lhs;
+					}
+				} else {
+					lhs = variable(true, true);
+					if (lhs != null) {
+						expression = lhs;
+					}
+					if (lhs != null && lhs instanceof FieldReference && token != TokenNameEQUAL && token != TokenNamePLUS_EQUAL
+							&& token != TokenNameMINUS_EQUAL && token != TokenNameMULTIPLY_EQUAL && token != TokenNameDIVIDE_EQUAL
+							&& token != TokenNameDOT_EQUAL && token != TokenNameREMAINDER_EQUAL && token != TokenNameAND_EQUAL
+							&& token != TokenNameOR_EQUAL && token != TokenNameXOR_EQUAL && token != TokenNameRIGHT_SHIFT_EQUAL
+							&& token != TokenNameLEFT_SHIFT_EQUAL) {
+						FieldReference ref = (FieldReference) lhs;
+						if (!containsVariableSet(ref.token)) {
+							problemReporter.uninitializedLocalVariable(new String(ref.token), ref.sourceStart(), ref.sourceEnd(),
+									referenceContext, compilationUnit.compilationResult);
+							addVariableSet(ref.token);
+						}
 					}
 				}
-
 				switch (token) {
 				case TokenNameEQUAL:
 					if (lhs != null && lhs instanceof FieldReference) {
@@ -3349,7 +3348,105 @@ public class Parser implements ITerminalSymbols, CompilerModifiers, ParserBasicI
 		// base_variable_with_function_calls T_OBJECT_OPERATOR
 		// object_property method_or_not variable_properties
 		// | base_variable_with_function_calls
-		Expression ref = base_variable_with_function_calls(lefthandside, ignoreVar);
+
+		// Expression ref = function_call(lefthandside, ignoreVar);
+
+		// function_call:
+		// T_STRING '(' function_call_parameter_list ')'
+		// | class_constant '(' function_call_parameter_list ')'
+		// | static_member '(' function_call_parameter_list ')'
+		// | variable_without_objects '(' function_call_parameter_list ')'
+		char[] defineName = null;
+		char[] ident = null;
+		int startPos = 0;
+		int endPos = 0;
+		Expression ref = null;
+		if (Scanner.TRACE) {
+			System.out.println("TRACE: function_call()");
+		}
+		if (token == TokenNameIdentifier) {
+			ident = scanner.getCurrentIdentifierSource();
+			defineName = ident;
+			startPos = scanner.getCurrentTokenStartPosition();
+			endPos = scanner.getCurrentTokenEndPosition();
+			getNextToken();
+
+			if (token == TokenNameEQUAL || token == TokenNamePLUS_EQUAL || token == TokenNameMINUS_EQUAL
+					|| token == TokenNameMULTIPLY_EQUAL || token == TokenNameDIVIDE_EQUAL || token == TokenNameDOT_EQUAL
+					|| token == TokenNameREMAINDER_EQUAL || token == TokenNameAND_EQUAL || token == TokenNameOR_EQUAL
+					|| token == TokenNameXOR_EQUAL || token == TokenNameRIGHT_SHIFT_EQUAL || token == TokenNameLEFT_SHIFT_EQUAL) {
+				String error = "Assignment operator '" + scanner.toStringAction(token) + "' not allowed after identifier '"
+						+ new String(ident) + "' (use 'define(...)' to define constants).";
+				reportSyntaxError(error);
+			}
+
+			switch (token) {
+			case TokenNamePAAMAYIM_NEKUDOTAYIM:
+				// static member:
+				defineName = null;
+				getNextToken();
+				if (token == TokenNameIdentifier) {
+					// class _constant
+					getNextToken();
+				} else {
+					// static member:
+					variable_without_objects(true, false);
+				}
+				break;
+			}
+		} else {
+			ref = variable_without_objects(lefthandside, ignoreVar);
+		}
+		if (token != TokenNameLPAREN) {
+			if (defineName != null) {
+				// does this identifier contain only uppercase characters?
+				if (defineName.length == 3) {
+					if (defineName[0] == 'd' && defineName[1] == 'i' && defineName[2] == 'e') {
+						defineName = null;
+					}
+				} else if (defineName.length == 4) {
+					if (defineName[0] == 't' && defineName[1] == 'r' && defineName[2] == 'u' && defineName[3] == 'e') {
+						defineName = null;
+					} else if (defineName[0] == 'n' && defineName[1] == 'u' && defineName[2] == 'l' && defineName[3] == 'l') {
+						defineName = null;
+					}
+				} else if (defineName.length == 5) {
+					if (defineName[0] == 'f' && defineName[1] == 'a' && defineName[2] == 'l' && defineName[3] == 's' && defineName[4] == 'e') {
+						defineName = null;
+					}
+				}
+				if (defineName != null) {
+					for (int i = 0; i < defineName.length; i++) {
+						if (Character.isLowerCase(defineName[i])) {
+							problemReporter.phpUppercaseIdentifierWarning(startPos, endPos, referenceContext, compilationUnit.compilationResult);
+							break;
+						}
+					}
+				}
+			}
+			// TODO is this ok ?
+			// return ref;
+			// throwSyntaxError("'(' expected in function call.");
+		} else {
+			getNextToken();
+
+			if (token == TokenNameRPAREN) {
+				getNextToken();
+				ref = null;
+			} else {
+				non_empty_function_call_parameter_list();
+				if (token != TokenNameRPAREN) {
+					String functionName;
+					if (ident == null) {
+						functionName = new String(" ");
+					} else {
+						functionName = new String(ident);
+					}
+					throwSyntaxError("')' expected in function call (" + functionName + ").");
+				}
+				getNextToken();
+			}
+		}
 		if (token == TokenNameMINUS_GREATER) {
 			ref = null;
 			getNextToken();