You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2006/09/19 11:47:33 UTC

svn commit: r447793 - /tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java

Author: remm
Date: Tue Sep 19 02:47:32 2006
New Revision: 447793

URL: http://svn.apache.org/viewvc?view=rev&rev=447793
Log:
- Pass the two new values to the PageInfo instance.

Modified:
    tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java

Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java?view=diff&rev=447793&r1=447792&r2=447793
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java Tue Sep 19 02:47:32 2006
@@ -129,6 +129,14 @@
         if (jspProperty.getIncludeCoda() != null) {
             pageInfo.setIncludeCoda(jspProperty.getIncludeCoda());
         }
+        if (jspProperty.isDefferedSyntaxAllowedAsLitteral() != null) {
+            pageInfo.setDeferredSyntaxAllowedAsLiteral(JspUtil.booleanValue(jspProperty
+                    .isDefferedSyntaxAllowedAsLitteral()));
+        }
+        if (jspProperty.isTrimDirectiveWhitespaces() != null) {
+            pageInfo.setTrimDirectiveWhitespaces(JspUtil.booleanValue(jspProperty
+                    .isTrimDirectiveWhitespaces()));
+        }
 
         ctxt.checkOutputDir();
         String javaFileName = ctxt.getServletJavaFileName();



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


Re: svn commit: r447793 - /tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java

Posted by Scott Johnson <sc...@us.ibm.com>.
And an extra t also :). 
Ought to follow the spec's name for the attribute, so:
setDeferredSyntaxAllowedAsLiteral
isDeferredSyntaxAllowedAsLiteral





Tim Funk <fu...@joedog.org> 
09/19/2006 06:33 AM
Please respond to
"Tomcat Developers List" <de...@tomcat.apache.org>


To
Tomcat Developers List <de...@tomcat.apache.org>
cc

Subject
Re: svn commit: r447793 - 
/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java






Is there a reason deferred is spelled 2 different ways? (One with an 
extra f)

setDeferredSyntaxAllowedAsLiteral vs
 isDefferedSyntaxAllowedAsLitteral


-Tim

remm@apache.org wrote:
> Author: remm
> Date: Tue Sep 19 02:47:32 2006
> New Revision: 447793
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=447793
> Log:
> - Pass the two new values to the PageInfo instance.
>
> Modified:
>     tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java
>
> Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java
> URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java?view=diff&rev=447793&r1=447792&r2=447793

> 
==============================================================================
> --- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java 
(original)
> +++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java 
Tue Sep 19 02:47:32 2006
> @@ -129,6 +129,14 @@
>          if (jspProperty.getIncludeCoda() != null) {
>              pageInfo.setIncludeCoda(jspProperty.getIncludeCoda());
>          }
> +        if (jspProperty.isDefferedSyntaxAllowedAsLitteral() != null) {
> + 
pageInfo.setDeferredSyntaxAllowedAsLiteral(JspUtil.booleanValue(jspProperty
> +                    .isDefferedSyntaxAllowedAsLitteral()));
> +        }
> +        if (jspProperty.isTrimDirectiveWhitespaces() != null) {
> + pageInfo.setTrimDirectiveWhitespaces(JspUtil.booleanValue(jspProperty
> +                    .isTrimDirectiveWhitespaces()));
> +        }
> 
 

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



Re: svn commit: r447793 - /tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java

Posted by Tim Funk <fu...@joedog.org>.
Is there a reason deferred is spelled 2 different ways? (One with an 
extra f)

setDeferredSyntaxAllowedAsLiteral vs
 isDefferedSyntaxAllowedAsLitteral


-Tim

remm@apache.org wrote:
> Author: remm
> Date: Tue Sep 19 02:47:32 2006
> New Revision: 447793
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=447793
> Log:
> - Pass the two new values to the PageInfo instance.
>
> Modified:
>     tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java
>
> Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java
> URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java?view=diff&rev=447793&r1=447792&r2=447793
> ==============================================================================
> --- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java (original)
> +++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Compiler.java Tue Sep 19 02:47:32 2006
> @@ -129,6 +129,14 @@
>          if (jspProperty.getIncludeCoda() != null) {
>              pageInfo.setIncludeCoda(jspProperty.getIncludeCoda());
>          }
> +        if (jspProperty.isDefferedSyntaxAllowedAsLitteral() != null) {
> +            pageInfo.setDeferredSyntaxAllowedAsLiteral(JspUtil.booleanValue(jspProperty
> +                    .isDefferedSyntaxAllowedAsLitteral()));
> +        }
> +        if (jspProperty.isTrimDirectiveWhitespaces() != null) {
> +            pageInfo.setTrimDirectiveWhitespaces(JspUtil.booleanValue(jspProperty
> +                    .isTrimDirectiveWhitespaces()));
> +        }
>  
 

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