On Sat, 10 Apr 2021 01:10:37 GMT, Alex Menkov <[email protected]> wrote:
> 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
test/jdk/java/lang/instrument/ATransformerManagementTestCase.java line 125:
> 123: manager.addTransformer(transformer, canRetransform);
> 124: // verbosePrint("Added transformer " + transformer
> 125: // + " with canRetransform=" + canRetransform);
I'd suggest to remove the commented out lines.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3426
