Merge pull request #60 from luneo7/JDK-8314794-backport

[Backport] svm: Adopt "JDK-8314794: Improve UTF8 String supports" [GR-58535]

Author: zakkak

substratevm/src/com.oracle.objectfile/src/com/oracle/objectfile/io/Utf8.java

@@ -36,8 +36,22 @@ public final class Utf8 {
     private Utf8() {
     }
 
+    private static int utf8Size(char c) {
+        // Based On
+        // https://github.com/openjdk/jdk21u/blob/jdk-21.0.4%2B7/src/hotspot/share/utilities/utf8.cpp#L409-L418
+        if ((0x0001 <= c) && (c <= 0x007F)) {
+            // ASCII character
+            return 1;
+        } else if (c <= 0x07FF) {
+            return 2;
+        } else {
+            return 3;
+        }
+    }
+
     /**
-     * @return the length in bytes of the UTF8 representation of the string
+     * @return the length as {@code int} in bytes of the UTF8 representation of the string. Might
+     *         return a truncated size if the value does not fit into {@code int} (see JDK-8328877).
      */
     public static int utf8Length(String string) {
         return utf8Length(string, 0, string.length());
@@ -46,24 +60,26 @@ public static int utf8Length(String string) {
     /**
      * @param beginIndex first index that is part of the region, inclusive
      * @param endIndex index at the end of the region, exclusive
-     * @return the length in bytes of the UTF8 representation of the string region
+     * @return the length as {@code int} in bytes of the UTF8 representation of the string region.
+     *         Might return a truncated size if the value does not fit into {@code int} (see
+     *         JDK-8328877).
      */
     public static int utf8Length(String s, int beginIndex, int endIndex) {
+        // Based on
+        // https://github.com/openjdk/jdk21u/blob/jdk-21.0.4%2B7/src/hotspot/share/utilities/utf8.cpp#L433-L444
         if (beginIndex < 0 || endIndex > s.length() || beginIndex > endIndex) {
             throw new StringIndexOutOfBoundsException();
         }
-        int length = 0;
-        for (int i = beginIndex; i < endIndex; i++) {
-            final int c = s.charAt(i);
-            if ((c >= 0x0001) && (c <= 0x007F)) {
-                length++;
-            } else if (c > 0x07FF) {
-                length += 3;
-            } else {
-                length += 2;
+        long result = 0;
+        for (int index = beginIndex; index < endIndex; index++) {
+            char c = s.charAt(index);
+            long sz = utf8Size(c);
+            if (result + sz > Integer.MAX_VALUE - 1) {
+                break;
             }
+            result += sz;
         }
-        return length;
+        return (int) result;
     }
 
     /**

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/util/Utf8.java

@@ -39,8 +39,21 @@ public final class Utf8 {
     private Utf8() {
     }
 
+    // @BasedOnJDKFile("https://github.com/openjdk/jdk21u/blob/jdk-21.0.4%2B7/src/hotspot/share/utilities/utf8.cpp#L409-L418")
+    private static int utf8Size(char c) {
+        if ((0x0001 <= c) && (c <= 0x007F)) {
+            // ASCII character
+            return 1;
+        } else if (c <= 0x07FF) {
+            return 2;
+        } else {
+            return 3;
+        }
+    }
+
     /**
-     * @return the length in bytes of the UTF8 representation of the string
+     * @return the length as {@code int} in bytes of the UTF8 representation of the string. Might
+     *         return a truncated size if the value does not fit into {@code int} (see JDK-8328877).
      */
     public static int utf8Length(String string) {
         return utf8Length(string, 0, string.length());
@@ -49,24 +62,24 @@ public static int utf8Length(String string) {
     /**
      * @param beginIndex first index that is part of the region, inclusive
      * @param endIndex index at the end of the region, exclusive
-     * @return the length in bytes of the UTF8 representation of the string region
+     * @return the length as {@code int} in bytes of the UTF8 representation of the string. Might
+     *         return a truncated size if the value does not fit into {@code int} (see JDK-8328877).
      */
+    // @BasedOnJDKFile("https://github.com/openjdk/jdk21u/blob/jdk-21.0.4%2B7/src/hotspot/share/utilities/utf8.cpp#L433-L444")
     public static int utf8Length(String s, int beginIndex, int endIndex) {
         if (beginIndex < 0 || endIndex > s.length() || beginIndex > endIndex) {
             throw new StringIndexOutOfBoundsException();
         }
-        int length = 0;
-        for (int i = beginIndex; i < endIndex; i++) {
-            final int c = s.charAt(i);
-            if ((c >= 0x0001) && (c <= 0x007F)) {
-                length++;
-            } else if (c > 0x07FF) {
-                length += 3;
-            } else {
-                length += 2;
+        long result = 0;
+        for (int index = beginIndex; index < endIndex; index++) {
+            char c = s.charAt(index);
+            long sz = utf8Size(c);
+            if (result + sz > Integer.MAX_VALUE - 1) {
+                break;
             }
+            result += sz;
         }
-        return length;
+        return (int) result;
     }
 
     /**