You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by mi...@apache.org on 2019/02/16 22:41:32 UTC

svn commit: r1853730 - in /velocity/engine/trunk/velocity-engine-core/src: main/java/org/apache/velocity/io/ main/java/org/apache/velocity/runtime/ test/java/org/apache/velocity/test/eventhandler/ test/java/org/apache/velocity/test/misc/

Author: michaelo
Date: Sat Feb 16 22:41:31 2019
New Revision: 1853730

URL: http://svn.apache.org/viewvc?rev=1853730&view=rev
Log:
[VELOCITY-908] Initialization failure on Windows with Turkish locale

Modified:
    velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/io/UnicodeInputStream.java
    velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/RuntimeInstance.java
    velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/eventhandler/Handler2.java
    velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/misc/TestContext.java

Modified: velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/io/UnicodeInputStream.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/io/UnicodeInputStream.java?rev=1853730&r1=1853729&r2=1853730&view=diff
==============================================================================
--- velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/io/UnicodeInputStream.java (original)
+++ velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/io/UnicodeInputStream.java Sat Feb 16 22:41:31 2019
@@ -24,6 +24,7 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.io.PushbackInputStream;
 
+import java.util.Locale;
 
 /**
  * This is an input stream that is unicode BOM aware. This allows you to e.g. read
@@ -344,8 +345,8 @@ public class UnicodeInputStream
      */
     public static boolean sameEncoding(String left, String right)
     {
-        left = left.toUpperCase().replace("-", "").replace("_","");
-        right = right.toUpperCase().replace("-", "").replace("_","");
+        left = left.toUpperCase(Locale.ROOT).replace("-", "").replace("_","");
+        right = right.toUpperCase(Locale.ROOT).replace("-", "").replace("_","");
         return left.equals(right);
     }
 

Modified: velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/RuntimeInstance.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/RuntimeInstance.java?rev=1853730&r1=1853729&r2=1853730&view=diff
==============================================================================
--- velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/RuntimeInstance.java (original)
+++ velocity/engine/trunk/velocity-engine-core/src/main/java/org/apache/velocity/runtime/RuntimeInstance.java Sat Feb 16 22:41:31 2019
@@ -64,6 +64,7 @@ import java.util.Enumeration;
 import java.util.HashMap;
 import java.util.Hashtable;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import java.util.NoSuchElementException;
 import java.util.Properties;
@@ -351,7 +352,7 @@ public class RuntimeInstance implements
         String im = getString(SPACE_GOBBLING, "lines");
         try
         {
-            spaceGobbling = SpaceGobbling.valueOf(im.toUpperCase());
+            spaceGobbling = SpaceGobbling.valueOf(im.toUpperCase(Locale.ROOT));
         }
         catch (NoSuchElementException nse)
         {

Modified: velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/eventhandler/Handler2.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/eventhandler/Handler2.java?rev=1853730&r1=1853729&r2=1853730&view=diff
==============================================================================
--- velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/eventhandler/Handler2.java (original)
+++ velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/eventhandler/Handler2.java Sat Feb 16 22:41:31 2019
@@ -25,6 +25,8 @@ import org.apache.velocity.app.event.Ref
 import org.apache.velocity.context.Context;
 import org.apache.velocity.util.introspection.Info;
 
+import java.util.Locale;
+
 /**
  * This is a test set of event handlers, used to test event handler sequences.
  *
@@ -42,7 +44,7 @@ public class Handler2
         if (value == null)
             return null;
         else
-            return value.toString().toUpperCase();
+            return value.toString().toUpperCase(Locale.ROOT);
     }
 
     /**

Modified: velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/misc/TestContext.java
URL: http://svn.apache.org/viewvc/velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/misc/TestContext.java?rev=1853730&r1=1853729&r2=1853730&view=diff
==============================================================================
--- velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/misc/TestContext.java (original)
+++ velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/misc/TestContext.java Sat Feb 16 22:41:31 2019
@@ -23,6 +23,7 @@ import org.apache.velocity.VelocityConte
 import org.apache.velocity.context.Context;
 
 import java.util.HashMap;
+import java.util.Locale;
 import java.util.Map;
 
 /**
@@ -73,7 +74,7 @@ public class TestContext implements Cont
         }
         else
         {
-            return key.toUpperCase();
+            return key.toUpperCase(Locale.ROOT);
         }
     }
 }