RFR: 8292054: Test runtime/posixSig/TestPosixSig.java fails with 'Test failed, bad output.' [v3]

[Thomas Stuefe]

On Wed, 17 Aug 2022 19:02:46 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> test/hotspot/gtest/runtime/test_signals.cpp line 51:
>> 
>>> 49:     act.sa_flags = 0;
>>> 50:     ASSERT_NE(sigaction(SIGFPE, &act, &old_SIGFPE_act), -1) << "Setting SIGFPE handler failed (errno)";
>>> 51:     ASSERT_NE(sigaction(SIGILL, &act, &old_SIGILL_act), -1) << "Setting SIGILL handler failed (errno)";
>> 
>> Does that actually work? I would have thought that just prints the literal "errno". I meant something like this:
>> 
>> https://github.com/openjdk/jdk/blob/081c89316a2f35cad0a5dcc93e88707530e03d72/test/hotspot/gtest/logging/logTestUtils.inline.hpp#L56
>
> That was rather stupid of me.  Commit 4 contains your suggested errno change.

No problem. For a moment I wondered if that was some formatting magic I didn't know :)

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

PR: https://git.openjdk.org/jdk/pull/9882