Re: [PATCH v8 0/2] MR8459: urlmon: Add PersistentZoneIdentifier semi-stub

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/sec_mgr.c:
> -
>    hres, HRESULT_FROM_WIN32(ERROR_NOT_FOUND));
> - ok(zone == URLZONE_UNTRUSTED,
> -
>    "Unexpected GetId default zone: 0x%08lx, expected zone: 0x%08x\n", zone, URLZONE_UNTRUSTED);
> -
> - hres = IPersistFile_GetClassID(persist_file, &clsid);
> - ok(hres == S_OK,
> -
>    "Unexpected GetClassId result: 0x%08lx, expected result: 0x%08lx\n",
> -
>    hres, S_OK);
> - ok(IsEqualCLSID(&clsid, &CLSID_PersistentZoneIdentifier),
> -
>    "Unexpected GetClassId id: %s, expected class id: %s\n",
> -
>    debugstr_guid(&clsid), debugstr_guid(&CLSID_PersistentZoneIdentifier));
> -
> - hres = IPersistFile_GetCurFile(persist_file, &file_name);
> - ok(hres == E_NOTIMPL,
> -
>    "Unexpected GetCurFile result: 0x%08lx, expected result: 0x%08lx\n", hres, E_NOTIMPL);
> - ok(!lstrcmpW(file_name, NULL),

This could be just `!file_name`, there is no need for `lstrcmpW` to check for NULL.