RFR: 8254042: gtest/GTestWrapper.java failed os.test_random [v2]
Coleen Phillimore
coleenp at openjdk.java.net
Mon Nov 30 12:54:16 UTC 2020
On Fri, 27 Nov 2020 00:59:27 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Restore old copyright
>> - Refix os.random test using Thomas Stuefe's better suggestion.
>
> test/hotspot/gtest/runtime/test_os.cpp line 125:
>
>> 123: int num;
>> 124: for (int k = 0; k < reps; k++) {
>> 125: num = seed = os::next_random(seed);
>
> I suggest adding a comment before this line:
> // Use next_random so the calculation is stateless.
> Or something to that affect.
Done.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1422
More information about the hotspot-dev
mailing list