On Thu, 9 Jan 2025 08:40:37 GMT, Adam Sotona <asot...@openjdk.org> wrote:

>> There are no more consumers of ASM library except for hotspot tests.
>> This patch moves ASM library from java.base module to the hotspot test 
>> libraries location and fixes the tests.
>> 
>> Please review.
>> 
>> Thanks,
>> Adam
>
> Adam Sotona has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed jdk.internal package prefix from asm

Thanks for the reviews. I'll update the copyright years and run again tier1 - 4 
tests (just to be sure).

Jtreg seems to have no problems to use ASM as a declared library. This PR 
primary purpose is to remove the ASM from java.base.  Further improvements of 
hotspot tests using ASM can follow. However I would prefer to see further 
conversions of the hotspot tests to use Class-File API instead. Long-term goal 
is to remove ASM from JDK completely.

FYI: Regarding the merge of this PR I've been asked to wait until some 
downstream projects dependent on jdk.internal.objectweb.asm are fixed.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/22946#issuecomment-2582985694

Reply via email to