Re: RFR: 8175891, 8192892: JrtPath improvements

2017-12-01 Thread Claes Redestad
Jim, Sundar, thanks for the quick reviews! /Claes On 2017-12-01 14:53, Sundararajan Athijegannathan wrote: +1 -Sundar On 01/12/17, 7:04 PM, Claes Redestad wrote: Hi, please review this patch that resolves two small but sometimes significant inefficiencies in jdk.internal.jrtfs.JrtPath:

Re: RFR: 8175891, 8192892: JrtPath improvements

2017-12-01 Thread Sundararajan Athijegannathan
+1 -Sundar On 01/12/17, 7:04 PM, Claes Redestad wrote: Hi, please review this patch that resolves two small but sometimes significant inefficiencies in jdk.internal.jrtfs.JrtPath: Improve JrtPath::getResolved fast-path test: https://bugs.openjdk.java.net/browse/JDK-8192892 Testing for '.'

Re: RFR: 8175891, 8192892: JrtPath improvements

2017-12-01 Thread Jim Laskey (Oracle)
+1 > On Dec 1, 2017, at 9:34 AM, Claes Redestad wrote: > > Hi, > > please review this patch that resolves two small but sometimes significant > inefficiencies in jdk.internal.jrtfs.JrtPath: > > Improve JrtPath::getResolved fast-path test: >

RFR: 8175891, 8192892: JrtPath improvements

2017-12-01 Thread Claes Redestad
Hi, please review this patch that resolves two small but sometimes significant inefficiencies in jdk.internal.jrtfs.JrtPath: Improve JrtPath::getResolved fast-path test: https://bugs.openjdk.java.net/browse/JDK-8192892 Testing for '.' is insufficient as it's likely all paths have a dot in