You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2013/01/10 14:15:50 UTC
svn commit: r1431337 - in /tomcat/tc7.0.x/trunk: ./
test/org/apache/catalina/core/TestStandardContext.java
test/org/apache/catalina/startup/TomcatBaseTest.java
Author: kkolinko
Date: Thu Jan 10 13:15:50 2013
New Revision: 1431337
URL: http://svn.apache.org/viewvc?rev=1431337&view=rev
Log:
Merged revision 1431298 from tomcat/trunk:
Remove unnecessary warnings
(It was me who added those @SuppressWarnings several days ago. These are unneeded if "Potential resource leaks" warning is turned off in IDE).
Modified:
tomcat/tc7.0.x/trunk/ (props changed)
tomcat/tc7.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java
tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/TomcatBaseTest.java
Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
Merged /tomcat/trunk:r1431298
Modified: tomcat/tc7.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java?rev=1431337&r1=1431336&r2=1431337&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java (original)
+++ tomcat/tc7.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java Thu Jan 10 13:15:50 2013
@@ -161,7 +161,6 @@ public class TestStandardContext extends
@Override
public void doFilter(ServletRequest request, ServletResponse response,
FilterChain chain) throws IOException, ServletException {
- @SuppressWarnings("resource") // No need to close this writer
PrintWriter out = response.getWriter();
out.print(getClass().getName());
chain.doFilter(request, response);
@@ -648,7 +647,6 @@ public class TestStandardContext extends
resp.setContentType("text/plain");
resp.setCharacterEncoding("UTF-8");
- @SuppressWarnings("resource") // No need to close this writer
PrintWriter out = resp.getWriter();
out.println("parts=" + (null == req.getParts()
Modified: tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/TomcatBaseTest.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/TomcatBaseTest.java?rev=1431337&r1=1431336&r2=1431337&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/TomcatBaseTest.java (original)
+++ tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/TomcatBaseTest.java Thu Jan 10 13:15:50 2013
@@ -180,7 +180,6 @@ public abstract class TomcatBaseTest ext
@Override
protected void doGet(HttpServletRequest req, HttpServletResponse resp)
throws ServletException, IOException {
- @SuppressWarnings("resource") // No need to close this writer
PrintWriter out = resp.getWriter();
out.print(RESPONSE_TEXT);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org