[PATCH] wineoss.drv: Fix IAudioRenderClient::{Get,Release}Buffer protocol

Andrew Eikum aeikum at codeweavers.com
Mon Jan 9 15:25:32 CST 2012


---

Here's the OSS counterpart to Jörg's winealsa patch.

With this applied, the output from Jörg's extended render tests is:

render.c:667: Test failed: Expected 10ms default period: 200000
render.c:692: Test succeeded inside todo block: GetBuffer 0 gave (nil)
render.c:728: Test succeeded inside todo block: NULL expected (nil)
render.c:745: Test succeeded inside todo block: GetBuffer 0 gave (nil)
render.c:750: Test succeeded inside todo block: GetBuffer 0 size #2 failed: 00000000
render.c:751: Test succeeded inside todo block: GetBuffer 0 #2 gave (nil)
render.c:758: Test succeeded inside todo block: GetBuffer failed: 00000000
render.c:1062: Tests skipped: Rerun with WINETEST_DEBUG=2 for GetPosition tests.
render.c:1166: Test failed: Position should have been further along...
render.c:1227: Test failed: GetCurrentPadding returned 512, should be 0
render.c:1228: Test failed: Position 142756 at end vs. 143268 submitted frames
render.c:1062: Tests skipped: Rerun with WINETEST_DEBUG=2 for GetPosition tests.
render.c:1166: Test failed: Position should have been further along...

I think the GetPosition and GetCurrentPadding failures are caused by
using GETOSPACE instead of GETODELAY in GetPosition().

 dlls/wineoss.drv/mmdevdrv.c |   38 +++++++++++++++++++++++++-------------
 1 files changed, 25 insertions(+), 13 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-wineoss.drv-Fix-IAudioRenderClient-Get-Release-Buffe.patch
Type: text/x-patch
Size: 4071 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20120109/d27a1a61/attachment.bin>


More information about the wine-patches mailing list