Author: markt
Date: Sat Nov  7 22:38:55 2015
New Revision: 1713194

URL: http://svn.apache.org/viewvc?rev=1713194&view=rev
Log:
Remove unused constant.
Reported by UC Detector

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferHolder.java
    tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferUtils.java
    tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java
    tomcat/trunk/java/org/apache/tomcat/util/buf/UEncoder.java
    tomcat/trunk/java/org/apache/tomcat/util/threads/Constants.java
    
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
    
tomcat/trunk/test/org/apache/catalina/authenticator/TestJaspicDigestAuthenticator.java
    tomcat/trunk/test/org/apache/tomcat/util/buf/TestUEncoder.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferHolder.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferHolder.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferHolder.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferHolder.java Sat Nov  
7 22:38:55 2015
@@ -52,13 +52,4 @@ public class ByteBufferHolder {
             return false;
         }
     }
-
-
-    public boolean hasData() {
-        if (flipped.get()) {
-            return buf.remaining()>0;
-        } else {
-            return buf.position()>0;
-        }
-    }
 }
\ No newline at end of file

Modified: tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferUtils.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferUtils.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferUtils.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/buf/ByteBufferUtils.java Sat Nov  
7 22:38:55 2015
@@ -45,20 +45,6 @@ public class ByteBufferUtils {
 
 
     /**
-     * Default byte buffer expansion. Doubles current size. Buffers are assumed
-     * to be in 'write to' mode since there would be no need to expand a buffer
-     * while it was in 'read from' mode.
-     *
-     * @param in Buffer to expand
-     * @return   The expanded buffer with any data from the input buffer copied
-     *           in to it
-     */
-    public static ByteBuffer expand(ByteBuffer in) {
-        return expand(in, in.capacity() * 2);
-    }
-
-
-    /**
      * Expands buffer to the given size unless it is already as big or bigger.
      * Buffers are assumed to be in 'write to' mode since there would be no 
need
      * to expand a buffer while it was in 'read from' mode.

Modified: tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java Sat Nov  7 
22:38:55 2015
@@ -142,7 +142,6 @@ public final class ByteChunk implements
      * Resets the message buff to an uninitialized state.
      */
     public void recycle() {
-        //        buff = null;
         charset=null;
         start=0;
         end=0;
@@ -150,10 +149,6 @@ public final class ByteChunk implements
         hasHashCode = false;
     }
 
-    public void reset() {
-        buff=null;
-    }
-
     // -------------------- Setup --------------------
 
     public void allocate( int initial, int limit  ) {

Modified: tomcat/trunk/java/org/apache/tomcat/util/buf/UEncoder.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/buf/UEncoder.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/buf/UEncoder.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/buf/UEncoder.java Sat Nov  7 
22:38:55 2015
@@ -56,36 +56,16 @@ public final class UEncoder {
     private ByteChunk bb=null;
     private CharChunk cb=null;
     private CharChunk output=null;
-    private final boolean readOnlySafeChars;
-
-    public UEncoder() {
-        this.safeChars = initialSafeChars();
-        readOnlySafeChars = false;
-    }
 
     /**
      * Create a UEncoder with an unmodifiable safe character set.
-     * <p>
-     * Calls to {@link UEncoder#addSafeCharacter(char) addSafeCharacter(char)}
-     * on instances created by this constructor will throw an
-     * {@link IllegalStateException}.
      *
-     * @param safeCharsSet
-     *            safe characters for this encoder
+     * @param safeCharsSet safe characters for this encoder
      */
     public UEncoder(SafeCharsSet safeCharsSet) {
         this.safeChars = safeCharsSet.getSafeChars();
-        readOnlySafeChars = true;
     }
 
-    public void addSafeCharacter( char c ) {
-        if (readOnlySafeChars) {
-            throw new IllegalStateException("UEncoders safeChararacters are 
read only");
-        }
-        safeChars.set( c );
-    }
-
-
    /**
     * URL Encode string, using a specified encoding.
     *

Modified: tomcat/trunk/java/org/apache/tomcat/util/threads/Constants.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/threads/Constants.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/threads/Constants.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/threads/Constants.java Sat Nov  7 
22:38:55 2015
@@ -21,8 +21,6 @@ package org.apache.tomcat.util.threads;
  */
 public final class Constants {
 
-    public static final String Package = "org.apache.tomcat.util.threads";
-
     public static final long DEFAULT_THREAD_RENEWAL_DELAY = 1000L;
 
     /**

Modified: 
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- 
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
 (original)
+++ 
tomcat/trunk/test/org/apache/catalina/authenticator/TestDigestAuthenticator.java
 Sat Nov  7 22:38:55 2015
@@ -245,7 +245,6 @@ public class TestDigestAuthenticator ext
         // Third request should succeed if we increment nc
         auth.clear();
         bc.recycle();
-        bc.reset();
         auth.add(buildDigestResponse(user, pwd, digestUri, realm,
                 getNonce(respHeaders), getOpaque(respHeaders), nc2, cnonce,
                 qop));

Modified: 
tomcat/trunk/test/org/apache/catalina/authenticator/TestJaspicDigestAuthenticator.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/authenticator/TestJaspicDigestAuthenticator.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- 
tomcat/trunk/test/org/apache/catalina/authenticator/TestJaspicDigestAuthenticator.java
 (original)
+++ 
tomcat/trunk/test/org/apache/catalina/authenticator/TestJaspicDigestAuthenticator.java
 Sat Nov  7 22:38:55 2015
@@ -251,7 +251,6 @@ public class TestJaspicDigestAuthenticat
         // Third request should succeed if we increment nc
         auth.clear();
         bc.recycle();
-        bc.reset();
         auth.add(buildDigestResponse(user, pwd, digestUri, realm,
                 getNonce(respHeaders), getOpaque(respHeaders), nc2, cnonce,
                 qop));

Modified: tomcat/trunk/test/org/apache/tomcat/util/buf/TestUEncoder.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/util/buf/TestUEncoder.java?rev=1713194&r1=1713193&r2=1713194&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/tomcat/util/buf/TestUEncoder.java (original)
+++ tomcat/trunk/test/org/apache/tomcat/util/buf/TestUEncoder.java Sat Nov  7 
22:38:55 2015
@@ -20,8 +20,6 @@ package org.apache.tomcat.util.buf;
 import java.io.IOException;
 
 import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
-
 import org.junit.Test;
 
 import org.apache.tomcat.util.buf.UEncoder.SafeCharsSet;
@@ -32,25 +30,6 @@ import org.apache.tomcat.util.buf.UEncod
 public class TestUEncoder {
 
     @Test
-    public void testEncodeURL() throws IOException {
-        UEncoder urlEncoder = new UEncoder();
-
-        String s = "a/b/c/d+e.class";
-        assertTrue(urlEncoder.encodeURL(s, 0, s.length()).equals(
-                "a%2fb%2fc%2fd%2be.class"));
-        assertTrue(urlEncoder.encodeURL(s, 2, s.length() - 2).equals(
-                "b%2fc%2fd%2be.cla"));
-
-        urlEncoder.addSafeCharacter('+');
-        assertTrue(urlEncoder.encodeURL(s, 0, s.length()).equals(
-                "a%2fb%2fc%2fd+e.class"));
-
-        s = new String(new char[] { 0xD801, 0xDC01 });
-        assertTrue(urlEncoder.encodeURL(s, 0, s.length())
-                .equals("%f0%90%90%81"));
-    }
-
-    @Test
     public void testEncodeURLWithSlashInit() throws IOException {
         UEncoder urlEncoder = new UEncoder(SafeCharsSet.WITH_SLASH);
 
@@ -60,16 +39,8 @@ public class TestUEncoder {
         assertTrue(urlEncoder.encodeURL(s, 2, s.length() - 2).equals(
                 "b/c/d%2be.cla"));
 
-        try {
-            urlEncoder.addSafeCharacter('+');
-            fail();
-        } catch (IllegalStateException e) {
-            // OK
-        }
-
         s = new String(new char[] { 0xD801, 0xDC01 });
         assertTrue(urlEncoder.encodeURL(s, 0, s.length())
                 .equals("%f0%90%90%81"));
     }
-
 }



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

Reply via email to