You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2012/08/31 22:33:45 UTC
svn commit: r1379590 - /tomcat/trunk/java/org/apache/el/parser/AstValue.java
Author: markt
Date: Fri Aug 31 20:33:44 2012
New Revision: 1379590
URL: http://svn.apache.org/viewvc?rev=1379590&view=rev
Log:
Remove unnecessary method calls. The Resolvers manage this flag.
Modified:
tomcat/trunk/java/org/apache/el/parser/AstValue.java
Modified: tomcat/trunk/java/org/apache/el/parser/AstValue.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/el/parser/AstValue.java?rev=1379590&r1=1379589&r2=1379590&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/el/parser/AstValue.java (original)
+++ tomcat/trunk/java/org/apache/el/parser/AstValue.java Fri Aug 31 20:33:44 2012
@@ -80,7 +80,6 @@ public final class AstValue extends Simp
@Override
public Class<?> getType(EvaluationContext ctx) throws ELException {
Target t = getTarget(ctx);
- ctx.setPropertyResolved(false);
Class<?> result = ctx.getELResolver().getType(ctx, t.base, t.property);
if (!ctx.isPropertyResolved()) {
throw new PropertyNotFoundException(MessageFactory.get(
@@ -118,7 +117,6 @@ public final class AstValue extends Simp
} else if (i + 2 == propCount &&
this.children[i + 1] instanceof AstMethodParameters) {
// Method call at end of expression
- ctx.setPropertyResolved(false);
property = this.children[i].getValue(ctx);
i += 2;
@@ -134,7 +132,6 @@ public final class AstValue extends Simp
} else {
// Object with property at end of expression
- ctx.setPropertyResolved(false);
property = this.children[i].getValue(ctx);
i++;
@@ -178,7 +175,6 @@ public final class AstValue extends Simp
return null;
}
- ctx.setPropertyResolved(false);
base = resolver.getValue(ctx, base, suffix);
i++;
}
@@ -193,7 +189,6 @@ public final class AstValue extends Simp
@Override
public boolean isReadOnly(EvaluationContext ctx) throws ELException {
Target t = getTarget(ctx);
- ctx.setPropertyResolved(false);
boolean result =
ctx.getELResolver().isReadOnly(ctx, t.base, t.property);
if (!ctx.isPropertyResolved()) {
@@ -207,7 +202,6 @@ public final class AstValue extends Simp
public void setValue(EvaluationContext ctx, Object value)
throws ELException {
Target t = getTarget(ctx);
- ctx.setPropertyResolved(false);
ELResolver resolver = ctx.getELResolver();
// coerce to the expected type
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org