Re: mmsystem function signatures
With the big movements that happened in mmsystem.h I noticed that mixerMessage16 returns a UINT16. But I thought that all the xxxMessage16 APIs were supposed to return a DWORD. Or is this an exception? -- Francois Gouget [EMAIL PROTECTED]http://fgouget.free.fr/ Linux: It is now safe to turn on your computer.
Re: mmsystem function signatures
Francois Gouget wrote: With the big movements that happened in mmsystem.h I noticed that mixerMessage16 returns a UINT16. But I thought that all the xxxMessage16 APIs were supposed to return a DWORD. Or is this an exception? yup... mixerMessage16 is wrong (but it was already wrong *before* the "big movements") A+ -- --- Eric Pouech (http://perso.wanadoo.fr/eric.pouech/) "The future will be better tomorrow", Vice President Dan Quayle
Re: mmsystem function signatures
On Wed, 27 Sep 2000, Eric Pouech wrote: Francois Gouget wrote: With the big movements that happened in mmsystem.h I noticed that mixerMessage16 returns a UINT16. But I thought that all the xxxMessage16 APIs were supposed to return a DWORD. Or is this an exception? yup... mixerMessage16 is wrong Ok. I sent a patch to fix it. (but it was already wrong *before* the "big movements") Yes. I just meant that I noticed it then (can you say merge conflicts) -- Francois Gouget [EMAIL PROTECTED]http://fgouget.free.fr/ 145 = 1! + 4! + 5!
mmsystem function signatures
Hi, I noticed that many functions in mmsystem.[ch] use UINT/UINT16 as their return type rather than the official MMRESULT/MMRESULT16, MCIDEVICEID/MCIDEVICEID16 or MCIERROR/MCIERROR16. So I started a patch to make things closer to the specs. But I found that some 16 bit functions return a DWORD. I propose to change their return types as follows, would it be ok? type current proposed API Documentation DWORD - WORD mciGetDriverData16 (undocumented) DWORD - WORD xxx{In,Out}Message16(DWORD in win32) DWORD - MCIERROR16 mciSendCommand16(MCIERROR in win32) (see grep WINAPI ../wine.ref/dlls/winmm/mmsystem.c | grep 16 | sort | more) The rest of the changes consist in replacing some UINT(16) and DWORD return types byt the appropriate MMRESULT(16), MMDEVICEID(16) and MCIERROR(16). -- Francois Gouget [EMAIL PROTECTED]http://fgouget.free.fr/ Demander si un ordinateur peut penser revient à demander si un sous-marin peut nager.
Re: mmsystem function signatures
Francois Gouget wrote: Hi, I noticed that many functions in mmsystem.[ch] use UINT/UINT16 as their return type rather than the official MMRESULT/MMRESULT16, MCIDEVICEID/MCIDEVICEID16 or MCIERROR/MCIERROR16. So I started a patch to make things closer to the specs. But I found that some 16 bit functions return a DWORD. I propose to change their return types as follows, would it be ok? type current proposed API Documentation DWORD - WORD mciGetDriverData16 (undocumented) no this should be a DWORD (as mciSetDriverData16 takes a DWORD) (to store a far pointer) DWORD - WORD xxx{In,Out}Message16(DWORD in win32) returns also a 32 bit value on 16bit mmsystem DWORD - MCIERROR16 mciSendCommand16(MCIERROR in win32) no, mciSendCommand returns a 32 bit entity in 16bit mmsystem (see grep WINAPI ../wine.ref/dlls/winmm/mmsystem.c | grep 16 | sort | more) I'll have to check out also the *Drv* which are likely to use 32bit values (so don't change them for now) I'd say that changing the size would be suspect to me (changing the type from UINT16 to MMRESULT16 is pure cosmetic to me, as MMRESULT16 is a typedef to UINT16) A+ -- --- Eric Pouech (http://perso.wanadoo.fr/eric.pouech/) "The future will be better tomorrow", Vice President Dan Quayle