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:
> - {
> -
>    skip("GetTempPathW failed with: 0x%08lx\n", GetLastError());
> -
>    return;
> - }
> -
> - lstrcatW(tmp_dir, sub_dir);
> -
> - if (!CreateDirectoryW(tmp_dir, NULL))
> - {
> -
>    skip("CreateDirectoryW failed with: 0x%08lx\n", GetLastError());
> -
>    return;
> - }
> -
> - for (i = 0; i < ARRAY_SIZE(persist_file_tests); ++i) {
> -
>    file_name = NULL;
> -
>    file_path = NULL;

You initialize `file_path` later, there is no need to initialize it here. Also, I think it would be easier to follow if you moved `file_name` initialization closer to its usage.