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

Aleksey Shipilev aleksey.shipilev at
Thu Nov 26 13:54:51 UTC 2015

Hi Andrej,

Thanks for the review, I'll send the updated webrevs once I address
Maurizio's review comments.

On 11/26/2015 11:42 AM, Andrej Golovnin wrote:
> src/java.base/share/classes/java/lang/
> src/java.base/share/classes/java/lang/
> I miss JavaDocs for the returned value of the methods #getChars(int,
> int, byte[]) and #getCharsUTF16(int, int, byte[]). And it would be
> nice to have JavaDocs for the parameters too.

Added, thanks!

> src/java.base/share/classes/java/lang/
> src/java.base/share/classes/java/lang/invoke/
> test/java/lang/String/concat/
> test/java/lang/String/concat/
> The files do not have a copyright header.

Oops, good catch, added.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <>

More information about the compiler-dev mailing list