RFR: 8367129: Move input validation checks to Java for java.lang.StringLatin1 intrinsics [v4]

Volkan Yazici vyazici at openjdk.org
Mon Feb 16 19:12:39 UTC 2026


On Mon, 16 Feb 2026 14:46:09 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:

>> Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   More `indexOf` et al. guards and other review feedback
>
> src/java.base/share/classes/java/lang/StringLatin1.java line 475:
> 
>> 473:             return 0;
>> 474:         }
>> 475:         if (value.length == 0) {
> 
> These checks could be removed altogether, and instead rely on those in the invoked method.
> 
> Alternatively, this line could be
> Suggestion:
> 
>         if (value.length < str.length) {
> 
> to match https://github.com/openjdk/jdk/pull/28832/changes#diff-0429b143c93fceff618ecb5bb78b584f2b2ea4771241c7dd450ddb9151c1c515R511 more closely.

Right. Removed the checks in 0966a6617d4.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28832#discussion_r2813738888


More information about the core-libs-dev mailing list