[Piglit] [PATCH] glean: Fix indentation.

Dylan Baker dylan at pnwbakers.com
Thu Oct 13 22:02:07 UTC 2016


Quoting Vinson Lee (2016-10-13 14:47:35)
> dsurf.cpp: In constructor 'GLEAN::Window::Window(GLEAN::WindowSystem&, GLEAN::DrawingSurfaceConfig&, int, int, int, int)':
> dsurf.cpp:153:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
>      if (!hWindow)
>      ^~
> dsurf.cpp:156:2: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
>   hDC = GetDC(hWindow);
>   ^~~
> 
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  tests/glean/dsurf.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/glean/dsurf.cpp b/tests/glean/dsurf.cpp
> index 3683db1..e371084 100644
> --- a/tests/glean/dsurf.cpp
> +++ b/tests/glean/dsurf.cpp
> @@ -150,8 +150,8 @@ legacyMethod:
>                 GetModuleHandle(NULL),
>                 NULL);
>  
> -    if (!hWindow)
> -        return;
> +       if (!hWindow)
> +               return;
>  
>         hDC = GetDC(hWindow);
>  
> -- 
> 2.10.1
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit

Reviewed-by: Dylan Baker <dylan at pnwbakers.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 455 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20161013/2520209b/attachment.sig>


More information about the Piglit mailing list