dplayx: Fix some crashes and wine_todo's ( try 2)

[Alistair Leslie-Hughes]

On 29/10/14 16:19, Nikolay Sivov wrote:
>
>
> On 10/29/2014 08:13 AM, Alistair Leslie-Hughes wrote:
>> +#if 0
>> +    /* Crashes under Win7 */
>>       hr = IDirectPlayX_GetSessionDesc( pDP[0], lpData[0], NULL );
>>       checkHR( DPERR_INVALIDPARAM, hr );
>>       dwDataSize=-1;
>>       hr = IDirectPlayX_GetSessionDesc( pDP[0], lpData[0], &dwDataSize );
>>       checkHR( DPERR_INVALIDPARAMS, hr );
>>       check( -1, dwDataSize );
>> +#endif
>
> That's the same. Wine test use if (0) {} block to disable code. Also you
> don't need checkHR() at all in such cases.
>
We technically don't need checkHR but it does show what was being tested 
for.


Alistair Leslie-Hughes