[Mesa-dev] [PATCH] glsl/parser: (trivial) fix indentation in one function

Matt Turner mattst88 at gmail.com
Wed Jan 21 13:22:10 PST 2015


On Wed, Jan 21, 2015 at 1:12 PM, Tobias Klausmann
<tobias.johannes.klausmann at mni.thm.de> wrote:
> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
> ---
>  src/glsl/glcpp/glcpp-parse.y | 19 ++++++++++---------
>  1 file changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y
> index e5bebe5..8b545da 100644
> --- a/src/glsl/glcpp/glcpp-parse.y
> +++ b/src/glsl/glcpp/glcpp-parse.y
> @@ -2401,8 +2401,8 @@ _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t versio
>                  add_builtin_define(parser, "GL_ARB_fragment_coord_conventions",
>                                     1);
>
> -              if (extensions->ARB_fragment_layer_viewport)
> -                 add_builtin_define(parser, "GL_ARB_fragment_layer_viewport", 1);
> +             if (extensions->ARB_fragment_layer_viewport)
> +                add_builtin_define(parser, "GL_ARB_fragment_layer_viewport", 1);
>
>               if (extensions->ARB_explicit_attrib_location)
>                  add_builtin_define(parser, "GL_ARB_explicit_attrib_location", 1);
> @@ -2449,7 +2449,8 @@ _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t versio
>                  add_builtin_define(parser, "GL_AMD_vertex_shader_layer", 1);
>
>               if (extensions->AMD_vertex_shader_viewport_index)
> -                add_builtin_define(parser, "GL_AMD_vertex_shader_viewport_index", 1);
> +                add_builtin_define(parser, "GL_AMD_vertex_shader_viewport_index",
> +                   1);
>
>               if (extensions->ARB_shading_language_420pack)
>                  add_builtin_define(parser, "GL_ARB_shading_language_420pack", 1);
> @@ -2466,17 +2467,17 @@ _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t versio
>               if (extensions->ARB_viewport_array)
>                  add_builtin_define(parser, "GL_ARB_viewport_array", 1);
>
> -              if (extensions->ARB_compute_shader)
> -                 add_builtin_define(parser, "GL_ARB_compute_shader", 1);
> +             if (extensions->ARB_compute_shader)
> +             add_builtin_define(parser, "GL_ARB_compute_shader", 1);

We're messing up the indentation here.

But really we want to get rid of tabs, not add more. glcpp is
unfortunately full of tabs.


More information about the mesa-dev mailing list