Re: minor regex speedup

Eric Blake <[email protected]>
Newsgroups gmane.comp.gnu.m4.patches
Message-ID <[email protected]>
Eric Blake <ebb9 <at> byu.net> writes:

>  /* V: very verbose --  print everything */
> -#define DEBUG_TRACE_VERBOSE 1023
> +#define DEBUG_TRACE_VERBOSE 0x377

What WAS I thinking?  Oh well.  At least the master branch is immune, and at 
least I didn't cherry-pick this into branch-1.4 before cutting 1.4.11, so the 
damage is only limited to 1.4.10b.

From: Eric Blake <[email protected]>
Date: Thu, 24 Apr 2008 13:41:36 -0600
Subject: [PATCH] Fix regression from 2008-02-18.

* src/m4.h (DEBUG_TRACE_VERBOSE): Make -dV verbose again.
* NEWS: Document this fix.

Signed-off-by: Eric Blake <[email protected]>
---
 ChangeLog |    6 ++++++
 NEWS      |    4 ++++
 src/m4.h  |    2 +-
 3 files changed, 11 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 8a29282..6033f82 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2008-04-24  Eric Blake  <[email protected]>
+
+	Fix regression from 2008-02-18.
+	* src/m4.h (DEBUG_TRACE_VERBOSE): Make -dV verbose again.
+	* NEWS: Document this fix.
+
 2008-04-21  Eric Blake  <[email protected]>
 
 	Fix spelling of attribution to Christopher Strachey.
diff --git a/NEWS b/NEWS
index 58fddef..0ca3094 100644
--- a/NEWS
+++ b/NEWS
@@ -9,6 +9,10 @@ Foundation, Inc.
    using `builtin' or `indir' to perform nested `shift' calls triggered an
    assertion failure.
 
+** Fix regression introduced in 1.4.10b (but not present in 1.4.11) where
+   the command-line option -dV, as well as the builtin `debugmode(V)',
+   failed to enable `t' and `c' debug options.
+
 ** Fix the `m4wrap' builtin to accumulate wrapped text in FIFO order, as
    required by POSIX.  The manual mentions a way to restore the LIFO order
    present in earlier GNU M4 versions.  NOTE: this change exposes a bug
diff --git a/src/m4.h b/src/m4.h
index db531a3..59d9be3 100644
--- a/src/m4.h
+++ b/src/m4.h
@@ -183,7 +183,7 @@ extern FILE *debug;
 #define DEBUG_TRACE_CALLID 0x200
 
 /* V: very verbose --  print everything */
-#define DEBUG_TRACE_VERBOSE 0x377
+#define DEBUG_TRACE_VERBOSE 0x3FF
 /* default flags -- equiv: aeq */
 #define DEBUG_TRACE_DEFAULT 0x007
 
-- 
1.5.5.1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.