JDK-8197594: String#repeat

Bernd Eckenfels ecki at zusammenkunft.net
Thu Mar 1 15:04:30 UTC 2018


I would not encourage makeshift number formatting by mentioning left padding as a major usecase:

Just remove that part:

+     * <p>
+     * This method may be used to create space padding for
+     * formatting text or zero padding for formatting numbers.

I think this error without an Explanation can be hard to debug. It should be clear that Memory allocation was not attempted and what the actual Parameters where:

+        if (Integer.MAX_VALUE / count < len) {
+            throw new OutOfMemoryError();
+        }

„Repeating „ + len + „ bytes „ + count + „ times would produce a String exceeding maximum size.“


Von: Jim Laskey
Gesendet: Mittwoch, 28. Februar 2018 21:01
An: Core-Libs-Dev
Betreff: RFR: JDK-8197594: String#repeat

Introduction of a new instance method String::repeat to allow an efficient and concise approach for generating repeated character sequences as strings.

Performance information in JBS.

Thank you.


— Jim

JBS: https://bugs.openjdk.java.net/browse/JDK-8197594 <https://bugs.openjdk.java.net/browse/JDK-8197594>
CSR: https://bugs.openjdk.java.net/browse/JDK-8198296
Webrev: http://cr.openjdk.java.net/~jlaskey/8197594/webrev-02/index.html <http://cr.openjdk.java.net/~jlaskey/8197594/webrev-02/index.html>
JavaDoc: http://cr.openjdk.java.net/~jlaskey/8197594/String.html <http://cr.openjdk.java.net/~jlaskey/8197594/String.html>

More information about the core-libs-dev mailing list