Re: TestBot job 77542 results: [PATCH 1/2] winmm: Add timeGetTime() implementation.

[Arkadiusz Hiler]

On Tue, Aug 25, 2020 at 03:11:21AM -0500, Marvin wrote:
> VM                   Status   Failures Command
> debiant              failed   4
> debiant              failed   2
>
> You can also see the results at:
> https://testbot.winehq.org/JobDetails.pl?Key=77542
>
> === debiant (32 bit report) ===
>
> winmm:
> midi.c:892: Test failed: expected greater than 51ms, got 51ms
>
> === debiant (32 bit French report) ===
>
> winmm:
> midi.c:892: Test failed: expected greater than 51ms, got 51ms
>
> === debiant (32 bit Japanese:Japan report) ===
>
> winmm:
> midi.c:892: Test failed: expected greater than 51ms, got 51ms
>
> === debiant (32 bit Chinese:China report) ===
>
> winmm:
> midi.c:892: Test failed: expected greater than 51ms, got 51ms
>
> === debiant (64 bit WoW report) ===
>
> winmm:
> midi.c:892: Test failed: expected greater than 51ms, got 51ms
> midi.c:892: Test failed: expected greater than 51ms, got 51ms

I don't know a lot about MIDI, but AFAIU this is an issue that also was
caused by the GetTickCount() change. It's shows up here because this
patch touches tests/midi.c

2/2 seems to be fixing it as well - I've run the MIDI tests in a loop
for a few minutes without any failures.