On 24/09/2015 15:56, Chris Hegarty wrote:
: Ah ha, good catch. Fixed and webrev updated in-place.
Looks okay.
Offline, Paul queried whether the modules now using jdk.internal.misc still require access to sun.misc. They do, all but one java.sql. I updated the modules.xml to reflect this.
This looks okay too.
Worth noting, is that this change includes a workaround for 8137017: Remove JDK internal API dependency from jrt-fs.jar [1], in jdk/internal/jimage/ImageNativeSubstrate.java
Ugly but okay on a temporary basis until jrt-fs/jimage is fixed. -Alan