Paul Vriens <[EMAIL PROTECTED]> writes:

> +    SetLastError(0xdeadbeef);
> +    bAppThemed = pIsAppThemed();
> +    todo_wine
> +    {
> +        ok( bAppThemed == FALSE, "Expected FALSE as this test executable is 
> not (yet) themed.\n");
> +        ok( GetLastError() == S_OK,
> +            "Expected S_OK, got 0x%08lx\n",
> +            GetLastError());
> +    }

This won't work right if theming is disabled, since IsAppThemed
returns FALSE in that case. Also note that GetLastError doesn't return
an HRESULT, so ERROR_SUCCESS is a better name than S_OK.

-- 
Alexandre Julliard
[EMAIL PROTECTED]


Reply via email to