This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new e2624ac  Remove unnecessary code
e2624ac is described below

commit e2624ac20bee1f1549109922ad7bc8d4ea80c6ed
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue May 11 21:49:07 2021 +0100

    Remove unnecessary code
    
    This method is called for a SimpleTag.
    pushBodyCountVar is always null for SimpleTag as the code where it is
    set in visit(CustomTag) is never called if n.implementsSimpleTag() is
    true.
---
 java/org/apache/jasper/compiler/Generator.java | 4 ----
 webapps/docs/changelog.xml                     | 8 ++++++++
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/java/org/apache/jasper/compiler/Generator.java 
b/java/org/apache/jasper/compiler/Generator.java
index b6e0cd5..ba36ffe 100644
--- a/java/org/apache/jasper/compiler/Generator.java
+++ b/java/org/apache/jasper/compiler/Generator.java
@@ -3313,10 +3313,6 @@ class Generator {
             boolean tmpIsFragment = isFragment;
             isFragment = true;
             String pushBodyCountVarSave = pushBodyCountVar;
-            if (pushBodyCountVar != null) {
-                // Use a fixed name for push body count, to simplify code gen
-                pushBodyCountVar = "_jspx_push_body_count";
-            }
             visitBody(n);
             out = outSave;
             parent = tmpParent;
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 3a6a78c..dc1bd17 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -104,6 +104,14 @@
   issues do not "pop up" wrt. others).
 -->
 <section name="Tomcat 9.0.47 (markt)" rtext="in development">
+  <subsection name="Jasper">
+    <changelog>
+      <scode>
+        Review code used to generate Java source from JSPs and tags and remove
+        code found to be unnecessary. (markt)
+      </scode>
+    </changelog>
+  </subsection>
 </section>
 <section name="Tomcat 9.0.46 (markt)" rtext="release in progress">
   <subsection name="Catalina">

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to