[PATCH 1/2] dmime/tests: Added tests for IDirectMusicPerformance.

Chris Robinson chris.kcat at gmail.com
Wed Aug 11 11:56:39 CDT 2010


On Wednesday, August 11, 2010 3:12:23 am David Adam wrote:
> Hello,
> 
> some comments about this patch
> +    if (hr != S_OK) return FALSE;
> +    else return TRUE;
> 
> could be simplified as
> return SUCCEEDED(hr);

SUCCEEDED encompasses more than just S_OK (S_FALSE, for instance; basically 
anything that doesn't have the high bit set). Whether to test SUCCEEDED or 
S_OK explicitly depends on what the function can return and what it means.

The original behavior could instead be simplified as:
return hr==S_OK;



More information about the wine-devel mailing list