RFR: 8328154: Convert sun/java2d/loops/CopyAreaSpeed.java applet test to main
Alexander Zuev
kizune at openjdk.org
Fri Mar 15 15:41:30 UTC 2024
On Thu, 14 Mar 2024 16:04:50 GMT, Tejesh R <tr at 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
More information about the client-libs-dev
mailing list