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
    
    
  
Hi
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.
Leonid
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