replace a wrong fixme in dlls/winmm/playsound.c

Eric Pouech pouech-eric at wanadoo.fr
Fri Nov 19 12:23:11 CST 2004


Rémi Assailly a écrit :
> Andreas Mohr wrote :
> 
>> Hi,
>>
>> On Thu, Nov 18, 2004 at 02:30:55PM +0100, Rémi Assailly wrote:
>>  
>>
>>> I tested a software ( a monopoly game ) and the error was that a 
>>> sound was still playing so I think that is not a wine bug.
>>> Before submitting this patch to wine-patches I would like to know 
>>> what do you think about it, what should be WARN, FIXME, if choosing 
>>> ERR to all is correct or if some cases are useless.
>>>   
>>
>> Way too big, please use
>> #define SND_ERR(err, text) \
>>     case err: \
>>       SOMETRACE( text "\n"); \
>>       break;
>>
>> A sound still playing could easily be a winmm bug, not only a card 
>> driver bug...
>>
>> Andreas Mohr
>>  
>>
> I think it is not a wine bug because :
> * there are 8 "players"
> * the error occures only with two ( shoe and iron )
> 
> Sorry, I do not understand what you advise me to do. Could you send me 
> an example using an error ?
> If there is a wine error I think this patch will be a good way to debug it.
> 
a couple of remarks
- use waveOutGetErrorText instead of you own macros
- don't declare variables inside block

more globally, I don't understand the aim of the patch ? what are you trying to do ?




More information about the wine-devel mailing list