Hi Jim!

I think the copying-loop can be slightly simplified, like this:

    for (; copied < limit - copied; copied <<= 1) {
        System.arraycopy(multiple, 0, multiple, copied, copied);
    }

(didn't actually run it to check for correctness.)

With kind regards,
Ivan


On 2/28/18 8:31 AM, Jim Laskey wrote:
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.

Cheers,

— 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
  <http://cr.openjdk.java.net/~jlaskey/8197594/String.html>
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>





--
With kind regards,
Ivan Gerasimov

Reply via email to