Rémi Bernon (@rbernon) commented about dlls/dmime/tests/dmime.c:
hr = IDirectMusicTool_Flush(tool, performance, &msg, 0); todo_wine ok(hr == S_OK, "got %#lx\n", hr);
+ hr = IDirectMusicPerformance_Init(performance, NULL, NULL, NULL); + ok(hr == S_OK, "got %#lx\n", hr); + + hr = IDirectMusicPerformance_AddPort(performance, NULL); + ok(hr == S_OK, "got %#lx\n", hr); + + hr = IDirectMusicPerformance_GetTime(performance, NULL, &mt); + ok(hr == S_OK, "got %#lx\n", hr); + hr = IDirectMusicPerformance_MusicToReferenceTime(performance, mt, &rt); + ok(hr == S_OK, "got %#lx\n", hr);
```suggestion:-3+0 hr = IDirectMusicPerformance_GetTime(performance, &rt, &mt); ok(hr == S_OK, "got %#lx\n", hr); ``` I also think it'd be better to get the time before each test, to avoid diverging. Fwiw, I'd just make note60 a local variable, and use its `rtTime` / `mtTime` members directly instead of `rt` / `mt`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4438#note_52846