On Tue, 5 Sep 2023 16:05:14 GMT, Qing Xiao <d...@openjdk.org> wrote: >> `/test/jdk/tools/lib/tests/JImageValidator.java`, tests in >> `/test/jdk/tools/jlink`, and `/test/jdk/tools/jimage`, >> `/test/jdk/java/time/nontestng/java/time/chrono/HijrahConfigTest.java` use >> on com.sun.tools.classfile and should be converted to Classfile API. > > Qing Xiao has updated the pull request incrementally with one additional > commit since the last revision: > > migrate test/jdk/java/time/nontestng/java/time/chrono/HijrahConfigTest.java > use Classfile API
In addition, I don't think we should open the `classfile.impl` package to tests; this package is not meant to be exported. test/jdk/tools/lib/tests/JImageValidator.java line 225: > 223: > 224: public static void readClass(byte[] clazz) throws IOException { > 225: Classfile.of().parse(clazz); Notice that the old API might throw different exceptions compared to the new one; the new one is lazy, e.g. if there's broken interface entry but the interfaces are not queried, no exception is thrown. ------------- PR Comment: https://git.openjdk.org/jdk/pull/15529#issuecomment-1707443036 PR Review Comment: https://git.openjdk.org/jdk/pull/15529#discussion_r1316517689