Re: [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(a)gmail.com>:
> On 15 February 2017 at 07:53, Alex Henrie <alexhenrie24(a)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