[PATCH 3/7] ddraw: Fix IDirectDraw_CreateSurface's invalid descriptor error code.

[Alex Henrie]

2017-02-15 6:49 GMT-07:00 Henri Verbeet <hverbeet at gmail.com>:
> On 15 February 2017 at 07:53, Alex Henrie <alexhenrie24 at gmail.com> wrote:
>> @@ -3000,7 +3000,7 @@ static HRESULT WINAPI ddraw1_CreateSurface(IDirectDraw *iface,
>>      {
>>          WARN("Application supplied invalid surface descriptor\n");
>>          wined3d_mutex_unlock();
>> -        return DDERR_INVALIDPARAMS;
>> +        return E_INVALIDARG;
> Did you verify the test fails without this change? How is
> DDERR_INVALIDPARAMS defined?

You're right; these patches are unnecessary. ddraw.h has:

#define DDERR_INVALIDPARAMS E_INVALIDARG

Which of the two is preferred for ddraw code?

-Aex