Re: [PATCH v2 0/6] MR716: windows.devices.enumeration: Add DeviceWatcher class and IDeviceInformationStatics2 interface.

[Bernhard Kölbl (＠besentv)]

Bernhard K��lbl (@besentv) commented about dlls/windows.devices.enumeration/tests/devices.c:
> - {
> -
>    win_skip("%s runtimeclass, not registered.\n", wine_dbgstr_w(device_info_name));
> -
>    return;
> - }
> -
> - hr = IActivationFactory_QueryInterface(factory, &IID_IInspectable, (void **)&inspectable);
> - ok(hr == S_OK, "got hr %#lx\n", hr);
> -
> - hr = IActivationFactory_QueryInterface(factory, &IID_IDeviceInformationStatics2, (void **)&device_info_statics2);
> - ok(hr == S_OK || broken(hr == E_NOINTERFACE), "got hr %#lx\n", hr);
> - IActivationFactory_Release(factory);
> - if (hr != S_OK)
> - {
> -
>    IInspectable_Release(inspectable);
> -
>    win_skip("IDeviceInformationStatics2 not supported.\n");
> -
>    return;

Instead of returning, I'd do a goto to a cleanup routine at the end of the function.