Re: [PATCH 0/2] MR5866: kernel32, dwrite: Tweak some tests to reduce test report sizes.

[eric pouech (＠epo)]

eric pouech (@epo) commented about dlls/kernel32/tests/debugger.c:
>  {
>      next_event(&ctx, WAIT_EVENT_TIMEOUT);
>      ok (ctx.ev.dwDebugEventCode != EXCEPTION_DEBUG_EVENT, "got exception\n");
> -
>    if (ctx.ev.dwDebugEventCode != EXCEPTION_DEBUG_EVENT) break;
> }
>    while (ctx.ev.dwDebugEventCode != EXIT_PROCESS_DEBUG_EVENT);
> - if (ctx.ev.dwDebugEventCode != EXIT_PROCESS_DEBUG_EVENT) TerminateProcess(pi.hProcess, 0);

If the thousand of "Test failed" lines come from the ok() inside the above loop, I wonder if the test to break out of the loop isn't inverted?
Otherwise you just skip the whole teardown, making the whole loop not useful.