Make the lexer reentrant (to avoid "still reachable" memory).

This allows the final program to be 100% "valgrind clean", (freeing
all memory that it allocates). This will make it much easier to ensure
that any allocation that parser actions perform are also cleaned up.

diff --git a/glcpp-lex.l b/glcpp-lex.l
index 9779f2b..276f50d 100644 (file)
--- a/glcpp-lex.l
+++ b/glcpp-lex.l
@@ -28,7 +28,7 @@
 #include "glcpp-parse.h"
 %}
 
-%option noyywrap
+%option reentrant noyywrap
 
 %%
 

diff --git a/glcpp-parse.y b/glcpp-parse.y
index 739b293..9acd549 100644 (file)
--- a/glcpp-parse.y
+++ b/glcpp-parse.y
@@ -28,10 +28,13 @@
 #define YYSTYPE int
 
 void
-yyerror (const char *error);
+yyerror (const char *error, void *scanner);
 
 %}
 
+%parse-param {void *scanner}
+%lex-param {void *scanner}
+
 %token TOKEN
 
 %%
@@ -51,7 +54,7 @@ token:                TOKEN
 %%
 
 void
-yyerror (const char *error)
+yyerror (const char *error, void *scanner)
 {
        fprintf (stderr, "Parse error: %s\n", error);
 }

diff --git a/glcpp.c b/glcpp.c
index 09641ce..90a0e89 100644 (file)
--- a/glcpp.c
+++ b/glcpp.c
@@ -24,5 +24,12 @@
 int
 main (void)
 {
-       return yyparse ();
+       int ret;
+       void *scanner;
+
+       yylex_init (&scanner);
+       ret = yyparse (scanner);
+       yylex_destroy (scanner);
+
+       return ret;
 }