If no existing Wine test covers this case, then you should add a test both demonstrating that your change is correct and to prevent any future changes from accidentally re-breaking things.
-Ken
On Mar 15, 2019, at 2:55 PM, Derek Lesho dereklesho52@gmail.com wrote:
I'm not sure why that test fails, that test shouldn't even trigger the code I change. By the way, I made a test case that compiles in MSVC. It's slightly modified from the MSDN's seh example to showcase the issue with leaf functions: https://godbolt.org/z/mo1-HV
On Fri, Mar 15, 2019 at 3:40 PM Marvin testbot@winehq.org wrote:
Hi,
While running your changed tests, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check?
Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=49463
Your paranoid android.
=== debian9b (64 bit WoW report) ===
ntdll: env.c:498: Test failed: wrong end ptr 0x1238c/0x12390