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:
> -
>        "%lu) Unexpected IsDirty result before Save: 0x%08lx, expected result: 0x%08lx\n",
> -
>        i, hres, S_OK);
> -
> -
>    if (test.save.name)
> -
>    {
> -
>        wsprintfW(tmp_file_path, L"%s\\%s", tmp_dir, test.save.name);
> -
>        file_path = tmp_file_path;
> -
>    }
> -
>    else
> -
>        file_path = NULL;
> -
> -
>    if (file_path)
> -
>    {
> -
>        hfile = CreateFileW(file_path, GENERIC_READ | GENERIC_WRITE,
> -
>            FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
> -
>        if (hfile == INVALID_HANDLE_VALUE)

This could be `ok()` too.