changeset 1724:4af49f9a7a5c

[xemacs-hg @ 2003-09-29 09:20:39 by stephent] fix a{4,4} 2003-09-22 Stephen J. Turnbull <stephen@xemacs.org> * Makefile.in.in (XE_CFLAGS): New variant on CFLAGS for XEMACS_CC. 2003-08-14 Stephen J. Turnbull <stephen@xemacs.org> * mule-charset.c (get_unallocated_leading_byte): Compiler quietus.
author stephent
date Mon, 29 Sep 2003 09:20:41 +0000
parents b7290aecae9d
children 7ff8f4d70aec
files src/ChangeLog src/search.c
diffstat 2 files changed, 10 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Sun Sep 28 21:52:27 2003 +0000
+++ b/src/ChangeLog	Mon Sep 29 09:20:41 2003 +0000
@@ -1,3 +1,9 @@
+2003-09-27  Stephen J. Turnbull  <stephen@xemacs.org>
+
+	* search.c (trivial_regexp_p): Regexps containing "\{" are
+	nontrivial.  Thanks to Manfred Bartz for reporting the bug and
+	Stefan Monnier for suggesting a diagnosis.
+
 2003-09-26  Steve Youngs  <youngs@xemacs.org>
 
 	* XEmacs 21.5.16 "celeriac" is released.
--- a/src/search.c	Sun Sep 28 21:52:27 2003 +0000
+++ b/src/search.c	Mon Sep 29 09:20:41 2003 +0000
@@ -1192,6 +1192,7 @@
     {
       switch (*s++)
 	{
+	/* #### howcum ']' doesn't appear here, but ... */
 	case '.': case '*': case '+': case '?': case '[': case '^': case '$':
 	  return 0;
 	case '\\':
@@ -1199,9 +1200,11 @@
 	    return 0;
 	  switch (*s++)
 	    {
+	    /* ... ')' does appear here?  ('<' and '>' can appear singly.) */
+	    /* #### are there other constructs to check? */
 	    case '|': case '(': case ')': case '`': case '\'': case 'b':
 	    case 'B': case '<': case '>': case 'w': case 'W': case 's':
-	    case 'S': case '=':
+	    case 'S': case '=': case '{': case '}':
 #ifdef MULE
 	    /* 97/2/25 jhod Added for category matches */
 	    case 'c': case 'C':