On Fri, 4 Feb 2022 11:18:39 GMT, Alex Menkov <amen...@openjdk.org> wrote:
> The test expects ClassFileReconstituter restores exactly the same bytes as > original classbytes. > This can be wrong if the class has more than 1 method (due to method sorting > in the VM). > MethodParametersTarget class had only 1 method (method1), but didn't have > constructors. This caused declaration of implicit default constructor, so the > class actually had 2 methods. > The fix converts the method to constructor to avoid default constructor > declaration. This pull request has now been integrated. Changeset: 95fd9d20 Author: Alex Menkov <amen...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/95fd9d20f329b15d68e613ec7f932254715e9130 Stats: 6 lines in 1 file changed: 1 ins; 0 del; 5 mod 8281243: Test java/lang/instrument/RetransformWithMethodParametersTest.java is failing Reviewed-by: sspitsyn, dcubed, lmesnik ------------- PR: https://git.openjdk.java.net/jdk/pull/7345
