Re: [PATCH 3/4] mshtml: Add helper function get_url_components

[Andrew Eikum]

Detlef Riekenberg wrote:
> On Fr, 2009-09-25 at 15:30 -0500, Andrew Eikum wrote:
> > +static HRESULT get_url_components(HTMLLocation *This, URL_COMPONENTSW
> > *url)
> > +{
> > - const WCHAR *doc_url;
> > - if(!InternetCrackUrlW(doc_url, 0, 0, url)) {
> > -
> >    FIXME("InternetCrackUrlW failed\n");
>
> Such a FIXME does not help to find the reason.
> Dumping doc_url helps, but might add personal data to the log

Thanks for the comments.  I've resent the patches with most of your 
corrections added.

Andrew