On 8 April 2013 10:23, Ian Romanick <i...@freedesktop.org> wrote: > On 04/06/2013 07:20 PM, Paul Berry wrote: > >> This patch updates the interp[] array to match the enum >> glsl_interp_qualifier. >> > > Can we use STATIC_ASSERT to make sure these arrays are at least the > correct size?
Sure, that seems reasonable. I'll do a similar thing for the mode[] array. > > > --- >> src/glsl/ir_print_visitor.cpp | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/glsl/ir_print_visitor.**cpp b/src/glsl/ir_print_visitor. >> **cpp >> index 597d281..8b445dc 100644 >> --- a/src/glsl/ir_print_visitor.**cpp >> +++ b/src/glsl/ir_print_visitor.**cpp >> @@ -149,7 +149,7 @@ void ir_print_visitor::visit(ir_**variable *ir) >> const char *const mode[] = { "", "uniform ", "shader_in ", >> "shader_out ", >> "in ", "out ", "inout ", >> "const_in ", "sys ", "temporary " }; >> - const char *const interp[] = { "", "flat", "noperspective" }; >> + const char *const interp[] = { "", "smooth", "flat", "noperspective" >> }; >> >> printf("(%s%s%s%s) ", >> cent, inv, mode[ir->mode], interp[ir->interpolation]); >> >> >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev