Re: [PATCH v4 0/8] MR6874: windows.devices.enumeration: Add a basic implementation for DeviceInformationStatics::FindAllAsync().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.enumeration/tests/devices.c:
> -
>    todo_wine check_deviceinformationcollection_async( infocollection_async, 1, Completed, S_OK, &info_collection );
> -
>    IAsyncOperation_DeviceInformationCollection_Release( infocollection_async );
> - }
> - if (info_collection)
> - {
> -
>    UINT32 idx = 0, size = 0;
> -
> -
>    hr = IVectorView_DeviceInformation_get_Size( info_collection, &size );
> -
>    todo_wine ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> -
>    for (idx = 0; idx < size ;idx++)
> -
>    {
> -
>        IDeviceInformation *info;
> -
>        winetest_push_context("info_collection %u", idx);
> -
>        hr = IVectorView_DeviceInformation_GetAt( info_collection, idx, &info );
> -
>        ok( SUCCEEDED( hr ), "got %#lx\n", hr);
> -
>        if (SUCCEEDED( hr ))

The `if` doesn't look necessary here after the test above, similar comment in multiple other places.