From: Richard M. Stallman <rms@gnu.org>
Date: Wed, 10 Nov 1993 08:29:34 +0000 (+0000)
Subject: (skip_chars): Finish reenabling checks for buffer bounds.
X-Git-Tag: emacs-19.34~10900
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=c52419107a8e1e97505b2460fbb61a09a0fd086b;p=emacs.git

(skip_chars): Finish reenabling checks for buffer bounds.
Use XINT, not XFASTINT, when testing.
---

diff --git a/src/search.c b/src/search.c
index 8a821837528..a1dc6769d6f 100644
--- a/src/search.c
+++ b/src/search.c
@@ -417,19 +417,13 @@ skip_chars (forwardp, syntaxp, string, lim)
     CHECK_NUMBER_COERCE_MARKER (lim, 1);
 
   /* In any case, don't allow scan outside bounds of buffer.  */
-
-  /* I don't know what things this breaks, and there is no entry in the
-     ChangeLog, so I reinstated the end of buffer limit check.  This code
-     breaks without it. (bfox) */
-/* #if 0				/* This breaks some things... jla. */
-#if 1
-  if (XFASTINT (lim) > ZV)
+  /* jla turned this off, for no known reason.
+     bfox turned the ZV part on, and rms turned the
+     BEGV part back on.  */
+  if (XINT (lim) > ZV)
     XFASTINT (lim) = ZV;
-#endif
-#if 0
-  if (XFASTINT (lim) < BEGV)
+  if (XINT (lim) < BEGV)
     XFASTINT (lim) = BEGV;
-#endif
 
   p = XSTRING (string)->data;
   pend = p + XSTRING (string)->size;