Whitelisting modules in layers

2017-12-01 Thread Mark Raynsford
oduleFinder API a little, and came up with the following: https://github.com/io7m/moduledemo-20171201/blob/master/src/main/java/com/io7m/moduledemo/WhitelistModuleDemo.java However, the boot_layer.defineModulesWithOneLoader() call raises an exception: "Class loader must be the boot c

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: > https://bugs.openjdk.java.net/browse/JDK-8192892 >

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 th