Re: [PATCH 0/5] MR7997: windows.device.enumeration: Implement initial device enumeration for DeviceWatcher.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.enumeration/tests/devices.c:
>  IDeviceWatcher_Release( device_watcher );
>
> - hr = IDeviceInformationStatics_CreateWatcher( device_info_statics, &device_watcher );
> - todo_wine ok( hr == S_OK, "got hr %#lx\n", hr );
> -
> - if (device_watcher)
> - {
> -
>    added_handler.test_deviceinformation = 1;
> -
>    hr = IDeviceWatcher_add_Added( device_watcher, (void *)&added_handler.ITypedEventHandler_DeviceWatcher_IInspectable_iface, &added_token );
> -
>    ok( hr == S_OK, "got hr %#lx\n", hr );
> -
>    hr = IDeviceWatcher_add_EnumerationCompleted( device_watcher, (void *)&enumerated_handler.ITypedEventHandler_DeviceWatcher_IInspectable_iface, &enumerated_token );
> -
>    todo_wine ok( hr == S_OK, "got hr %#lx\n", hr );
> -
> -
>    ref = IDeviceWatcher_AddRef( device_watcher );
> -
>    ok( ref == 2, "got ref %lu\n", ref );

What is this checking exactly? If this is meant to show that weak ref are used, I don't think it really does that.