On Thu, 14 Mar 2024 16:04:50 GMT, Tejesh R <t...@openjdk.org> wrote:

>> test/jdk/sun/java2d/loops/CopyAreaSpeed.java line 131:
>> 
>>> 129:             }
>>> 130:             try {
>>> 131:                 Thread.sleep(1);
>> 
>> I know it is an old code but i just wonder why do we add sleep to every 
>> iteration of something that is supposed to be a performance test. What are 
>> we achieving by doing this sleep?
>
> Maybe because of Graphics paint and copyArea ?

It is still a little bit suspicious because adding fixed delay into the test 
loop makes the test less accurate the fastest hardware gets but i guess in 
order to maintain the test value compared to the previous versions of the test 
it will be wrong to change time calculations.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18296#discussion_r1526473747

Reply via email to