svn commit: r1401814 - /tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java

[markt]

Author: markt
Date: Wed Oct 24 18:32:56 2012
New Revision: 1401814

URL: http://svn.apache.org/viewvc?rev=1401814&view=rev
Log:
Follow up to https://issues.apache.org/bugzilla/show_bug.cgi?id=54011
Simplify code.
Based on a patch by Sheldon Shao

Modified:
    tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java

Modified: tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java?rev=1401814&r1=1401813&r2=1401814&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java (original)
+++ tomcat/trunk/java/org/apache/jasper/tagplugins/jstl/core/Out.java Wed Oct 
24 18:32:56 2012
@@ -18,6 +18,10 @@
 
 package org.apache.jasper.tagplugins.jstl.core;
 
+import java.io.IOException;
+
+import javax.servlet.jsp.JspWriter;
+
 import org.apache.jasper.compiler.tagplugin.TagPlugin;
 import org.apache.jasper.compiler.tagplugin.TagPluginContext;
 
@@ -72,20 +76,20 @@ public final class Out implements TagPlu
         }
 
         //main part.
-        ctxt.generateJavaSource("if(null != " + strValName +"){");
-        ctxt.generateJavaSource("    if(" + strEscapeXmlName + "){");
-        ctxt.generateJavaSource("        " + strValName + " = 
org.apache.jasper.tagplugins.jstl.Util.escapeXml(" + strValName + ");");
-        ctxt.generateJavaSource("    }");
-        ctxt.generateJavaSource("    out.write(" + strValName + ");");
-        ctxt.generateJavaSource("}else{");
-        ctxt.generateJavaSource("    if(null != " + strDefName + "){");
-        ctxt.generateJavaSource("        if(" + strEscapeXmlName + "){");
-        ctxt.generateJavaSource("            " + strDefName + " = 
org.apache.jasper.tagplugins.jstl.Util.escapeXml(" + strDefName + ");");
-        ctxt.generateJavaSource("        }");
-        ctxt.generateJavaSource("        out.write(" + strDefName + ");");
-        ctxt.generateJavaSource("    }else{");
-        ctxt.generateBody();
-        ctxt.generateJavaSource("    }");
-        ctxt.generateJavaSource("}");
+        ctxt.generateJavaSource(
+                "org.apache.jasper.tagplugins.jstl.core.Out.output(out, " +
+                strValName + ", " + strDefName + ", " + strEscapeXmlName +
+                ");");
+    }
+
+    public static void output(JspWriter out, String value, String defaultValue,
+            boolean escapeXml) throws IOException {
+        String v = value != null ? value : defaultValue;
+        if (v != null) {
+            if(escapeXml){
+                v = org.apache.jasper.tagplugins.jstl.Util.escapeXml(v);
+            }
+            out.write(v);
+        }
     }
 }



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