Re: glr.c cleanup

[Paul Hilfinger]

> >>    (yyglrShift): Change parameter YYLTYPE* yylocp to YYLTYPE yyloc for
> >>    consistency with parameter YYSTYPE yysval.
> >
> > I used yylocp here because the default location object is a multiword
> > thing, and I wanted to avoid passing its value.
> 
> Minor issue, but what about YYSTYPE* yyvalp instead?  This would be fairly 
> consistent with other functions.

You could do that if you wanted.  These are parameters that would be
const Foo& in C++, to avoid gratuitous invocation of a copy
constructor. 

Paul Hilfinger