Set all front-end %define defaults in one place.

* src/main.c (main): Move lr.keep_unreachable_states default...
* src/reader.c (reader): ... to here.

diff --git a/ChangeLog b/ChangeLog
index af426e5cd04229a7cb438d251f22e46286ab862f..9551184b81bf6c7866496330cf2b2b5983ae6ecb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-04-30  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+       Set all front-end %define defaults in one place.
+       * src/main.c (main): Move lr.keep_unreachable_states default...
+       * src/reader.c (reader): ... to here.
+

 2009-04-29  Joel E. Denny  <jdenny@ces.clemson.edu>
 
        Rename lr.default_reductions to lr.default-reductions.
 2009-04-29  Joel E. Denny  <jdenny@ces.clemson.edu>
 
        Rename lr.default_reductions to lr.default-reductions.

diff --git a/src/main.c b/src/main.c
index 6c93a4ef9d5ce0707096f7fc65964f939df3a9b6..5769693efaae24e19d0f208b5f3e8fe84d97f6fd 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -114,7 +114,6 @@ main (int argc, char *argv[])
      declarations.  */
   timevar_push (TV_CONFLICTS);
   conflicts_solve ();
      declarations.  */
   timevar_push (TV_CONFLICTS);
   conflicts_solve ();

-  muscle_percent_define_default ("lr.keep_unreachable_states", "false");

   if (!muscle_percent_define_flag_if ("lr.keep_unreachable_states"))
     {
       state_number *old_to_new = xnmalloc (nstates, sizeof *old_to_new);
   if (!muscle_percent_define_flag_if ("lr.keep_unreachable_states"))
     {
       state_number *old_to_new = xnmalloc (nstates, sizeof *old_to_new);

diff --git a/src/reader.c b/src/reader.c
index 76e379fad452b1522f82d1cbdac3f39b15b436dd..2f3cd8245910246b0897c29239b676da569ea994 100644 (file)
--- a/src/reader.c
+++ b/src/reader.c
@@ -555,9 +555,12 @@ reader (void)
   gram_scanner_initialize ();
   gram_parse ();
 
   gram_scanner_initialize ();
   gram_parse ();
 

-  /* IELR would be a better default, but LALR is historically the default.  */
+  /* Set front-end %define variable defaults.  */
+  muscle_percent_define_default ("lr.keep_unreachable_states", "false");

   {
     char *lr_type;
   {
     char *lr_type;

+    /* IELR would be a better default, but LALR is historically the
+       default.  */

     muscle_percent_define_default ("lr.type", "LALR");
     lr_type = muscle_percent_define_get ("lr.type");
     if (0 != strcmp (lr_type, "canonical LR"))
     muscle_percent_define_default ("lr.type", "LALR");
     lr_type = muscle_percent_define_get ("lr.type");
     if (0 != strcmp (lr_type, "canonical LR"))


@@ -567,7 +570,7 @@ reader (void)
     free (lr_type);
   }
 
     free (lr_type);
   }
 

-  /* Check front-end %define variable values.  */
+  /* Check front-end %define variables.  */

   {
     static char const * const values[] = {
       "lr.type", "LALR", "IELR", "canonical LR", NULL,
   {
     static char const * const values[] = {
       "lr.type", "LALR", "IELR", "canonical LR", NULL,