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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8459#note_109919