Re: [ddraw/tests] Fix some test failures on Vista+

[Paul Vriens]

On 12/10/2009 04:04 PM, Stefan Dösinger wrote:
> Am 10.12.2009 um 13:21 schrieb Paul Vriens:
> > <0004-Fix-some-test-failures-on-Vista.patch>
> >      const char *name;
> >      DDPIXELFORMAT fmt;
> >      BOOL getdc_capable;
> > -
> >    HRESULT alt_result;
> > } testdata[] = {
>
> You are setting the alt_result only for the 4 failing tests. I think C sets the others to 0 because the initializer is static data I think its better to explicitly set it in all other test formats to avoid confusion

Yeah, but that's why I added the extra check for alt_result:

(testdata[i].alt_result && hr == testdata[i].alt_result)

It will look a bit strange adding S_OK (or the likes) for the alternate 
result.

I'm not saying it's common practice in the code but we have multiple of 
these structs where the last one(s) is not set.