Re: [PATCH 0/4] MR6733: Mfplat HTTP byte stream implementation

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfplat/network.c:
> - }
> -
> - LeaveCriticalSection(&object->cs);
> -
> - *out = &object->IMFByteStream_iface;
> - return S_OK;
> -
>
> +error:
> - LeaveCriticalSection(&object->cs);
> - IMFByteStream_Release(&object->IMFByteStream_iface);
> -
> - if (hr == NS_E_SERVER_NOT_FOUND && wcsncmp(url, L"http:", 5))
> -
>    hr = WININET_E_NAME_NOT_RESOLVED;
> - else if (!wcsncmp(url, L"mms:", 4))
> -
>    hr = MF_E_UNSUPPORTED_BYTESTREAM_TYPE;
> - return hr;

This suggest that wininet is involved then? Also, why only check for mms: or for any scheme at all? I think it should be registered in scheme handler section, so if we don't need one for mms, maybe it should not call http one?