Re: [PATCH v2 0/1] MR8378: urlmon: Add PersistentZoneIdentifier COM object

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/zone_id.c:
> +        remember = FALSE;
> +    }
> +
> +    try = 1;
> +    for (;;)
> +    {
> +        hfile = CreateFileW(file_name, GENERIC_READ | GENERIC_WRITE, 0, NULL, disposition, 0, NULL);
> +        if (hfile != INVALID_HANDLE_VALUE) { break; }
> +
> +        if (try++ >= 3)
> +        {
> +            hres = HRESULT_FROM_WIN32(GetLastError());
> +            goto cleanup;
> +        }
> +        Sleep(100);
> +    }
> Why is it needed? Could we just return failure on the first attempt?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8378#note_107378