Hi Akim. I'm finding bench.pl to be quite useful. :)
I'd like to push this patch to master, branch-2.5, and maybe even branch-2.4.2 in case it might be around a little longer. Any objections? >From c93d4b165d865028f3f06e13bcc36d8bd98f19df Mon Sep 17 00:00:00 2001 From: Joel E. Denny <[email protected]> Date: Sun, 13 Sep 2009 16:11:12 -0400 Subject: [PATCH] benchmarks: do not activate %debug by default. * etc/bench.pl.in (generate_grammar_triangular): Here. It can affect the timings even if yydebug=0. --- ChangeLog | 6 ++++++ etc/bench.pl.in | 3 ++- 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index f99cc59..50ac4dd 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2009-09-13 Joel E. Denny <[email protected]> + benchmarks: do not activate %debug by default. + * etc/bench.pl.in (generate_grammar_triangular): Here. It can + affect the timings even if yydebug=0. + +2009-09-13 Joel E. Denny <[email protected]> + tests: clean up push.at test group titles. * tests/push.at: Remove "Push Parsing: " from test group titles because these are already under the banner "Push Parsing Tests". diff --git a/etc/bench.pl.in b/etc/bench.pl.in index 2290fd2..623583c 100755 --- a/etc/bench.pl.in +++ b/etc/bench.pl.in @@ -246,7 +246,6 @@ sub generate_grammar_triangular ($$@) or die; print $out <<EOF; %error-verbose -%debug %{ #include <stdio.h> #include <stdlib.h> @@ -318,7 +317,9 @@ yyerror (const char *msg) int main (void) { +#if YYDEBUG yydebug = !!getenv ("YYDEBUG"); +#endif return yyparse (); } EOF -- 1.5.4.3
