Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49550

Trivial code clean up

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@960698 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/java/org/apache/catalina/connector/Response.java b/java/org/apache/catalina/connector/Response.java
index 401c4d0..f414cd6 100644 (file)
--- a/java/org/apache/catalina/connector/Response.java
+++ b/java/org/apache/catalina/connector/Response.java
@@ -746,6 +746,7 @@ public class Response
      *
      * @param type The new content type
      */
+    @SuppressWarnings("deprecation") // isSpace (deprecated) cannot be replaced by isWhiteSpace
     public void setContentType(String type) {
 
         if (isCommitted())
@@ -773,6 +774,7 @@ public class Response
             if (index != -1) {
                 int len = type.length();
                 index++;
+                // N.B. isSpace (deprecated) cannot be replaced by isWhiteSpace
                 while (index < len && Character.isSpace(type.charAt(index))) {
                     index++;
                 }

diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 0d8a486..97cc5cf 100644 (file)
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -103,6 +103,11 @@
         Additional debug logging in StandardContext to provide information on
         Manager selection. (markt)
       </add>
+      <fix>
+        <bug>49550</bug>: Supress deprecation warning where deprecated code is
+        required to be used. No functional change. Patch provided by Sebb.
+        (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Jasper">