Re: [PATCH 0/4] MR3983: windows.storage.applicationdata: Implement IApplicationDataStatics::get_Current().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.applicationmodel/tests/application.c:
> -
>    win_skip( "%s runtimeclass not registered, skipping tests.\n", wine_dbgstr_w( application_data_statics_name ) );
> -
>    return;
> - }
> -
> - check_interface( factory, &IID_IUnknown );
> - check_interface( factory, &IID_IInspectable );
> - check_interface( factory, &IID_IAgileObject );
> -
> - hr = IActivationFactory_QueryInterface( factory, &IID_IApplicationDataStatics, (void **)&application_data_statics );
> - ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
> - hr = IApplicationDataStatics_get_Current( application_data_statics, NULL );
> - todo_wine ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> - hr = IApplicationDataStatics_get_Current( application_data_statics, &application_data );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> - todo_wine ok( application_data != NULL, "got NULL application_data %p.\n", application_data );

At this point, you're leaking application_data on Windows. You should add the Release from the next commit there, with a if for Wine (though Wine doesn't really reach this point for the moment).