Code (Pre-)Review for JEP 280: Indify String Concat
Maurizio Cimadamore
maurizio.cimadamore at oracle.com
Sun Nov 29 21:41:53 UTC 2015
Looks great - thanks!
Maurizio
On 29/11/15 18:23, Aleksey Shipilev wrote:
> Hi Maurizio,
>
> Updated webrevs:
> http://cr.openjdk.java.net/~shade/8085796/webrev.langtools.04/
> http://cr.openjdk.java.net/~shade/8085796/webrev.jdk.05/
>
> On 11/27/2015 10:13 PM, Maurizio Cimadamore wrote:
>> Looks great - the only minor quibble is that now StringConcat looks like
>> a regular javac context class; it even has an instance method - it's
>> therefore best to follow usual initialization pattern for javac components:
> That makes sense. I had a concern that Gen and StringConcat are
> circularly dependent on each other, but this seems to be handled by
> exposing the underconstructed Gen early.
>
> Thanks,
> -Aleksey
>
>
More information about the compiler-dev
mailing list