Re: RFR (L) JEP 280: Indify String Concatenation (integration)

2016-01-29 Thread Erik Joelsson
On 2016-01-27 14:55, Aleksey Shipilev wrote: c) (XS) Build changes that force emitting the "legacy" inline StringBuilder concat in a few cases (e.g. when pre-JDK 9 bytecode is expected): http://cr.openjdk.java.net/~shade/8085796/webrev.root.00/ In what context do we need pre-JDK 9 bytecod

Re: RFR (L) JEP 280: Indify String Concatenation (integration)

2016-01-28 Thread Maurizio Cimadamore
Langtools changes look fine to me (as discussed on previous review thread). Thanks! Maurizio On 27/01/16 13:55, Aleksey Shipilev wrote: Hi again, This is a formal pre-integration review thread for JEP 280 ("Indify String Concatenation") integration: http://openjdk.java.net/jep

Re: RFR (L) JEP 280: Indify String Concatenation (integration)

2016-01-27 Thread Vladimir Ivanov
JDK changes looks good! Best regards, Vladimir Ivanov On 1/27/16 4:55 PM, Aleksey Shipilev wrote: Hi again, This is a formal pre-integration review thread for JEP 280 ("Indify String Concatenation") integration: http://openjdk.java.net/jeps/280 The JEP is Targeted, the CCC i

Re: RFR (L) JEP 280: Indify String Concatenation (integration)

2016-01-27 Thread Paul Sandoz
I think it quite reasonable to push everything to jdk9/dev. Paul. > On 27 Jan 2016, at 14:55, Aleksey Shipilev > wrote: > > Hi again, > > This is a formal pre-integration review thread for JEP 280 ("Indify > String Concatenation") integration: > http://

RFR (L) JEP 280: Indify String Concatenation (integration)

2016-01-27 Thread Aleksey Shipilev
Hi again, This is a formal pre-integration review thread for JEP 280 ("Indify String Concatenation") integration: http://openjdk.java.net/jeps/280 The JEP is Targeted, the CCC is approved, the code reviews and pre-integration checks are clean. Code changes are happening simulta

JEP 280: Indify String Concatenation

2015-11-18 Thread mark . reinhold
New JEP Candidate: http://openjdk.java.net/jeps/280 - Mark