> 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

Reply via email to