Re: [PATCH v7 0/3] MR8515: setupapi: Support in-built properties for device instances in SetupDiGetDevicePropertyW.

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about dlls/setupapi/tests/devinst.c:
> -
>    ok(ret, "Failed to set property, error %#lx.\n", GetLastError());
> -
> -
>    ret = SetupDiGetDeviceRegistryPropertyW(set, &device_data, prop->reg_prop, &type, buf, sizeof(buf), &size);
> -
>    todo_wine_if(todo) ok(ret, "Failed to get property, error %#lx.\n", GetLastError());
> -
>    todo_wine_if(todo) ok(type == prop->reg_type, "Got unexpected type %#lx.\n", type);
> -
>    todo_wine_if(todo) ok(size == prop->reg_size, "Got unexpected size %lu.\n", size);
> -
>    if (size == prop->reg_size)
> -
>        todo_wine ok(!memcmp(buf, prop->reg_value, size), "Got unexpected property value.\n");
> -
>    winetest_pop_context();
> - }
> -
> - for (i = 0; i < ARRAY_SIZE(invalid_reg_props); i++)
> - {
> -
>    const struct reg_property *prop = &invalid_reg_props[i];
> -
>    DEVPROPKEY key;
> -
>    BYTE buf[80];

buf isn't being used in this loop :)