Re: [PATCH 1/2] setupapi/tests: Add error tests for SetupDiDestroyDeviceInfoList.

[Jeff Smith]

On Sat, Jun 6, 2020, 11:58 PM Dmitry Timoshkov  wrote:
> Zebediah Figura  wrote:
> > On 6/6/20 11:45 PM, Dmitry Timoshkov wrote:
> > > Jeff Smith  wrote:
> > > > - SetLastError(0xcafef00d);
> > >
> > > Please use a commonly accepted value of 0xdeadbeef.
> >
> > Is this really worthwhile? It's pretty obvious that both are sentinel
> > values.
>
> It's just one of common practices in Wine tests, IMO it would be nice to
> use an existing one instead of inventing new all other the place.

As deadbeef is also retuned by class_error and co_error, I intentionally
chose a different value so that it would possible to determine the true
source of the value.  What solution do you recommend?

Thanks,
Jeff