JDK-8239787 : AArch64: String.indexOf may incorrectly handle empty strings

Leonid Mesnik leonid.mesnik at oracle.com
Fri Feb 28 21:12:25 UTC 2020


I have few comments about added test:

1. Are there any reasons to limit to aarch64 only? If it is valid for 
other platforms then let run it on these platforms also.

2. Please fix whit spaces in lines 33,34;

3. The meaningful directory names are preferred rather than bugid. Not a 
strict requirement but usually it increase readability.


On 2/28/20 3:47 AM, Alexey Bakhtin wrote:
> Hello Yuri,
> Here is webrev with fix and new jtreg test :
> http://cr.openjdk.java.net/~bae/8239787/webrev.01/
> Thank you
> Alexey
>> On 28 Feb 2020, at 11:45, Yuri Nesterenko <yan at azul.com> wrote:
>> /Hi Alexey, could you please provide a regression test with this fix?
>> Thanks, --yan

More information about the jdk-dev mailing list