winhttp: Avoid -Wmisleading-indentation warning in test_IWinHttpRequest_Invoke.
Gerald Pfeifer
gerald at pfeifer.com
Wed Aug 10 14:23:14 CDT 2016
In case anyone is tracking patches, this one can be remove from
that list since it was re-done and committed independently (after
not getting a response back in January):
commit 1895345df813dfb728b40a0f0f6f7f9ba89c6e4e
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Wed Aug 10 13:19:04 2016 +0300
winhttp/tests: Fix indentation warning on GCC 6.
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Hans Leidekker <hans at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
Gerald
On Sun, 10 Jan 2016, Gerald Pfeifer wrote:
> This is a bit unusual formatting, though I understand where it's coming
> from. Adding braces silences GCC 6's new -Wmisleading-indentation.
>
> Gerald
>
> Signed-off-by: Gerald Pfeifer <gerald at pfeifer.com>
> ---
> dlls/winhttp/tests/winhttp.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/dlls/winhttp/tests/winhttp.c b/dlls/winhttp/tests/winhttp.c
> index d676dd8..1408f35 100644
> --- a/dlls/winhttp/tests/winhttp.c
> +++ b/dlls/winhttp/tests/winhttp.c
> @@ -3820,8 +3820,9 @@ static void test_IWinHttpRequest_Invoke(void)
> ok(hr == DISP_E_UNKNOWNINTERFACE, "error %#x\n", hr);
>
> VariantInit(&ret);
> -if (0) /* crashes */
> +if (0) /* crashes */ {
> hr = IWinHttpRequest_Invoke(request, DISPID_HTTPREQUEST_OPTION, &IID_NULL, 0, DISPATCH_PROPERTYPUT, NULL, &ret, NULL, &err);
> +}
>
> params.cArgs = 1;
> hr = IWinHttpRequest_Invoke(request, DISPID_HTTPREQUEST_OPTION, &IID_NULL, 0, DISPATCH_PROPERTYPUT, ¶ms, &ret, NULL, &err);
>
More information about the wine-patches
mailing list