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 826528aeb7 Remove comments related to the ability to inline these methods. 826528aeb7 is described below commit 826528aeb786c59e7c90ca1b928c5ba8a44fbd72 Author: Mark Thomas <ma...@apache.org> AuthorDate: Thu Mar 6 16:09:32 2025 +0000 Remove comments related to the ability to inline these methods. This comment that the functionality had been forgotten dates from Tomcat 4.1.x or earlier. I can find no remains (part from these comments) of the feature to inline these methods. Therefore, remove the comments. --- .../org/apache/jasper/runtime/JspRuntimeLibrary.java | 20 +++++--------------- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/java/org/apache/jasper/runtime/JspRuntimeLibrary.java b/java/org/apache/jasper/runtime/JspRuntimeLibrary.java index fdfc39ab13..19a7599def 100644 --- a/java/org/apache/jasper/runtime/JspRuntimeLibrary.java +++ b/java/org/apache/jasper/runtime/JspRuntimeLibrary.java @@ -47,11 +47,6 @@ import org.apache.tomcat.InstanceManager; * Bunch of util methods that are used by code generated for useBean, * getProperty and setProperty. * - * The __begin, __end stuff is there so that the JSP engine can - * actually parse this file and inline them if people don't want - * runtime dependencies on this class. However, I'm not sure if that - * works so well right now. It got forgotten at some point. -akv - * * @author Mandar Raje * @author Shawn Bayern */ @@ -226,7 +221,7 @@ public class JspRuntimeLibrary { } } - // __begin convertMethod + public static Object convert(String propertyName, String s, Class<?> t, Class<?> propertyEditorClass) throws JasperException @@ -298,9 +293,8 @@ public class JspRuntimeLibrary { throw new JasperException(ex); } } - // __end convertMethod - // __begin introspectMethod + public static void introspect(Object bean, ServletRequest request) throws JasperException { @@ -311,9 +305,8 @@ public class JspRuntimeLibrary { introspecthelper(bean, name, value, request, name, true); } } - // __end introspectMethod - // __begin introspecthelperMethod + public static void introspecthelper(Object bean, String prop, String value, ServletRequest request, String param, boolean ignoreMethodNF) @@ -391,12 +384,11 @@ public class JspRuntimeLibrary { } } } - // __end introspecthelperMethod + //------------------------------------------------------------------- // functions to convert builtin Java data types to string. //------------------------------------------------------------------- - // __begin toStringMethod public static String toString(Object o) { return String.valueOf(o); } @@ -432,7 +424,6 @@ public class JspRuntimeLibrary { public static String toString(char c) { return Character.toString(c); } - // __end toStringMethod /** @@ -600,7 +591,7 @@ public class JspRuntimeLibrary { return escStringBuilder.toString(); } - // __begin lookupReadMethodMethod + public static Object handleGetProperty(Object o, String prop) throws JasperException { if (o == null) { @@ -618,7 +609,6 @@ public class JspRuntimeLibrary { } return value; } - // __end lookupReadMethodMethod // handles <jsp:setProperty> with EL expression for 'value' attribute public static void handleSetPropertyExpression(Object bean, --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org