Re: URLMON: Added ReleaseBindInfo implementation

[Robert Shearman]

Jacek Caban wrote:
> Index: dlls/urlmon/urlmon_main.c
>
> RCS file: /home/wine/wine/dlls/urlmon/urlmon_main.c,v
> retrieving revision 1.32
> diff -u -p -r1.32 urlmon_main.c
> --- dlls/urlmon/urlmon_main.c	6 Sep 2005 09:27:04 -0000	1.32
> +++ dlls/urlmon/urlmon_main.c	7 Sep 2005 16:44:48 -0000
> @@ -362,3 +362,27 @@ HRESULT WINAPI CoGetClassObjectFromURL( 
>    debugstr_guid(riid), ppv);
>     return E_NOINTERFACE;
> }
> -
>
> +/***********************************************************************
> -
> -
>       ReleaseBindInfo (URLMON.@)
> -
> -
> -
> - Release the resources used by the specified BINDINFO structure.
> -
> -
> -
> - PARAMS
> -
> - pbindinfo [I] BINDINFO to release.
> -
> -
> -
> - RETURNS
> -
> - Nothing.
> - */
>
> +void WINAPI ReleaseBindInfo(BINDINFO* pbindinfo)
> +{
> - TRACE("(%p)\n", pbindinfo);
> -
> - if(!pbindinfo)
> -
>    return;
> -
> - if(pbindinfo->szExtraInfo)
> -
>    CoTaskMemFree(pbindinfo->szExtraInfo);

There is no need for the NULL check above.
> - if(pbindinfo->pUnk)
> -
>    IUnknown_Release(pbindinfo->pUnk);
>
> +}