On Fri, 11 Oct 2024 16:02:29 GMT, Chen Liang <[email protected]> wrote:
>> src/java.base/share/classes/jdk/internal/classfile/impl/ClassRemapperImpl.java
>> line 276:
>>
>>> 274: @Override
>>> 275: public ClassDesc map(ClassDesc desc) {
>>> 276: if (desc == null) return null;
>>
>> I'm not sure this won't break some use case.
>> Here I would say null -> null mapping might be OK.
>
> Makes sense. I will push the changes when you finish the review.
It is all I have.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21458#discussion_r1797160380
