Re: [ros-dev] [ros-diffs] [hyperion] 38461: modified tests/pseh2/psehtest.c finally_13 test re-enabled, as it doesn't crash anymore finally_14 test fixed. Now we know how exceptions thrown in a __finally are supposed to be handled PSEH test suite now has 91 tests and passes all of them with both GCC and Visual C++

[KJK::Hyperion]

Timo Kreuzer wrote:
>   1.) All used variables are static. That means the
> optimizer is mostly
>  out of the game. 
It's intentional. Using functions instead of constants is intentional,
too. Ensures variables are actually instanced, even if they are
"unused". And PSEH macros have slightly different behavior when
compile-time constants are used in some places. I'm unsure how to test
*reliably* interactions between optimizer bugs and PSEH

>   So I did some testing. After removing the static, I
> got 1 error:
>  psehtest.c:2490: Test failed: test_continue_search_3 failed 
The return value of return_positive() is not stored anywhere, and ret
remains uninitialized. Congratulations! you found a compiler bug

>   After replacing the return_x, ... functions with
> macros I got 3 errors: 
I don't. Send me your modifications