Code (Pre-)Review for JEP 280: Indify String Concat

Andrej Golovnin andrej.golovnin at gmail.com
Mon Nov 30 14:03:47 UTC 2015


Hi Aleksey,

> Yes, thanks for more polishing, I have uploaded the updates to:
>   http://cr.openjdk.java.net/~shade/8085796/webrev.jdk.06/

Maybe one more:

1342                     Class<?>[] cls = new Class<?>[cnt];
1343                     for (int i = 0; i < cnt; i++) {
1344                         cls[i] = int.class;
1345                     }

The for-loop can be replaced by:

Arrays.fill(cls, int.class);

Best regards,
Andrej Golovnin


More information about the compiler-dev mailing list