Matt Turner <matts...@gmail.com> writes: > On Wed, Dec 19, 2012 at 9:41 AM, Paul Berry <stereotype...@gmail.com> wrote: >> On 19 December 2012 09:13, Matt Turner <matts...@gmail.com> wrote: >>> >>> On Wed, Dec 19, 2012 at 8:42 AM, Paul Berry <stereotype...@gmail.com> >>> wrote: >>> > This patch adds logic to allow the ast_to_hir function >>> > apply_type_qualifier_to_variable() to tell whether it is acting on a >>> > variable declaration or a function parameter. This will allow it to >>> > correctly interpret the meaning of "out" and "in" keywords (which have >>> > different meanings in those two contexts). >>> > --- >>> > src/glsl/ast_to_hir.cpp | 7 ++++--- >>> > 1 file changed, 4 insertions(+), 3 deletions(-) >>> > >>> > diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp >>> > index 94b63f6..a14fe7d 100644 >>> > --- a/src/glsl/ast_to_hir.cpp >>> > +++ b/src/glsl/ast_to_hir.cpp >>> > @@ -1915,7 +1915,8 @@ apply_type_qualifier_to_variable(const struct >>> > ast_type_qualifier *qual, >>> > ir_variable *var, >>> > struct _mesa_glsl_parse_state *state, >>> > YYLTYPE *loc, >>> > - bool ubo_qualifiers_valid) >>> > + bool ubo_qualifiers_valid, >>> > + bool is_parameter) >>> >>> Whitespace might be a little off here. >> >> >> It's actually fine--this is another case of the old "tabs vs spaces look >> different in diffs" problem. > > The problem seems to be that the whitespace before > ubo_qualifiers_valid is tabs, and the whitespace before is_parameter > is spaces. I don't think that's intentional?
Basically, the tabs are my fault and Ian's (I think) from before we picked up the style that had been set down in 2006, and it's accepted for new code to follow that style even when the surrounding code doesn't follow it.
pgpdSxFmDetpj.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
