<div dir="ltr">On 19 December 2012 09:13, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Dec 19, 2012 at 8:42 AM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>
> This patch adds logic to allow the ast_to_hir function<br>
> apply_type_qualifier_to_variable() to tell whether it is acting on a<br>
> variable declaration or a function parameter.  This will allow it to<br>
> correctly interpret the meaning of "out" and "in" keywords (which have<br>
> different meanings in those two contexts).<br>
> ---<br>
>  src/glsl/ast_to_hir.cpp | 7 ++++---<br>
>  1 file changed, 4 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp<br>
> index 94b63f6..a14fe7d 100644<br>
> --- a/src/glsl/ast_to_hir.cpp<br>
> +++ b/src/glsl/ast_to_hir.cpp<br>
> @@ -1915,7 +1915,8 @@ apply_type_qualifier_to_variable(const struct ast_type_qualifier *qual,<br>
>                                  ir_variable *var,<br>
>                                  struct _mesa_glsl_parse_state *state,<br>
>                                  YYLTYPE *loc,<br>
> -                                bool ubo_qualifiers_valid)<br>
> +                                bool ubo_qualifiers_valid,<br>
> +                                 bool is_parameter)<br>
<br>
</div>Whitespace might be a little off here.<br>
</blockquote></div><br></div><div class="gmail_extra">It's actually fine--this is another case of the old "tabs vs spaces look different in diffs" problem.<br></div></div>