summaryrefslogtreecommitdiff
path: root/TODO
diff options
context:
space:
mode:
authorAkim Demaille <akim.demaille@gmail.com>2020-12-06 07:30:51 +0100
committerAkim Demaille <akim.demaille@gmail.com>2020-12-06 14:02:38 +0100
commite72eda7aee59b8e7aa43da7d51529b6082d8ef76 (patch)
treeda1788feb81f961d502f4467b51b765664568c4f /TODO
parent4f24f5f304e44acc6d25c9da8be6ff4052e0528b (diff)
downloadbison-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--TODO6
1 files changed, 6 insertions, 0 deletions
diff --git a/TODO b/TODO
index a7dc19c2..fa9cee52 100644
--- a/TODO
+++ b/TODO
@@ -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