Re: [PATCH 1/6] comctl32/tests: Don't dump successful sequences on Windows.

[Stefan Dösinger]

Am 2017-04-05 um 14:36 schrieb Nikolay Sivov:
> This patch is marked is failed by testbot, but it looks like bot
> inability to deal with header file change in tests directory, as it
> still tries to run some tests.

Yeah, I think so, especially since the follow-up patches run ok.
> Anyway, could you make it look similar to user32/tests/msg.c? It's using:
> if( todo && !failcount) /* succeeded yet marked todo */
>     todo_wine {
>         if (!strcmp(winetest_platform, "wine")) dump++;
>
> Platform is tested directly. It's not the only place where it's done
> like that, and it's more readable than broken(1), I think.

Sure, I'll change it :-)