diff options
author | Akim Demaille <akim.demaille@gmail.com> | 2020-12-06 07:30:51 +0100 |
---|---|---|
committer | Akim Demaille <akim.demaille@gmail.com> | 2020-12-06 14:02:38 +0100 |
commit | e72eda7aee59b8e7aa43da7d51529b6082d8ef76 (patch) | |
tree | da1788feb81f961d502f4467b51b765664568c4f /TODO | |
parent | 4f24f5f304e44acc6d25c9da8be6ff4052e0528b (diff) | |
download | bison-e72eda7aee59b8e7aa43da7d51529b6082d8ef76.tar.gz |
glr2.cc: turn some pointers into references
* data/skeletons/glr2.cc: Prefer references to pointers.
Add a few more const.
Diffstat (limited to 'TODO')
-rw-r--r-- | TODO | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -187,6 +187,12 @@ And use YYCDEBUG rather than YY_DEBUG_STREAM. *** Avoid pointers There are many places where pointers should be replaced with references. +Some occurrences were fixed, but now some have improper names: + +-yygetToken (int *yycharp, ]b4_namespace_ref[::]b4_parser_class[& yyparser][]b4_pure_if([, glr_stack* yystackp])[]b4_user_formals[) ++yygetToken (int& yycharp, ]b4_namespace_ref[::]b4_parser_class[& yyparser][]b4_pure_if([, glr_stack* yystackp])[]b4_user_formals[) + +yycharp is no longer a Pointer. And yystackp should probably also be a reference. *** Use proper type names Don't use YYSTYPE and YYLTYPE but parser::semantic_type and |