Re: [PATCH v4 0/4] MR3439: mf: Implement basic urlmon scheme handler.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
> +    static const WCHAR *urls[] =
> +    {
> +        L"http://test.winehq.org/tests/test.mp3",
> +        L"https://test.winehq.org/tests/test.mp3",
> +        L"httpd://test.winehq.org/tests/test.mp3",
> +        L"httpsd://test.winehq.org/tests/test.mp3",
> +        L"mms://test.winehq.org/tests/test.mp3",
> +    };
> +    static const WCHAR *expect_urls[] =
> +    {
> +        /* Win10 */ L"http://test.winehq.org:80/tests/test.mp3", /* Win8 */ L"http://test.winehq.org/tests/test.mp3",
> +        /* Win10 */ L"https://test.winehq.org:443/tests/test.mp3", /* Win8 */ L"https://test.winehq.org/tests/test.mp3",
> +        /* Win10 */ L"http://test.winehq.org:80/tests/test.mp3", /* Win8 */ L"http://test.winehq.org/tests/test.mp3",
> +        /* Win10 */ L"https://test.winehq.org:443/tests/test.mp3", /* Win8 */ L"https://test.winehq.org/tests/test.mp3",
> +        /* Win10 */ L"http://test.winehq.org:80/tests/test.mp3", /* Win8 */ L"http://test.winehq.org/tests/test.mp3",
> +    };
> Is it important to match those exactly? Maybe test for "http://test.winehq.org" part only?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3439#note_42998