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

Nikolay Sivov nsivov at codeweavers.com
Wed Oct 29 00:19:48 CDT 2014



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.



More information about the wine-devel mailing list