Re: shell32 patch 24

[Alexandre Julliard]

"Martin Fuchs" <martin-fuchs(a)gmx.net> writes:
> +	if (This->bDirty)
> +	    return S_OK;
> +
>   return NOERROR;

S_OK and NOERROR mean the same thing, so this doesn't look quite
right.

-- 
Alexandre Julliard
julliard(a)winehq.org

shell32 patch 24

[Martin Fuchs]

On 25.03.2004 01:00:25 Alexandre Julliard wrote:
> "Martin Fuchs" <martin-fuchs(a)gmx.net> writes:
> > +	if (This->bDirty)
> > +	    return S_OK;
> > +
> >   return NOERROR;
>
> S_OK and NOERROR mean the same thing, so this doesn't look quite
> right.

Sure. It should be S_FALSE instead of NOERROR.