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

[Rémi Bernon]

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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3548#note_46315