Author: markt
Date: Thu Jan 19 15:13:16 2012
New Revision: 1233423

URL: http://svn.apache.org/viewvc?rev=1233423&view=rev
Log:
Take advantage of newly exposed Charset constants

Modified:
    tomcat/trunk/java/org/apache/catalina/util/Base64.java
    tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java
    
tomcat/trunk/test/org/apache/tomcat/util/http/TesterParametersPerformance.java

Modified: tomcat/trunk/java/org/apache/catalina/util/Base64.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/Base64.java?rev=1233423&r1=1233422&r2=1233423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/util/Base64.java (original)
+++ tomcat/trunk/java/org/apache/catalina/util/Base64.java Thu Jan 19 15:13:16 
2012
@@ -17,8 +17,6 @@
 
 package org.apache.catalina.util;
 
-import java.io.UnsupportedEncodingException;
-
 import org.apache.tomcat.util.buf.B2CConverter;
 import org.apache.tomcat.util.buf.ByteChunk;
 import org.apache.tomcat.util.buf.CharChunk;
@@ -172,15 +170,7 @@ public final class  Base64
             encodedData[encodedIndex + 3] = PAD;
         }
 
-        String result;
-        try {
-            result = new String(encodedData,
-                    B2CConverter.getCharset("ISO-8859-1"));
-        } catch (UnsupportedEncodingException e) {
-            // Should never happen but in case it does...
-            result = new String(encodedData);
-        }
-        return result;
+        return new String(encodedData, B2CConverter.ISO_8859_1);
     }
 
     /**

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=1233423&r1=1233422&r2=1233423&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/buf/ByteChunk.java Thu Jan 19 
15:13:16 2012
@@ -18,7 +18,6 @@ package org.apache.tomcat.util.buf;
 
 import java.io.IOException;
 import java.io.Serializable;
-import java.io.UnsupportedEncodingException;
 import java.nio.ByteBuffer;
 import java.nio.CharBuffer;
 import java.nio.charset.Charset;
@@ -100,17 +99,7 @@ public final class ByteChunk implements 
         as most standards seem to converge, but the servlet API requires
         8859_1, and this object is used mostly for servlets.
     */
-    public static final Charset DEFAULT_CHARSET;
-
-    static {
-        Charset c = null;
-        try {
-            c = B2CConverter.getCharset("ISO-8859-1");
-        } catch (UnsupportedEncodingException e) {
-            // Should never happen since all JVMs must support ISO-8859-1
-        }
-        DEFAULT_CHARSET = c;
-    }
+    public static final Charset DEFAULT_CHARSET = B2CConverter.ISO_8859_1;
 
     // byte[]
     private byte[] buff;

Modified: 
tomcat/trunk/test/org/apache/tomcat/util/http/TesterParametersPerformance.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/util/http/TesterParametersPerformance.java?rev=1233423&r1=1233422&r2=1233423&view=diff
==============================================================================
--- 
tomcat/trunk/test/org/apache/tomcat/util/http/TesterParametersPerformance.java 
(original)
+++ 
tomcat/trunk/test/org/apache/tomcat/util/http/TesterParametersPerformance.java 
Thu Jan 19 15:13:16 2012
@@ -109,13 +109,7 @@ public class TesterParametersPerformance
         System.arraycopy(inputBytes, 0, bytes, 0, inputLength);
 
         String[] result = new String[loops];
-        Charset charset = null;
-        try {
-            charset = B2CConverter.getCharset("ISO-8859-1");
-        } catch (UnsupportedEncodingException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
-        }
+        Charset charset = B2CConverter.ISO_8859_1;
 
         long start = System.nanoTime();
         for (int i = 0; i < loops; i++) {



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

Reply via email to