Re: [PATCH v7 0/6] MR3548: windows.applicationmodel: Implement some functions.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.applicationmodel/tests/model.c:
>  hr = IActivationFactory_QueryInterface( factory, &IID_IPackageStatics, (void **)&package_statics );
>  ok( hr == S_OK, "got hr %#lx.\n", hr );
>
> - hr = IPackageStatics_get_Current( package_statics, NULL );
> - ok( hr == E_INVALIDARG, "got hr %#lx.\n", hr );
> - hr = IPackageStatics_get_Current( package_statics, &package );
> - ok( hr == 0x80073d54, "got hr %#lx.\n", hr );
> - ok( !package, "got package %p.\n", package );
> - if (package) IPackage_Release( package );

The check before asserts that it's NULL, you don't need that.