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

Dmitry Timoshkov dmitry at baikal.ru
Sat Jun 6 23:57:50 CDT 2020


Zebediah Figura <z.figura12 at gmail.com> wrote:

> On 6/6/20 11:45 PM, Dmitry Timoshkov wrote:
> > Jeff Smith <whydoubt at gmail.com> 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.

-- 
Dmitry.



More information about the wine-devel mailing list