Re: RFR: 8146686: Create the schemeSpecificPart for non-opaque URIs lazily

2016-01-10 Thread Chris Hegarty 
On 8 Jan 2016, at 14:41, Claes Redestad  wrote:

> Hi,
> 
> please review this patch to lazily create schemeSpecificPart for non-opaque 
> URIs. This change accounts for some improvement in footprint characteristics 
> and a 10-20% improvement on URI creation microbenchmarks, while not 
> regressing notably on a targetted microbenchmark which explicitly generate 
> and retrieve the schemeSpecificPart. 
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8146686
> Webrev: http://cr.openjdk.java.net/~redestad/8146686/webrev.01/

Looks ok to me Claes.

-Chris.

> Thanks!
> 
> /Claes

Re: RFR: 8146686: Create the schemeSpecificPart for non-opaque URIs lazily

2016-01-10 Thread Claes Redestad 


On 2016-01-10 10:39, Chris Hegarty wrote:

On 8 Jan 2016, at 14:41, Claes Redestad  wrote:


Hi,

please review this patch to lazily create schemeSpecificPart for non-opaque 
URIs. This change accounts for some improvement in footprint characteristics 
and a 10-20% improvement on URI creation microbenchmarks, while not regressing 
notably on a targetted microbenchmark which explicitly generate and retrieve 
the schemeSpecificPart.

Bug: https://bugs.openjdk.java.net/browse/JDK-8146686
Webrev: http://cr.openjdk.java.net/~redestad/8146686/webrev.01/

Looks ok to me Claes.

-Chris.


Thanks for reviews, Chris and Alan!

/Claes