From c52419107a8e1e97505b2460fbb61a09a0fd086b Mon Sep 17 00:00:00 2001 From: "Richard M. Stallman" Date: Wed, 10 Nov 1993 08:29:34 +0000 Subject: [PATCH] (skip_chars): Finish reenabling checks for buffer bounds. Use XINT, not XFASTINT, when testing. --- src/search.c | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) 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; -- 2.39.5