skeletons: simplify after knr removal

[bison.git] / TODO

diff --git a/TODO b/TODO
index d86d8d8b588bd588d952900f5af80f12ded78acb..ea878c9ead8dff06fcb575410327642b9c6081b6 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,13 +1,30 @@
 * Short term
 * Short term

-** Variable names.
-What should we name `variant' and `lex_symbol'?
+** scan-code.l
+Avoid variables for format strings, as then GCC cannot check them.
+show_sub_messages should call show_sub_message.
+
+** m4 names
+b4_shared_declarations is no longer what it is.  Make it
+b4_parser_declaration for instance.
+
+** glr.cc: %defines
+it should not be mandatory.

 
 

-** Use b4_symbol in all the skeleton
-Move its definition in the more standard places and deploy it in other
-skeletons.  Then remove the older system, including the tables
-generated by output.c
+** $ and others in epilogue
+A stray $ is a warning in the actions, but an error in the epilogue.
+IMHO, it should not even be a warning in the epilogue.

 
 

-** Update the documentation on gnu.org
+** obstack_copy etc.
+There seems to be some other interesting functions for obstacks that
+we should consider using.
+
+** stack.hh
+Get rid of it.  The original idea is nice, but actually it makes
+the code harder to follow, and uselessly different from the other
+skeletons.
+
+** Variable names.
+What should we name `variant' and `lex_symbol'?

 
 ** Get rid of fake #lines [Bison: ...]
 Possibly as simple as checking whether the column number is nonnegative.
 
 ** Get rid of fake #lines [Bison: ...]
 Possibly as simple as checking whether the column number is nonnegative.


@@ -39,10 +56,6 @@ as lr0.cc, why upper case?
 Enhance bench.pl with %b to run different bisons.
 
 * Various
 Enhance bench.pl with %b to run different bisons.
 
 * Various

-** Warnings
-Warnings about type tags that are used in printer and dtors, but not
-for symbols?
-

 ** YYERRCODE
 Defined to 256, but not used, not documented.  Probably the token
 number for the error token, which POSIX wants to be 256, but which
 ** YYERRCODE
 Defined to 256, but not used, not documented.  Probably the token
 number for the error token, which POSIX wants to be 256, but which


@@ -88,9 +101,6 @@ so both 256 and 257 are "mysterious".
   "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
 
 
   "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
 
 

-** YYFAIL
-It is seems to be *really* obsolete now, shall we remove it?
-

 ** yychar == yyempty_
 The code in yyerrlab reads:
 
 ** yychar == yyempty_
 The code in yyerrlab reads:
 


@@ -125,16 +135,6 @@ we do the same in yacc.c.
 The code bw glr.c and yacc.c is really alike, we can certainly factor
 some parts.
 
 The code bw glr.c and yacc.c is really alike, we can certainly factor
 some parts.
 

-* Header guards
-
-From François: should we keep the directory part in the CPP guard?
-
-
-* Yacc.c: CPP Macros
-
-Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite?
-They should not: it is not documented.  But if they need to, let's
-find something clean (not like YYLSP_NEEDED...).

 
 * Report
 
 
 * Report