You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by jb...@apache.org on 2013/11/10 20:16:49 UTC

svn commit: r1540522 - /tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java

Author: jboynes
Date: Sun Nov 10 19:16:48 2013
New Revision: 1540522

URL: http://svn.apache.org/r1540522
Log:
fix IDE warning about uneeded boxing

Modified:
    tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java?rev=1540522&r1=1540521&r2=1540522&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java (original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java Sun Nov 10 19:16:48 2013
@@ -49,7 +49,7 @@ public class JasperInitializer implement
             LOG.debug(Localizer.getMessage(MSG + ".onStartup", context.getServletContextName()));
         }
 
-        boolean validate = Boolean.valueOf(context.getInitParameter(VALIDATE)).booleanValue();
+        boolean validate = Boolean.valueOf(context.getInitParameter(VALIDATE));
 
         // scan the application for TLDs
         TldScanner scanner = new TldScanner(context, true, validate);



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


Re: svn commit: r1540522 - /tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java

Posted by Mark Thomas <ma...@apache.org>.
On 10/11/2013 19:16, jboynes@apache.org wrote:
> Author: jboynes
> Date: Sun Nov 10 19:16:48 2013
> New Revision: 1540522
> 
> URL: http://svn.apache.org/r1540522
> Log:
> fix IDE warning about uneeded boxing

Huh?

Please revert this commit. It creates an IDE warning as Boolean now has
to be unboxed to boolean.

Mark


> 
> Modified:
>     tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java
> 
> Modified: tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java
> URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java?rev=1540522&r1=1540521&r2=1540522&view=diff
> ==============================================================================
> --- tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java (original)
> +++ tomcat/trunk/java/org/apache/jasper/servlet/JasperInitializer.java Sun Nov 10 19:16:48 2013
> @@ -49,7 +49,7 @@ public class JasperInitializer implement
>              LOG.debug(Localizer.getMessage(MSG + ".onStartup", context.getServletContextName()));
>          }
>  
> -        boolean validate = Boolean.valueOf(context.getInitParameter(VALIDATE)).booleanValue();
> +        boolean validate = Boolean.valueOf(context.getInitParameter(VALIDATE));
>  
>          // scan the application for TLDs
>          TldScanner scanner = new TldScanner(context, true, validate);
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
> 


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