Re: [PATCH v28 0/7] MR6766: dlls/wintypes: Add stubs for Windows.Foundation.Collections.PropertySet.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/wintypes/tests/propertyset.c:
> - }
> -
> - hr = IPropertySet_QueryInterface( propset, &IID_IObservableMap_HSTRING_IInspectable,
> -
>                                  (void *)&observable_map );
> - IPropertySet_Release(propset);
> - todo_wine ok(SUCCEEDED(hr), "QueryInterface failed, got %#lx\n", hr);
> -
> - hr = IMap_HSTRING_IInspectable_QueryInterface(
> -
>    map, &IID_IIterable_IKeyValuePair_HSTRING_IInspectable, (void **)&iterable );
> - todo_wine ok( SUCCEEDED( hr ), "QueryInterface failed, got %#lx\n", hr );
> - if (SUCCEEDED( hr ))
> - {
> -
>    hr = IIterable_IKeyValuePair_HSTRING_IInspectable_First( iterable, &iterator );
> -
>    IIterable_IKeyValuePair_HSTRING_IInspectable_Release( iterable );
> -
>    ok( SUCCEEDED( hr ), "got %#lx\n", hr );
> -
>    IIterator_IKeyValuePair_HSTRING_IInspectable_Release( iterator );

Should you check if iterator is valid first?