> The test actually failed starting from jdk13, but the error is masked by > JDK-8264667 (and shell part of the test didn't verify result of the java > execution) > The fix: > - updates JvmtiClassFileReconstituter to add attributes in the same order as > javac does > - makes the test java instead of shell > - added workaround for JDK-8264667 > - test code is ready to ignore order of attributes
Alex Menkov has updated the pull request incrementally with one additional commit since the last revision: Updated the fix accordingly feedback ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/3426/files - new: https://git.openjdk.java.net/jdk/pull/3426/files/62b5b474..67ce2ea0 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3426&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3426&range=00-01 Stats: 30 lines in 2 files changed: 1 ins; 6 del; 23 mod Patch: https://git.openjdk.java.net/jdk/pull/3426.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/3426/head:pull/3426 PR: https://git.openjdk.java.net/jdk/pull/3426