Re: urlmon: Fix a test that fails on all platforms

[Dmitry Timoshkov]

"James Hawkins" <jhawkins(a)codeweavers.com> wrote:
> -        ok(IBindCtx_Release(bctx) == 0, "bctx should be destroyed here\n");
> +
> +        if(bindf & BINDF_ASYNCHRONOUS) {
> +            ok(IBindCtx_Release(bctx) == 1, "bctx should not be destroyed here\n");
> +            IBindCtx_Release(bctx); /* actually destroy it */
> +        }
> +        else
> +            ok(IBindCtx_Release(bctx) == 0, "bctx should be destroyed here\n");

Wouldn't it be better to change it this way:

if(bindf & BINDF_ASYNCHRONOUS)
    ok(IBindCtx_Release(bctx) == 1, "bctx should not be destroyed here\n");

ok(IBindCtx_Release(bctx) == 0, "bctx should be destroyed here\n");

-- 
Dmitry.