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

[Dmitry Timoshkov]

"James Hawkins"  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");