Re: mmsystem function signatures

2000-09-27 Thread Francois Gouget



   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

2000-09-27 Thread Eric Pouech

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

2000-09-27 Thread Francois Gouget

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

2000-09-22 Thread Francois Gouget


   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

2000-09-22 Thread Eric Pouech

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