comparison src/search.c @ 4951:be690aaa7194

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

4950:145c69e39666

    XSET (lim, Lisp_Int, forwardp ? ZV : BEGV);
  else
    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)
    XFASTINT (lim) = ZV;
#endif
#if 0
  if (XFASTINT (lim) < BEGV)
    XFASTINT (lim) = BEGV;
#endif

  p = XSTRING (string)->data;
  pend = p + XSTRING (string)->size;
  bzero (fastmap, sizeof fastmap);

4951:be690aaa7194

    XSET (lim, Lisp_Int, forwardp ? ZV : BEGV);
  else
    CHECK_NUMBER_COERCE_MARKER (lim, 1);

  /* In any case, don't allow scan outside bounds of buffer.  */
  /* 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;
  if (XINT (lim) < BEGV)
    XFASTINT (lim) = BEGV;

  p = XSTRING (string)->data;
  pend = p + XSTRING (string)->size;
  bzero (fastmap, sizeof fastmap);