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

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.enumeration/tests/devices.c:
> +                hr = IVectorView_DeviceInformation_GetAt( info_collection, idx, &info);
> +                ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> +                if (SUCCEEDED( hr ))
> +                {
> +                    hr = IDeviceInformation_get_Id( info, &id2 );
> +                    ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> +                }
> +                if (id1 && id2)
> +                {
> +                    INT32 order = 1;
> +                    WindowsCompareStringOrdinal( id1, id2, &order );
> +                    ok( !order, "%s != %s\n", debugstr_hstring( id1 ), debugstr_hstring( id2 ) );
> +                }
> +                WindowsDeleteString( id1 );
> +                WindowsDeleteString( id2 );
> +                if (info)
> This also doesn't look useful, info shouldn't be NULL unless tests have failed already.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6874#note_102556