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, &params, &ret, NULL, &err);
> 



More information about the wine-patches mailing list