Author: markt Date: Fri Feb 3 21:56:43 2017 New Revision: 1781610 URL: http://svn.apache.org/viewvc?rev=1781610&view=rev Log: Clean-up prior to back-port NPE warnings
Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java?rev=1781610&r1=1781609&r2=1781610&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/Generator.java Fri Feb 3 21:56:43 2017 @@ -1806,7 +1806,8 @@ class Generator { private static final String DOUBLE_QUOTE = "\\\""; - @Override + @SuppressWarnings("null") // NPEs identified by IDE are not possible + @Override public void visit(Node.UninterpretedTag n) throws JasperException { n.setBeginJavaLine(out.getJavaLine()); @@ -2830,7 +2831,8 @@ class Generator { return JspUtil.makeJavaIdentifier(varName); } - private String evaluateAttribute(TagHandlerInfo handlerInfo, + @SuppressWarnings("null") // NPE identified by IDE is not possible + private String evaluateAttribute(TagHandlerInfo handlerInfo, Node.JspAttribute attr, Node.CustomTag n, String tagHandlerVar) throws JasperException { @@ -2927,13 +2929,13 @@ class Generator { // should the expression be evaluated before passing to // the setter? boolean evaluate = false; - if (tai.canBeRequestTime()) { + if (tai != null && tai.canBeRequestTime()) { evaluate = true; // JSP.2.3.2 } if (attr.isDeferredInput()) { evaluate = false; // JSP.2.3.3 } - if (attr.isDeferredInput() && tai.canBeRequestTime()) { + if (attr.isDeferredInput() && tai != null && tai.canBeRequestTime()) { evaluate = !attrValue.contains("#{"); // JSP.2.3.5 } if (evaluate) { --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org