Author: markt
Date: Sun Mar 17 14:11:35 2013
New Revision: 1457453

URL: http://svn.apache.org/r1457453
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54712
Code clean-up. Use Locale.ENGLISH consistently for upper/lower case conversions.

Modified:
    tomcat/tc7.0.x/trunk/   (props changed)
    tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/buf/B2CConverter.java
    tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
    tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/MediaType.java
    
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
    tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java

Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
  Merged /tomcat/trunk:r1457452

Modified: tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/buf/B2CConverter.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/buf/B2CConverter.java?rev=1457453&r1=1457452&r2=1457453&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/buf/B2CConverter.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/buf/B2CConverter.java Sun 
Mar 17 14:11:35 2013
@@ -50,10 +50,10 @@ public class B2CConverter {
     static {
         for (Charset charset: Charset.availableCharsets().values()) {
             encodingToCharsetCache.put(
-                    charset.name().toLowerCase(Locale.US), charset);
+                    charset.name().toLowerCase(Locale.ENGLISH), charset);
             for (String alias : charset.aliases()) {
                 encodingToCharsetCache.put(
-                        alias.toLowerCase(Locale.US), charset);
+                        alias.toLowerCase(Locale.ENGLISH), charset);
             }
         }
         Charset iso88591 = null;
@@ -73,7 +73,7 @@ public class B2CConverter {
             throws UnsupportedEncodingException {
 
         // Encoding names should all be ASCII
-        String lowerCaseEnc = enc.toLowerCase(Locale.US);
+        String lowerCaseEnc = enc.toLowerCase(Locale.ENGLISH);
 
         return getCharsetLower(lowerCaseEnc);
     }

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java?rev=1457453&r1=1457452&r2=1457453&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
Sun Mar 17 14:11:35 2013
@@ -132,7 +132,7 @@ public class HttpParser {
                 return null;
             }
             String value = null;
-            Integer type = fieldTypes.get(field.toLowerCase(Locale.US));
+            Integer type = fieldTypes.get(field.toLowerCase(Locale.ENGLISH));
             if (type == null) {
                 // auth-param = token "=" ( token | quoted-string )
                 type = FIELD_TYPE_TOKEN_OR_QUOTED_STRING;
@@ -212,9 +212,9 @@ public class HttpParser {
 
             if (skipConstant(input, "=") == SkipConstantResult.FOUND) {
                 String value = readTokenOrQuotedString(input, true);
-                parameters.put(attribute.toLowerCase(Locale.US), value);
+                parameters.put(attribute.toLowerCase(Locale.ENGLISH), value);
             } else {
-                parameters.put(attribute.toLowerCase(Locale.US), "");
+                parameters.put(attribute.toLowerCase(Locale.ENGLISH), "");
             }
 
             lookForSemiColon = skipConstant(input, ";");

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/MediaType.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/MediaType.java?rev=1457453&r1=1457452&r2=1457453&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/MediaType.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/http/parser/MediaType.java 
Sun Mar 17 14:11:35 2013
@@ -60,7 +60,7 @@ public class MediaType {
     }
 
     public String getParameterValue(String parameter) {
-        return parameters.get(parameter.toLowerCase(Locale.US));
+        return parameters.get(parameter.toLowerCase(Locale.ENGLISH));
     }
 
     @Override

Modified: 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolProperties.java?rev=1457453&r1=1457452&r2=1457453&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 Sun Mar 17 14:11:35 2013
@@ -806,7 +806,7 @@ public class PoolProperties implements P
                 final String[] prefix = new String[] {"get","is"};
                 for (int j=0; j<prefix.length; j++) {
 
-                    String name = prefix[j] + fields[i].substring(0, 
1).toUpperCase(Locale.US) +
+                    String name = prefix[j] + fields[i].substring(0, 
1).toUpperCase(Locale.ENGLISH) +
                                   fields[i].substring(1);
                     Method m = null;
                     try {

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java?rev=1457453&r1=1457452&r2=1457453&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java 
(original)
+++ tomcat/tc7.0.x/trunk/test/org/apache/catalina/startup/SimpleHttpClient.java 
Sun Mar 17 14:11:35 2013
@@ -60,7 +60,7 @@ public abstract class SimpleHttpClient {
 
     protected static final String SESSION_COOKIE_NAME = "JSESSIONID";
     protected static final String SESSION_PARAMETER_NAME =
-            SESSION_COOKIE_NAME.toLowerCase(Locale.US);
+            SESSION_COOKIE_NAME.toLowerCase(Locale.ENGLISH);
 
     private static final String COOKIE_HEADER_PREFIX = "Set-Cookie: ";
     private static final String SESSION_COOKIE_HEADER_PREFIX =



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

Reply via email to