RFR: 8328874: Class::forName0 should validate the class name length early [v8]
Guanqiang Han
ghan at openjdk.org
Thu Aug 28 00:48:50 UTC 2025
On Wed, 27 Aug 2025 17:52:11 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Guanqiang Han has updated the pull request incrementally with one additional commit since the last revision:
>>
>> change copyright year
>
> src/java.base/share/classes/java/lang/Class.java line 95:
>
>> 93: import sun.reflect.annotation.*;
>> 94:
>> 95: import static jdk.internal.util.ModifiedUtf.utfLen;
>
> Do not use static imports across package boundaries, it make the code harder to read and to know where that method came from.
Hi @RogerRiggs , thank you for pointing out this detail. I’ve updated the PR accordingly and will pay more attention to such readability issues in future changes. Please have another look.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26802#discussion_r2305755787
More information about the core-libs-dev
mailing list