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?