svn commit: r836177 - /tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java

[markt]

Author: markt
Date: Sat Nov 14 13:46:43 2009
New Revision: 836177

URL: http://svn.apache.org/viewvc?rev=836177&view=rev
Log:
Reduce visibility (thanks Sebb)
Remove unused code

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java?rev=836177&r1=836176&r2=836177&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/CookieSupport.java Sat Nov 14 
13:46:43 2009
@@ -65,16 +65,16 @@
      * spec, these are comma, semi-colon and white-space. The HTTP spec
      * definition of linear white space is [CRLF] 1*( SP | HT )
      */
-    public static final char[] V0_SEPARATORS = {',', ';', ' ', '\t'};
-    public static final boolean[] V0_SEPARATOR_FLAGS = new boolean[128];
+    private static final char[] V0_SEPARATORS = {',', ';', ' ', '\t'};
+    private static final boolean[] V0_SEPARATOR_FLAGS = new boolean[128];
     
     /**
      * The list of separators that apply to version 1 cookies. This may or may
      * not include '/' depending on the setting of
      * {...@link #FWD_SLASH_IS_SEPARATOR}.
      */
-    public static final char[] HTTP_SEPARATORS;
-    public static final boolean[] HTTP_SEPARATOR_FLAGS = new boolean[128];
+    private static final char[] HTTP_SEPARATORS;
+    private static final boolean[] HTTP_SEPARATOR_FLAGS = new boolean[128];
     
     static {
         STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
@@ -208,20 +208,6 @@
         return false;
     }
 
-    public static boolean containsCTL(String value) {
-        if (value==null) return false;
-        int len = value.length();
-        for (int i = 0; i < len; i++) {
-            char c = value.charAt(i);
-            if (c < 0x20 || c >= 0x7f) {
-                if (c == 0x09)
-                    continue; //allow horizontal tabs
-                return true;
-            }
-        }
-        return false;
-    }
-
     public static boolean alreadyQuoted (String value) {
         if (value==null || value.length() < 2) return false;
         return (value.charAt(0)=='\"' && value.charAt(value.length()-1)=='\"');



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