Re: [PATCH v2 0/5] MR6260: mshtml: Misc improvements and fixes.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmllocation.c:
> +static BOOL is_default_port(URL_COMPONENTSW *url)
> +{
> - INTERNET_PORT def;
> -
> - switch (url->nScheme)
> - {
> - case INTERNET_SCHEME_HTTP:      def = INTERNET_DEFAULT_HTTP_PORT;       break;
> - case INTERNET_SCHEME_HTTPS:     def = INTERNET_DEFAULT_HTTPS_PORT;      break;
> - case INTERNET_SCHEME_FTP:       def = INTERNET_DEFAULT_FTP_PORT;        break;
> - case INTERNET_SCHEME_SOCKS:     def = INTERNET_DEFAULT_SOCKS_PORT;      break;
> - case INTERNET_SCHEME_GOPHER:    def = INTERNET_DEFAULT_GOPHER_PORT;     break;
> - default:                        return FALSE;
> - }
> -
> - return (url->nPort == def);
>
> +}

We already have logic for that in `IUri` implementation, I think it would be better to try harder to use `IUri` for `get_host` instead of duplicating it.