[lworld] Integrated: 8336682: [lworld] java/lang/instrument/IsModifiableClassAgent.java test fails when run with migrates classes
Coleen Phillimore
coleenp at openjdk.org
Fri Jul 19 20:43:48 UTC 2024
On Fri, 19 Jul 2024 12:12:29 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> The test uses the class file reconstituter to provide the redefined bytecodes, leaving out the LoadableDescriptors. Adding them to the reconsistituted class file makes the test pass, also fix the verification code since these descriptors are utf8s now. Tested with java/lang/instrument with --enable-preview. Should write a test but I don't know how, and the code is already tested with permitted subclasses and other attributes.
This pull request has now been integrated.
Changeset: 82f977fd
Author: Coleen Phillimore <coleenp at openjdk.org>
Committer: Frederic Parain <fparain at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/82f977fd7317892caaf960c9e4b41f11be9316f8
Stats: 47 lines in 3 files changed: 27 ins; 18 del; 2 mod
8336682: [lworld] java/lang/instrument/IsModifiableClassAgent.java test fails when run with migrates classes
Reviewed-by: heidinga, fparain
-------------
PR: https://git.openjdk.org/valhalla/pull/1172
More information about the valhalla-dev
mailing list