Saulius Krasuckas wrote:
* 6 years, 7 months ago Eric Pouech did a change [*]:

Started calling mixer as a 32 bit low level driver (no longer 16 bit)
Added support for yield proc in mci struct
Added mmTaskXXX16 and mmThreadXXX16 functions
Refreshed implementation of 16 bit native & asynchronous MCI driver command 
(through mmtask.tsk).

I'm do not follow a logic, as nowadays PSDK defines DRV_SUCCESS as 0x0001. Eric, can't both cases of 1 and DRV_SUCCESS be merged into a single case? Maybe this change address some 16-bit issues. No?

trying to get my mind 6 years & 7 months ago. As it was a Monday, it makes things even harder ;-) It seems that the main change in that area was to change all return values, but IIRC some were documented (with DRV_SUCCESS) some other weren't (or were documented as boolean).
Setting everything as DRV_SUCCESS should be fine.

But why are you looking at this patch ?
A+

--
Eric Pouech



Reply via email to