[PATCH 1/3] browseui: Add IOleWindow to IProgressDialog

Michael Stefaniuc mstefani at redhat.com
Sun Apr 15 15:17:19 CDT 2012


Hello Detlef,

On 04/15/2012 10:08 PM, Detlef Riekenberg wrote:
> Used by apps to adjust the dialog position
> or remove the cancel button before vista
> 
> --
> By by ... Detlef
> ---
>  dlls/browseui/progressdlg.c |   69 ++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 68 insertions(+), 1 deletions(-)
> 
> diff --git a/dlls/browseui/progressdlg.c b/dlls/browseui/progressdlg.c
> index 9b970e2..2807622 100644
> --- a/dlls/browseui/progressdlg.c
> +++ b/dlls/browseui/progressdlg.c
> @@ -270,9 +276,20 @@ static HRESULT WINAPI ProgressDialog_QueryInterface(IProgressDialog *iface, REFI
>      ProgressDialog *This = impl_from_IProgressDialog(iface);
>      *ppvOut = NULL;
>  
> -    if (IsEqualIID(iid, &IID_IUnknown) || IsEqualIID(iid, &IID_IProgressDialog))
> +    if (IsEqualIID(iid, &IID_IUnknown))
>      {
>          *ppvOut = This;
> +        TRACE("(%p, IID_IUnknown, %p) -> %p\n", This, ppvOut, This);
> +    }
> +    else if (IsEqualIID(iid, &IID_IProgressDialog))
> +    {
> +        *ppvOut = This;
this is not correct as ppvOut takes an interface and not an object.
While at it please fix the above one too.

thanks
bye
	michael



More information about the wine-devel mailing list