Re: [PATCH v3 0/1] MR4820: dlls/winmm: waveOutGetID: Return correct id for WAVE_MAPPER.

[Carlo Bramini (＠carlo-bramini)]

On Sun Jan 14 11:56:07 2024 +0000, eric pouech wrote:
> please also add a test (in wimm/tests/wave.c) to demonstrate that your
> fix is correct
> the demo you wrote in the bug report is a good start, but
> * has to be integrated into test framework
> * waveOutGetId is not tested yet :-(
> * there's no need to get waveOutGetDevCaps as it's already tested
> * adding a test for another dev id different from mapper is also interesting
> I added the test for checking if `waveOutGetID()` returns the expected value.
> It is called for each audio device, including `WAVE_MAPPER`, so it should fit the request.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4820#note_57367