[Mesa-dev] [PATCH V3 2/8] glsl: Add arrays_of_arrays to yacc definition

Timothy Arceri t_arceri at yahoo.com.au
Thu Jan 23 01:31:15 PST 2014


On Thu, 2014-01-23 at 08:58 +0200, Pohjolainen, Topi wrote:
> On Wed, Jan 22, 2014 at 10:33:01PM +1100, Timothy Arceri wrote:
> > Signed-off-by: Timothy Arceri <t_arceri at yahoo.com.au>
> > ---
> >  src/glsl/glsl_parser.yy | 128 +++++++++++++++++++++---------------------------
> >  1 file changed, 56 insertions(+), 72 deletions(-)
> > 
> > diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
> > index 5451b76..2786e92 100644
> > --- a/src/glsl/glsl_parser.yy
> > +++ b/src/glsl/glsl_parser.yy
> > @@ -97,6 +97,7 @@ static bool match_layout_qualifier(const char *s1, const char *s2,
> >  
> >     ast_node *node;
> >     ast_type_specifier *type_specifier;
> > +   ast_array_specifier *array_specifier;
> 
> Isn't this undefined symbol at this point? I see that class
> "ast_array_specifier" is defined in the next patch of the series.

Yes patches 2 and 3 will be squashed when committed they are were just
spilt to try and make reviewing easier.




More information about the mesa-dev mailing list