[OpenJDK 2D-Dev] Fix for 8235904 "Infinite loop when rendering huge lines"
Phil Race
philip.race at oracle.com
Tue Jan 14 20:33:13 UTC 2020
Here's your webrev with a jtreg compliant test added :
http://cr.openjdk.java.net/~prr/8235904/
I verified the test behaves properly under jtreg -
before the fix jtreg kills it on time out
after the fix it finishes quickly and successfully
Outside of jtreg it also exits properly although jtreg is the normal way
to run it.
This is all +1 from me but I think someone else should sign off on this
too since I
don't want to self-review the test.
-phil.
On 1/12/20 11:54 AM, Phil Race wrote:
> Hi Clemens,
>
> That needs reworking in location as well as adding jtreg boiler plate and legal notice. Also there are code changes required to be a well behaved test. I need to sit in front of a headful Linux system to test those so I will have to follow up on this (hopefully) tomorrow.
>
> -Phil.
>
>> On Jan 11, 2020, at 12:40 PM, Clemens Eisserer <linuxhippy at gmail.com> wrote:
>>
>> Hi Phil,
>>
>>> Is there a regression test ?
>>> I don't see a noreg- label.
>> Sorry I forgot to mention, the regression test is in the "main"
>> directory of the bugfix:
>> http://cr.openjdk.java.net/~ceisserer/8235904/
>>
>> Thanks and best regards, Clemens
More information about the 2d-dev
mailing list