> >> (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
