RFR: 8290059: Do not use std::thread in panama tests [v2]
Jorn Vernee
jvernee at openjdk.org
Tue Jul 26 13:43:05 UTC 2022
On Tue, 26 Jul 2022 12:58:09 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> test/lib/native/testlib_threads.h line 50:
>>
>>> 48: static void fatal(const char* message) {
>>> 49: perror(message);
>>> 50: exit(-1);
>>
>> Won't work as intended for Windows APIs. I would print the result of `GetLastError()` instead.
>>
>> Alternatively I am fine fine with just omitting the error code, because I think the old tests did not handle errors either. Or did we catch std::thread exceptions somewhere?
>
> The intent was to exit the test with a non-zero exit code, in order to avoid any accidental false positives.
>
> I could return the error code from `GetLastError` and from the respective pthread apis as an exit code instead. Is that what you mean?
FWIW, `perror` just prints to `stderr`: https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/perror-wperror?view=msvc-170
-------------
PR: https://git.openjdk.org/jdk/pull/9599
More information about the build-dev
mailing list