This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 632fdb9cb5 Align with 11.0.x onwards
632fdb9cb5 is described below
commit 632fdb9cb5a80f237784af04643f0bd54a8cf477
Author: Mark Thomas <[email protected]>
AuthorDate: Fri Feb 7 15:36:31 2025 +0000
Align with 11.0.x onwards
---
java/org/apache/jasper/compiler/JspUtil.java | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/java/org/apache/jasper/compiler/JspUtil.java
b/java/org/apache/jasper/compiler/JspUtil.java
index fc20a38ae6..5e00d198d0 100644
--- a/java/org/apache/jasper/compiler/JspUtil.java
+++ b/java/org/apache/jasper/compiler/JspUtil.java
@@ -352,17 +352,6 @@ public class JspUtil {
/*
* Build up the base call to the interpreter.
*/
- // XXX - We use a proprietary call to the interpreter for now
- // as the current standard machinery is inefficient and requires
- // lots of wrappers and adapters. This should all clear up once
- // the EL interpreter moves out of JSTL and into its own project.
- // In the future, this should be replaced by code that calls
- // ExpressionEvaluator.parseExpression() and then cache the resulting
- // expression objects. The interpreterCall would simply select
- // one of the pre-cached expressions and evaluate it.
- // Note that PageContextImpl implements VariableResolver and
- // the generated Servlet/SimpleTag implements FunctionMapper, so
- // that machinery is already in place (mroth).
targetType = toJavaSourceType(targetType);
StringBuilder call = new StringBuilder("(" + returnType + ") " +
"org.apache.jasper.runtime.PageContextImpl.proprietaryEvaluate" + "(" +
Generator.quote(expression) +
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
