diff src/data.c @ 5581:56144c8593a8

Mechanically change INT to FIXNUM in our sources. src/ChangeLog addition: 2011-10-09 Aidan Kehoe <kehoea@parhasard.net> [...] Mechanically change INT (where it refers to non-bignum Lisp integers) to FIXNUM in our sources. Done for the following functions, enums, and macros: Lisp_Type_Int_Even, Lisp_Type_Int_Odd, INT_GCBITS, INT_VALBITS, make_int(), INTP(), XINT(), CHECK_INT(), XREALINT(), INT_PLUS(), INT_MINUS(), EMACS_INT_MAX (to MOST_POSITIVE_FIXNUM), EMACS_INT_MIN (to MOST_NEGATIVE_FIXNUM), NUMBER_FITS_IN_AN_EMACS_INT() to NUMBER_FITS_IN_A_FIXNUM(), XFLOATINT, XCHAR_OR_INT, INT_OR_FLOAT. The EMACS_INT typedef was not changed, it does not describe non-bignum Lisp integers. Script that did the change available in http://mid.gmane.org/20067.17650.181273.12014@parhasard.net . modules/ChangeLog addition: 2011-10-09 Aidan Kehoe <kehoea@parhasard.net> [...] Mechanically change INT to FIXNUM, where the usage describes non-bignum Lisp integers. See the src/ChangeLog entry for more details. man/ChangeLog addition: 2011-10-09 Aidan Kehoe <kehoea@parhasard.net> * internals/internals.texi (How Lisp Objects Are Represented in C): * internals/internals.texi (Integers and Characters): Mechanically change INT to FIXNUM, where the usage describes non-bignum Lisp integers.
author Aidan Kehoe <kehoea@parhasard.net>
date Sun, 09 Oct 2011 09:51:57 +0100
parents 6b3caa55668c
children 873d7425c1ad
line wrap: on
line diff
--- a/src/data.c	Sat Oct 08 12:26:09 2011 +0100
+++ b/src/data.c	Sun Oct 09 09:51:57 2011 +0100
@@ -78,7 +78,7 @@
 eq_with_ebola_notice (Lisp_Object obj1, Lisp_Object obj2)
 {
   if (debug_issue_ebola_notices
-      && ((CHARP (obj1) && INTP (obj2)) || (CHARP (obj2) && INTP (obj1))))
+      && ((CHARP (obj1) && FIXNUMP (obj2)) || (CHARP (obj2) && FIXNUMP (obj1))))
     {
       /* #### It would be really nice if this were a proper warning
          instead of brain-dead print to Qexternal_debugging_output.  */
@@ -321,7 +321,7 @@
        (subr))
 {
   CHECK_SUBR (subr);
-  return make_int (XSUBR (subr)->min_args);
+  return make_fixnum (XSUBR (subr)->min_args);
 }
 
 DEFUN ("subr-max-args", Fsubr_max_args, 1, 1, 0, /*
@@ -336,7 +336,7 @@
   if (nargs == MANY || nargs == UNEVALLED)
     return Qnil;
   else
-    return make_int (nargs);
+    return make_fixnum (nargs);
 }
 
 DEFUN ("subr-interactive", Fsubr_interactive, 1, 1, 0, /*
@@ -395,7 +395,7 @@
        (character))
 {
   CHECK_CHAR (character);
-  return make_int (XCHAR (character));
+  return make_fixnum (XCHAR (character));
 }
 
 DEFUN ("int-to-char", Fint_to_char, 1, 1, 0, /*
@@ -408,7 +408,7 @@
 {
   CHECK_INTEGER (integer);
   if (CHAR_INTP (integer))
-    return make_char (XINT (integer));
+    return make_char (XFIXNUM (integer));
   else
     return Qnil;
 }
@@ -452,7 +452,7 @@
 */
        (object))
 {
-  return INTP (object) ? Qt : Qnil;
+  return FIXNUMP (object) ? Qt : Qnil;
 }
 DEFUN ("integerp", Fintegerp, 1, 1, 0, /*
 Return t if OBJECT is an integer, nil otherwise.
@@ -714,7 +714,7 @@
 
  retry:
 
-  if      (INTP  (index_)) idx = XINT  (index_);
+  if      (FIXNUMP  (index_)) idx = XFIXNUM  (index_);
   else if (CHARP (index_)) idx = XCHAR (index_); /* yuck! */
 #ifdef HAVE_BIGNUM
   else if (BIGNUMP (index_))
@@ -746,7 +746,7 @@
     {
       if (idx >= (EMACS_INT) bit_vector_length (XBIT_VECTOR (array)))
 	goto range_error;
-      return make_int (bit_vector_bit (XBIT_VECTOR (array), idx));
+      return make_fixnum (bit_vector_bit (XBIT_VECTOR (array), idx));
     }
   else if (STRINGP (array))
     {
@@ -775,7 +775,7 @@
 
  retry:
 
-  if      (INTP  (index_)) idx = XINT (index_);
+  if      (FIXNUMP  (index_)) idx = XFIXNUM (index_);
   else if (CHARP (index_)) idx = XCHAR (index_); /* yuck! */
 #ifdef HAVE_BIGNUM
   else if (BIGNUMP (index_))
@@ -851,7 +851,7 @@
 {
  retry:
   p->int_p = 1;
-  if      (INTP    (obj)) p->c.ival = XINT  (obj);
+  if      (FIXNUMP    (obj)) p->c.ival = XFIXNUM  (obj);
   else if (CHARP   (obj)) p->c.ival = XCHAR (obj);
   else if (MARKERP (obj)) p->c.ival = marker_position (obj);
   else if (FLOATP  (obj)) p->c.dval = XFLOAT_DATA (obj), p->int_p = 0;
@@ -866,7 +866,7 @@
 number_char_or_marker_to_double (Lisp_Object obj)
 {
  retry:
-  if      (INTP    (obj)) return (double) XINT  (obj);
+  if      (FIXNUMP    (obj)) return (double) XFIXNUM  (obj);
   else if (CHARP   (obj)) return (double) XCHAR (obj);
   else if (MARKERP (obj)) return (double) marker_position (obj);
   else if (FLOATP  (obj)) return XFLOAT_DATA (obj);
@@ -882,7 +882,7 @@
 fixnum_char_or_marker_to_int (Lisp_Object obj)
 {
  retry:
-  if      (INTP    (obj)) return XINT  (obj);
+  if      (FIXNUMP    (obj)) return XFIXNUM  (obj);
   else if (CHARP   (obj)) return XCHAR (obj);
   else if (MARKERP (obj)) return marker_position (obj);
   else
@@ -939,7 +939,7 @@
       switch (promote_args (&obj1, &obj2))			\
 	{							\
 	case FIXNUM_T:						\
-	  if (!(XREALINT (obj1) c_op XREALINT (obj2)))		\
+	  if (!(XREALFIXNUM (obj1) c_op XREALFIXNUM (obj2)))		\
 	    return Qnil;					\
 	  break;						\
 	BIGNUM_CASE (op)					\
@@ -1069,7 +1069,7 @@
 	  switch (promote_args (&obj1, &obj2))
 	    {
 	    case FIXNUM_T:
-	      if (XREALINT (obj1) == XREALINT (obj2))
+	      if (XREALFIXNUM (obj1) == XREALFIXNUM (obj2))
 		return Qnil;
 	      break;
 #ifdef HAVE_BIGNUM
@@ -1130,7 +1130,7 @@
        (number))
 {
  retry:
-  if (INTP (number))
+  if (FIXNUMP (number))
     return EQ (number, Qzero) ? Qt : Qnil;
 #ifdef HAVE_BIGNUM
   else if (BIGNUMP (number))
@@ -1163,21 +1163,21 @@
 Lisp_Object
 word_to_lisp (unsigned int item)
 {
-  return Fcons (make_int (item >> 16), make_int (item & 0xffff));
+  return Fcons (make_fixnum (item >> 16), make_fixnum (item & 0xffff));
 }
 
 unsigned int
 lisp_to_word (Lisp_Object item)
 {
-  if (INTP (item))
-    return XINT (item);
+  if (FIXNUMP (item))
+    return XFIXNUM (item);
   else
     {
       Lisp_Object top = Fcar (item);
       Lisp_Object bot = Fcdr (item);
-      CHECK_INT (top);
-      CHECK_INT (bot);
-      return (XINT (top) << 16) | (XINT (bot) & 0xffff);
+      CHECK_FIXNUM (top);
+      CHECK_FIXNUM (bot);
+      return (XFIXNUM (top) << 16) | (XFIXNUM (bot) & 0xffff);
     }
 }
 
@@ -1230,7 +1230,7 @@
   {
     Ascbyte buffer[DECIMAL_PRINT_SIZE (long)];
 
-    long_to_string (buffer, XINT (number));
+    long_to_string (buffer, XFIXNUM (number));
     return build_ascstring (buffer);
   }
 }
@@ -1270,8 +1270,8 @@
     b = 10;
   else
     {
-      check_integer_range (base, make_int (2), make_int (16));
-      b = XINT (base);
+      check_integer_range (base, make_fixnum (2), make_fixnum (16));
+      b = XFIXNUM (base);
     }
 
   p = XSTRING_DATA (string);
@@ -1379,7 +1379,7 @@
     save = *end;
     *end = '\0';
     if (*p == '\0')
-      retval = make_int (0);
+      retval = make_fixnum (0);
     else
       {
 	bignum_set_string (scratch_bignum, (const char *) p, b);
@@ -1393,11 +1393,11 @@
     {
       /* Use the system-provided functions for base 10. */
 #if   SIZEOF_EMACS_INT == SIZEOF_INT
-      return make_int (atoi ((char*) p));
+      return make_fixnum (atoi ((char*) p));
 #elif SIZEOF_EMACS_INT == SIZEOF_LONG
-      return make_int (atol ((char*) p));
+      return make_fixnum (atol ((char*) p));
 #elif SIZEOF_EMACS_INT == SIZEOF_LONG_LONG
-      return make_int (atoll ((char*) p));
+      return make_fixnum (atoll ((char*) p));
 #endif
     }
   else
@@ -1419,7 +1419,7 @@
 	    break;
 	  v = v * b + digit;
 	}
-      return make_int (negative * v);
+      return make_fixnum (negative * v);
     }
 #endif /* HAVE_BIGNUM */
 }
@@ -1435,7 +1435,7 @@
 {
 #ifdef WITH_NUMBER_TYPES
   REGISTER int i;
-  Lisp_Object accum = make_int (0), addend;
+  Lisp_Object accum = make_fixnum (0), addend;
 
   for (i = 0; i < nargs; i++)
     {
@@ -1443,7 +1443,7 @@
       switch (promote_args (&accum, &addend))
 	{
 	case FIXNUM_T:
-	  accum = make_integer (XREALINT (accum) + XREALINT (addend));
+	  accum = make_integer (XREALFIXNUM (accum) + XREALFIXNUM (addend));
 	  break;
 #ifdef HAVE_BIGNUM
 	case BIGNUM_T:
@@ -1494,7 +1494,7 @@
 	}
     }
 
-  return make_int (iaccum);
+  return make_fixnum (iaccum);
 #endif /* WITH_NUMBER_TYPES */
 }
 
@@ -1514,16 +1514,16 @@
   if (nargs == 1)
     {
       if (CHARP (accum))
-	accum = make_int (XCHAR (accum));
+	accum = make_fixnum (XCHAR (accum));
       else if (MARKERP (accum))
-	accum = make_int (marker_position (accum));
+	accum = make_fixnum (marker_position (accum));
 
       /* Invert the sign of accum */
       CHECK_NUMBER (accum);
       switch (get_number_type (accum))
 	{
 	case FIXNUM_T:
-	  return make_integer (-XREALINT (accum));
+	  return make_integer (-XREALFIXNUM (accum));
 #ifdef HAVE_BIGNUM
 	case BIGNUM_T:
 	  bignum_neg (scratch_bignum, XBIGNUM_DATA (accum));
@@ -1553,7 +1553,7 @@
 	  switch (promote_args (&accum, &subtrahend))
 	    {
 	    case FIXNUM_T:
-	      accum = make_integer (XREALINT (accum) - XREALINT (subtrahend));
+	      accum = make_integer (XREALFIXNUM (accum) - XREALFIXNUM (subtrahend));
 	      break;
 #ifdef HAVE_BIGNUM
 	    case BIGNUM_T:
@@ -1614,7 +1614,7 @@
 	}
     }
 
-  return make_int (iaccum);
+  return make_fixnum (iaccum);
 
  do_float:
   for (; args < args_end; args++)
@@ -1690,7 +1690,7 @@
 	}
     }
 
-  return make_int (iaccum);
+  return make_fixnum (iaccum);
 #endif /* WITH_NUMBER_TYPES */
 }
 
@@ -1711,7 +1711,7 @@
   if (nargs == 1)
     {
       i = 0;
-      accum = make_int (1);
+      accum = make_fixnum (1);
     }
   else
     {
@@ -1724,9 +1724,9 @@
       switch (promote_args (&accum, &divisor))
 	{
 	case FIXNUM_T:
-	  if (XREALINT (divisor) == 0) goto divide_by_zero;
-	  bignum_set_long (scratch_bignum, XREALINT (accum));
-	  bignum_set_long (scratch_bignum2, XREALINT (divisor));
+	  if (XREALFIXNUM (divisor) == 0) goto divide_by_zero;
+	  bignum_set_long (scratch_bignum, XREALFIXNUM (accum));
+	  bignum_set_long (scratch_bignum2, XREALFIXNUM (divisor));
 	  accum = make_ratio_bg (scratch_bignum, scratch_bignum2);
 	  break;
 	case BIGNUM_T:
@@ -1781,7 +1781,7 @@
   if (nargs == 1)
     {
       i = 0;
-      accum = make_int (1);
+      accum = make_fixnum (1);
     }
   else
     {
@@ -1794,8 +1794,8 @@
       switch (promote_args (&accum, &divisor))
 	{
 	case FIXNUM_T:
-	  if (XREALINT (divisor) == 0) goto divide_by_zero;
-	  accum = make_integer (XREALINT (accum) / XREALINT (divisor));
+	  if (XREALFIXNUM (divisor) == 0) goto divide_by_zero;
+	  accum = make_integer (XREALFIXNUM (accum) / XREALFIXNUM (divisor));
 	  break;
 #ifdef HAVE_BIGNUM
 	case BIGNUM_T:
@@ -1868,7 +1868,7 @@
 	}
     }
 
-  return make_int (iaccum);
+  return make_fixnum (iaccum);
 
  divide_floats:
   for (; args < args_end; args++)
@@ -1902,9 +1902,9 @@
   while (!(CHARP (args[0]) || MARKERP (args[0]) || REALP (args[0])))
     args[0] = wrong_type_argument (Qnumber_char_or_marker_p, args[0]);
   if (CHARP (args[0]))
-    args[0] = make_int (XCHAR (args[0]));
+    args[0] = make_fixnum (XCHAR (args[0]));
   else if (MARKERP (args[0]))
-    args[0] = make_int (marker_position (args[0]));
+    args[0] = make_fixnum (marker_position (args[0]));
   for (i = 1; i < nargs; i++)
     {
       comp1 = args[maxindex];
@@ -1912,7 +1912,7 @@
       switch (promote_args (&comp1, &comp2))
 	{
 	case FIXNUM_T:
-	  if (XREALINT (comp1) < XREALINT (comp2))
+	  if (XREALFIXNUM (comp1) < XREALFIXNUM (comp2))
 	    maxindex = i;
 	  break;
 #ifdef HAVE_BIGNUM
@@ -1970,7 +1970,7 @@
 	}
     }
 
-  return make_int (imax);
+  return make_fixnum (imax);
 
  max_floats:
   while (args < args_end)
@@ -1999,9 +1999,9 @@
   while (!(CHARP (args[0]) || MARKERP (args[0]) || REALP (args[0])))
     args[0] = wrong_type_argument (Qnumber_char_or_marker_p, args[0]);
   if (CHARP (args[0]))
-    args[0] = make_int (XCHAR (args[0]));
+    args[0] = make_fixnum (XCHAR (args[0]));
   else if (MARKERP (args[0]))
-    args[0] = make_int (marker_position (args[0]));
+    args[0] = make_fixnum (marker_position (args[0]));
   for (i = 1; i < nargs; i++)
     {
       comp1 = args[minindex];
@@ -2009,7 +2009,7 @@
       switch (promote_args (&comp1, &comp2))
 	{
 	case FIXNUM_T:
-	  if (XREALINT (comp1) > XREALINT (comp2))
+	  if (XREALFIXNUM (comp1) > XREALFIXNUM (comp2))
 	    minindex = i;
 	  break;
 #ifdef HAVE_BIGNUM
@@ -2067,7 +2067,7 @@
 	}
     }
 
-  return make_int (imin);
+  return make_fixnum (imin);
 
  min_floats:
   while (args < args_end)
@@ -2092,16 +2092,16 @@
   Lisp_Object result, other;
 
   if (nargs == 0)
-    return make_int (~0);
+    return make_fixnum (~0);
 
   while (!(CHARP (args[0]) || MARKERP (args[0]) || INTEGERP (args[0])))
     args[0] = wrong_type_argument (Qnumber_char_or_marker_p, args[0]);
 
   result = args[0];
   if (CHARP (result))
-    result = make_int (XCHAR (result));
+    result = make_fixnum (XCHAR (result));
   else if (MARKERP (result))
-    result = make_int (marker_position (result));
+    result = make_fixnum (marker_position (result));
   for (i = 1; i < nargs; i++)
     {
       while (!(CHARP (args[i]) || MARKERP (args[i]) || INTEGERP (args[i])))
@@ -2110,7 +2110,7 @@
       switch (promote_args (&result, &other))
 	{
 	case FIXNUM_T:
-	  result = make_int (XREALINT (result) & XREALINT (other));
+	  result = make_fixnum (XREALFIXNUM (result) & XREALFIXNUM (other));
 	  break;
 	case BIGNUM_T:
 	  bignum_and (scratch_bignum, XBIGNUM_DATA (result),
@@ -2127,7 +2127,7 @@
   while (args < args_end)
     bits &= fixnum_char_or_marker_to_int (*args++);
 
-  return make_int (bits);
+  return make_fixnum (bits);
 #endif /* HAVE_BIGNUM */
 }
 
@@ -2144,16 +2144,16 @@
   Lisp_Object result, other;
 
   if (nargs == 0)
-    return make_int (0);
+    return make_fixnum (0);
 
   while (!(CHARP (args[0]) || MARKERP (args[0]) || INTEGERP (args[0])))
     args[0] = wrong_type_argument (Qnumber_char_or_marker_p, args[0]);
 
   result = args[0];
   if (CHARP (result))
-    result = make_int (XCHAR (result));
+    result = make_fixnum (XCHAR (result));
   else if (MARKERP (result))
-    result = make_int (marker_position (result));
+    result = make_fixnum (marker_position (result));
   for (i = 1; i < nargs; i++)
     {
       while (!(CHARP (args[i]) || MARKERP (args[i]) || INTEGERP (args[i])))
@@ -2162,7 +2162,7 @@
       switch (promote_args (&result, &other))
 	{
 	case FIXNUM_T:
-	  result = make_int (XREALINT (result) | XREALINT (other));
+	  result = make_fixnum (XREALFIXNUM (result) | XREALFIXNUM (other));
 	  break;
 	case BIGNUM_T:
 	  bignum_ior (scratch_bignum, XBIGNUM_DATA (result),
@@ -2179,7 +2179,7 @@
   while (args < args_end)
     bits |= fixnum_char_or_marker_to_int (*args++);
 
-  return make_int (bits);
+  return make_fixnum (bits);
 #endif /* HAVE_BIGNUM */
 }
 
@@ -2196,16 +2196,16 @@
   Lisp_Object result, other;
 
   if (nargs == 0)
-    return make_int (0);
+    return make_fixnum (0);
 
   while (!(CHARP (args[0]) || MARKERP (args[0]) || INTEGERP (args[0])))
     args[0] = wrong_type_argument (Qinteger_char_or_marker_p, args[0]);
 
   result = args[0];
   if (CHARP (result))
-    result = make_int (XCHAR (result));
+    result = make_fixnum (XCHAR (result));
   else if (MARKERP (result))
-    result = make_int (marker_position (result));
+    result = make_fixnum (marker_position (result));
   for (i = 1; i < nargs; i++)
     {
       while (!(CHARP (args[i]) || MARKERP (args[i]) || INTEGERP (args[i])))
@@ -2213,7 +2213,7 @@
       other = args[i];
       if (promote_args (&result, &other) == FIXNUM_T)
 	{
-	  result = make_int (XREALINT (result) ^ XREALINT (other));
+	  result = make_fixnum (XREALFIXNUM (result) ^ XREALFIXNUM (other));
 	}
       else
 	{
@@ -2230,7 +2230,7 @@
   while (args < args_end)
     bits ^= fixnum_char_or_marker_to_int (*args++);
 
-  return make_int (bits);
+  return make_fixnum (bits);
 #endif /* !HAVE_BIGNUM */
 }
 
@@ -2251,7 +2251,7 @@
     }
 #endif /* HAVE_BIGNUM */
 
-  return make_int (~ fixnum_char_or_marker_to_int (number));
+  return make_fixnum (~ fixnum_char_or_marker_to_int (number));
 }
 
 DEFUN ("%", Frem, 2, 2, 0, /*
@@ -2268,9 +2268,9 @@
 
   if (promote_args (&number1, &number2) == FIXNUM_T)
     {
-      if (XREALINT (number2) == 0)
+      if (XREALFIXNUM (number2) == 0)
 	Fsignal (Qarith_error, Qnil);
-      return make_int (XREALINT (number1) % XREALINT (number2));
+      return make_fixnum (XREALFIXNUM (number1) % XREALFIXNUM (number2));
     }
   else
     {
@@ -2287,7 +2287,7 @@
   if (ival2 == 0)
     Fsignal (Qarith_error, Qnil);
 
-  return make_int (ival1 % ival2);
+  return make_fixnum (ival1 % ival2);
 #endif /* HAVE_BIGNUM */
 }
 
@@ -2324,12 +2324,12 @@
     case FIXNUM_T:
       {
 	EMACS_INT ival;
-	if (XREALINT (y) == 0) goto divide_by_zero;
-	ival = XREALINT (x) % XREALINT (y);
+	if (XREALFIXNUM (y) == 0) goto divide_by_zero;
+	ival = XREALFIXNUM (x) % XREALFIXNUM (y);
 	/* If the "remainder" comes out with the wrong sign, fix it.  */
-	if (XREALINT (y) < 0 ? ival > 0 : ival < 0)
-	  ival += XREALINT (y);
-	return make_int (ival);
+	if (XREALFIXNUM (y) < 0 ? ival > 0 : ival < 0)
+	  ival += XREALFIXNUM (y);
+	return make_fixnum (ival);
       }
 #ifdef HAVE_BIGNUM
     case BIGNUM_T:
@@ -2398,7 +2398,7 @@
     if (iod2.c.ival < 0 ? ival > 0 : ival < 0)
       ival += iod2.c.ival;
 
-    return make_int (ival);
+    return make_fixnum (ival);
   }
 #endif /* WITH_NUMBER_TYPES */
 
@@ -2416,12 +2416,12 @@
 */
        (value, count))
 {
-  CHECK_INT_COERCE_CHAR (value);
-  CONCHECK_INT (count);
-
-  return make_int (XINT (count) > 0 ?
-		   XINT (value) <<  XINT (count) :
-		   XINT (value) >> -XINT (count));
+  CHECK_FIXNUM_COERCE_CHAR (value);
+  CONCHECK_FIXNUM (count);
+
+  return make_fixnum (XFIXNUM (count) > 0 ?
+		   XFIXNUM (value) <<  XFIXNUM (count) :
+		   XFIXNUM (value) >> -XFIXNUM (count));
 }
 
 DEFUN ("lsh", Flsh, 2, 2, 0, /*
@@ -2438,11 +2438,11 @@
 
   if (promote_args (&value, &count) == FIXNUM_T)
     {
-      if (XREALINT (count) <= 0)
-	return make_int (XREALINT (value) >> -XREALINT (count));
+      if (XREALFIXNUM (count) <= 0)
+	return make_fixnum (XREALFIXNUM (value) >> -XREALFIXNUM (count));
       /* Use bignums to avoid overflow */
-      bignum_set_long (scratch_bignum2, XREALINT (value));
-      bignum_lshift (scratch_bignum, scratch_bignum2, XREALINT (count));
+      bignum_set_long (scratch_bignum2, XREALFIXNUM (value));
+      bignum_lshift (scratch_bignum, scratch_bignum2, XREALFIXNUM (count));
       return Fcanonicalize_number (make_bignum_bg (scratch_bignum));
     }
   else
@@ -2465,12 +2465,12 @@
       return Fcanonicalize_number (make_bignum_bg (scratch_bignum2));
     }
 #else /* !HAVE_BIGNUM */
-  CHECK_INT_COERCE_CHAR (value);
-  CONCHECK_INT (count);
-
-  return make_int (XINT (count) > 0 ?
-		   XUINT (value) <<  XINT (count) :
-		   XUINT (value) >> -XINT (count));
+  CHECK_FIXNUM_COERCE_CHAR (value);
+  CONCHECK_FIXNUM (count);
+
+  return make_fixnum (XFIXNUM (count) > 0 ?
+		   XUINT (value) <<  XFIXNUM (count) :
+		   XUINT (value) >> -XFIXNUM (count));
 #endif /* HAVE_BIGNUM */
 }
 
@@ -2482,7 +2482,7 @@
 {
  retry:
 
-  if (INTP    (number)) return make_integer (XINT (number) + 1);
+  if (FIXNUMP    (number)) return make_integer (XFIXNUM (number) + 1);
   if (CHARP   (number)) return make_integer (XCHAR (number) + 1);
   if (MARKERP (number)) return make_integer (marker_position (number) + 1);
   if (FLOATP  (number)) return make_float (XFLOAT_DATA (number) + 1.0);
@@ -2526,7 +2526,7 @@
 {
  retry:
 
-  if (INTP    (number)) return make_integer (XINT (number) - 1);
+  if (FIXNUMP    (number)) return make_integer (XFIXNUM (number) - 1);
   if (CHARP   (number)) return make_integer (XCHAR (number) - 1);
   if (MARKERP (number)) return make_integer (marker_position (number) - 1);
   if (FLOATP  (number)) return make_float (XFLOAT_DATA (number) - 1.0);
@@ -3650,12 +3650,12 @@
   DEFVAR_CONST_INT ("most-negative-fixnum", &Vmost_negative_fixnum /*
 The fixnum closest in value to negative infinity.
 */);
-  Vmost_negative_fixnum = EMACS_INT_MIN;
+  Vmost_negative_fixnum = MOST_NEGATIVE_FIXNUM;
 
   DEFVAR_CONST_INT ("most-positive-fixnum", &Vmost_positive_fixnum /*
 The fixnum closest in value to positive infinity.
 */);
-  Vmost_positive_fixnum = EMACS_INT_MAX;
+  Vmost_positive_fixnum = MOST_POSITIVE_FIXNUM;
 
 #ifdef DEBUG_XEMACS
   DEFVAR_BOOL ("debug-issue-ebola-notices", &debug_issue_ebola_notices /*