2006-01-08 Chris Burdess <dog@gnu.org>

	* java/lang/Character.java (toChars,toCodePoint): Correct these
	  methods to use algorithms from Unicode specification.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@109516 138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 3770c67..e8060ab 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2006-01-08  Chris Burdess  <dog@gnu.org>
+
+       * java/lang/Character.java (toChars,toCodePoint): Correct these
+         methods to use algorithms from Unicode specification.
+
 2006-01-08  Tom Tromey  <tromey@redhat.com>
 
        * java/lang/StringBuilder.java (appendCodePoint): New method.

diff --git a/libjava/java/lang/Character.java b/libjava/java/lang/Character.java
index 3cb73d0..f56117f 100644 (file)
--- a/libjava/java/lang/Character.java
+++ b/libjava/java/lang/Character.java
@@ -2320,11 +2320,11 @@ public final class Character implements Serializable, Comparable
       {
         // Write second char first to cause IndexOutOfBoundsException
         // immediately.
-        dst[dstIndex + 1] = (char) ((codePoint & 0x3ff)
-                                    + (int) MIN_LOW_SURROGATE );
-        dst[dstIndex] = (char) ((codePoint >> 10) + (int) MIN_HIGH_SURROGATE);
+        final int cp2 = codePoint - 0x10000;
+        dst[dstIndex + 1] = (char) ((cp2 % 0x400) + (int) MIN_LOW_SURROGATE);
+        dst[dstIndex] = (char) ((cp2 / 0x400) + (int) MIN_HIGH_SURROGATE);
         result = 2;
-    }
+      }
     else
       {
         dst[dstIndex] = (char) codePoint;
@@ -2433,7 +2433,8 @@ public final class Character implements Serializable, Comparable
    */
   public static int toCodePoint(char high, char low)
   {
-    return ((high - MIN_HIGH_SURROGATE) << 10) + (low - MIN_LOW_SURROGATE);
+    return ((high - MIN_HIGH_SURROGATE) * 0x400) +
+      (low - MIN_LOW_SURROGATE) + 0x10000;
   }
 
   /**