mmdevapi: SetEventHandle is allowed only once.

Joerg-Cyril.Hoehle at t-systems.com Joerg-Cyril.Hoehle at t-systems.com
Mon Dec 17 09:33:23 CST 2012


Hi,

I wondered about the lifetime of the various objects.

MSDN is silent about this.  Our testing indicates that SetEventHandle
is not allowed twice.

>+        FIXME("called twice\n");
It's not that I don't trust my own code since we have tests.  But
tests are just tests, no proof that it won't work using another
combination of flags or ordering of methods.  Therefore, I want a
FIXME to show up should some app ever try to change the event.

Why do I care at all?  Because it would be useful to know (or ensure)
that This->event is a constant for the lifetime of This between
IAC_Start and IAC_Release -- see my next patch.

Regards,
	Jörg Höhle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-mmdevapi-SetEventHandle-is-allowed-only-once.patch
Type: application/octet-stream
Size: 2827 bytes
Desc: 0002-mmdevapi-SetEventHandle-is-allowed-only-once.patch
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20121217/496eb923/attachment.obj>


More information about the wine-patches mailing list