mmdevapi/tests: Perform renderer padding & position tests. (try2)

Joerg-Cyril.Hoehle at t-systems.com Joerg-Cyril.Hoehle at t-systems.com
Fri Feb 10 03:32:07 CST 2012


Hi,

This testsuite has already proven useful at detecting a 64bit error.

I've also added comments about API behavior. I think the tests
are a better place than the code as they are less likely to
be removed by a rewrite of a component: old code gets nuked
and ignored, the tests remain.

I'm not satisfied with the lack of comments about important
things to know in winmm, DSound and mmdevapi.

Try2: I had badly merged the audible patch, revealing a new testcase:
RenderClient_ReleaseBuffer(flags=1) yields 80070057
which Wine does not follow. That's why the tests passed in Wine, not in testbot.

Regards,
	Jörg Höhle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-mmdevapi-tests-Perform-renderer-padding-position.patch
Type: application/octet-stream
Size: 28375 bytes
Desc: 0003-mmdevapi-tests-Perform-renderer-padding-position.patch
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20120210/055a7261/attachment-0001.obj>


More information about the wine-patches mailing list