Merge branch 'master' into boehm-demers-weiser-gc
[bpt/guile.git] / libguile / numbers.c
index 330844a..0452e43 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001,2002,2003,2004 Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001,2002,2003,2004,2005, 2006, 2007, 2008 Free Software Foundation, Inc.
  *
  * Portions Copyright 1990, 1991, 1992, 1993 by AT&T Bell Laboratories
  * and Bellcore.  See scm_divide.
@@ -16,7 +16,7 @@
  *
  * You should have received a copy of the GNU Lesser General Public
  * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
 \f
 
  */
 
-/* tell glibc (2.3) to give prototype for C99 trunc() */
-#define _GNU_SOURCE
-
-#if HAVE_CONFIG_H
+#ifdef HAVE_CONFIG_H
 #  include <config.h>
 #endif
 
 #include <math.h>
 #include <ctype.h>
 #include <string.h>
-#include <gmp.h>
+
+#if HAVE_COMPLEX_H
+#include <complex.h>
+#endif
 
 #include "libguile/_scm.h"
 #include "libguile/feature.h"
 
 #include "libguile/discouraged.h"
 
+/* values per glibc, if not already defined */
+#ifndef M_LOG10E
+#define M_LOG10E   0.43429448190325182765
+#endif
+#ifndef M_PI
+#define M_PI       3.14159265358979323846
+#endif
+
 \f
 
 /*
@@ -116,10 +124,10 @@ isinf (double x)
 #endif
 
 
-/* mpz_cmp_d only recognises infinities in gmp 4.2 and up.
-   For prior versions use an explicit check here.  */
-#if __GNU_MP_VERSION < 4                                        \
-  || (__GNU_MP_VERSION == 4 && __GNU_MP_VERSION_MINOR < 2)
+/* mpz_cmp_d in gmp 4.1.3 doesn't recognise infinities, so xmpz_cmp_d uses
+   an explicit check.  In some future gmp (don't know what version number),
+   mpz_cmp_d is supposed to do this itself.  */
+#if 1
 #define xmpz_cmp_d(z, d)                                \
   (xisinf (d) ? (d < 0.0 ? 1 : -1) : mpz_cmp_d (z, d))
 #else
@@ -151,13 +159,33 @@ xisnan (double x)
 #endif
 }
 
+#if defined (GUILE_I)
+#if HAVE_COMPLEX_DOUBLE
+
+/* For an SCM object Z which is a complex number (ie. satisfies
+   SCM_COMPLEXP), return its value as a C level "complex double". */
+#define SCM_COMPLEX_VALUE(z)                                    \
+  (SCM_COMPLEX_REAL (z) + GUILE_I * SCM_COMPLEX_IMAG (z))
+
+static inline SCM scm_from_complex_double (complex double z) SCM_UNUSED;
+
+/* Convert a C "complex double" to an SCM value. */
+static inline SCM
+scm_from_complex_double (complex double z)
+{
+  return scm_c_make_rectangular (creal (z), cimag (z));
+}
+
+#endif /* HAVE_COMPLEX_DOUBLE */
+#endif /* GUILE_I */
+
 \f
 
 static mpz_t z_negative_one;
 
 \f
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_mkbig ()
 {
   /* Return a newly created bignum. */
@@ -166,7 +194,7 @@ scm_i_mkbig ()
   return z;
 }
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_long2big (long x)
 {
   /* Return a newly created bignum initialized to X. */
@@ -175,7 +203,7 @@ scm_i_long2big (long x)
   return z;
 }
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_ulong2big (unsigned long x)
 {
   /* Return a newly created bignum initialized to X. */
@@ -184,7 +212,7 @@ scm_i_ulong2big (unsigned long x)
   return z;
 }
 
-SCM_C_INLINE_KEYWORD static SCM
+SCM
 scm_i_clonebig (SCM src_big, int same_sign_p)
 {
   /* Copy src_big's value, negate it if same_sign_p is false, and return. */
@@ -195,7 +223,7 @@ scm_i_clonebig (SCM src_big, int same_sign_p)
   return z;
 }
 
-SCM_C_INLINE_KEYWORD int
+int
 scm_i_bigcmp (SCM x, SCM y)
 {
   /* Return neg if x < y, pos if x > y, and 0 if x == y */
@@ -205,7 +233,7 @@ scm_i_bigcmp (SCM x, SCM y)
   return result;
 }
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_dbl2big (double d)
 {
   /* results are only defined if d is an integer */
@@ -216,7 +244,7 @@ scm_i_dbl2big (double d)
 
 /* Convert a integer in double representation to a SCM number. */
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_dbl2num (double u)
 {
   /* SCM_MOST_POSITIVE_FIXNUM+1 and SCM_MOST_NEGATIVE_FIXNUM are both
@@ -244,23 +272,29 @@ scm_i_dbl2num (double u)
    with R5RS exact->inexact.
 
    The approach is to use mpz_get_d to pick out the high DBL_MANT_DIG bits
-   (ie. it truncates towards zero), then adjust to get the closest double by
-   examining the next lower bit and adding 1 if necessary.
-
-   Note that bignums exactly half way between representable doubles are
-   rounded to the next higher absolute value (ie. away from zero).  This
-   seems like an adequate interpretation of R5RS "numerically closest", and
-   it's easier and faster than a full "nearest-even" style.
-
-   The bit test is done on the absolute value of the mpz_t, which means we
-   must use mpz_getlimbn.  mpz_tstbit is not right, it treats negatives as
-   twos complement.
-
-   Prior to GMP 4.2, the rounding done by mpz_get_d was unspecified.  It
-   happened to follow the hardware rounding mode, but on the absolute value
-   of its operand.  This is not what we want, so we put the high
-   DBL_MANT_DIG bits into a temporary.  This extra init/clear is a slowdown,
-   but doesn't matter too much since it's only for older GMP.  */
+   (ie. truncate towards zero), then adjust to get the closest double by
+   examining the next lower bit and adding 1 (to the absolute value) if
+   necessary.
+
+   Bignums exactly half way between representable doubles are rounded to the
+   next higher absolute value (ie. away from zero).  This seems like an
+   adequate interpretation of R5RS "numerically closest", and it's easier
+   and faster than a full "nearest-even" style.
+
+   The bit test must be done on the absolute value of the mpz_t, which means
+   we need to use mpz_getlimbn.  mpz_tstbit is not right, it treats
+   negatives as twos complement.
+
+   In current gmp 4.1.3, mpz_get_d rounding is unspecified.  It ends up
+   following the hardware rounding mode, but applied to the absolute value
+   of the mpz_t operand.  This is not what we want so we put the high
+   DBL_MANT_DIG bits into a temporary.  In some future gmp, don't know when,
+   mpz_get_d is supposed to always truncate towards zero.
+
+   ENHANCE-ME: The temporary init+clear to force the rounding in gmp 4.1.3
+   is a slowdown.  It'd be faster to pick out the relevant high bits with
+   mpz_getlimbn if we could be bothered coding that, and if the new
+   truncating gmp doesn't come out.  */
 
 double
 scm_i_big2dbl (SCM b)
@@ -270,10 +304,9 @@ scm_i_big2dbl (SCM b)
 
   bits = mpz_sizeinbase (SCM_I_BIG_MPZ (b), 2);
 
-#if __GNU_MP_VERSION < 4                                        \
-  || (__GNU_MP_VERSION == 4 && __GNU_MP_VERSION_MINOR < 2)
+#if 1
   {
-    /* GMP prior to 4.2, force truncate towards zero */
+    /* Current GMP, eg. 4.1.3, force truncation towards zero */
     mpz_t  tmp;
     if (bits > DBL_MANT_DIG)
       {
@@ -289,7 +322,7 @@ scm_i_big2dbl (SCM b)
       }
   }
 #else
-  /* GMP 4.2 and up */
+  /* Future GMP */
   result = mpz_get_d (SCM_I_BIG_MPZ (b));
 #endif
 
@@ -308,7 +341,7 @@ scm_i_big2dbl (SCM b)
   return result;
 }
 
-SCM_C_INLINE_KEYWORD SCM
+SCM
 scm_i_normbig (SCM b)
 {
   /* convert a big back to a fixnum if it'll fit */
@@ -343,8 +376,8 @@ scm_i_mpz2num (mpz_t b)
 /* this is needed when we want scm_divide to make a float, not a ratio, even if passed two ints */
 static SCM scm_divide2real (SCM x, SCM y);
 
-SCM
-scm_make_ratio (SCM numerator, SCM denominator)
+static SCM
+scm_i_make_ratio (SCM numerator, SCM denominator)
 #define FUNC_NAME "make-ratio"
 {
   /* First make sure the arguments are proper.
@@ -421,28 +454,21 @@ scm_make_ratio (SCM numerator, SCM denominator)
 
   /* No, it's a proper fraction.
    */
-  return scm_double_cell (scm_tc16_fraction,
-                         SCM_UNPACK (numerator),
-                         SCM_UNPACK (denominator), 0);
+  {
+    SCM divisor = scm_gcd (numerator, denominator);
+    if (!(scm_is_eq (divisor, SCM_I_MAKINUM(1))))
+      {
+       numerator = scm_divide (numerator, divisor);
+       denominator = scm_divide (denominator, divisor);
+      }
+      
+    return scm_double_cell (scm_tc16_fraction,
+                           SCM_UNPACK (numerator),
+                           SCM_UNPACK (denominator), 0);
+  }
 }
 #undef FUNC_NAME
 
-static void scm_i_fraction_reduce (SCM z)
-{
-  if (!(SCM_FRACTION_REDUCED (z)))
-    {
-      SCM divisor;
-      divisor = scm_gcd (SCM_FRACTION_NUMERATOR (z), SCM_FRACTION_DENOMINATOR (z));
-      if (!(scm_is_eq (divisor, SCM_I_MAKINUM(1))))
-       {
-         /* is this safe? */
-         SCM_FRACTION_SET_NUMERATOR (z, scm_divide (SCM_FRACTION_NUMERATOR (z), divisor));
-         SCM_FRACTION_SET_DENOMINATOR (z, scm_divide (SCM_FRACTION_DENOMINATOR (z), divisor));
-       }
-      SCM_FRACTION_REDUCED_SET (z);
-    }
-}
-
 double
 scm_i_fraction2double (SCM z)
 {
@@ -539,16 +565,16 @@ SCM_DEFINE (scm_even_p, "even?", 1, 0, 0,
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_inf_p, "inf?", 1, 0, 0, 
-            (SCM n),
-           "Return @code{#t} if @var{n} is infinite, @code{#f}\n"
-           "otherwise.")
+            (SCM x),
+           "Return @code{#t} if @var{x} is either @samp{+inf.0}\n"
+           "or @samp{-inf.0}, @code{#f} otherwise.")
 #define FUNC_NAME s_scm_inf_p
 {
-  if (SCM_REALP (n))
-    return scm_from_bool (xisinf (SCM_REAL_VALUE (n)));
-  else if (SCM_COMPLEXP (n))
-    return scm_from_bool (xisinf (SCM_COMPLEX_REAL (n))
-                    || xisinf (SCM_COMPLEX_IMAG (n)));
+  if (SCM_REALP (x))
+    return scm_from_bool (xisinf (SCM_REAL_VALUE (x)));
+  else if (SCM_COMPLEXP (x))
+    return scm_from_bool (xisinf (SCM_COMPLEX_REAL (x))
+                         || xisinf (SCM_COMPLEX_IMAG (x)));
   else
     return SCM_BOOL_F;
 }
@@ -594,7 +620,7 @@ guile_ieee_init (void)
 #elif HAVE_DINFINITY
   /* OSF */
   extern unsigned int DINFINITY[2];
-  guile_Inf = (*(X_CAST(double *, DINFINITY)));
+  guile_Inf = (*((double *) (DINFINITY)));
 #else
   double tmp = 1e+10;
   guile_Inf = tmp;
@@ -615,9 +641,11 @@ guile_ieee_init (void)
   /* C99 NAN, when available */
   guile_NaN = NAN;
 #elif HAVE_DQNAN
-  /* OSF */
-  extern unsigned int DQNAN[2];
-  guile_NaN =  (*(X_CAST(double *, DQNAN)));
+  {
+    /* OSF */
+    extern unsigned int DQNAN[2];
+    guile_NaN = (*((double *)(DQNAN)));
+  }
 #else
   guile_NaN = guile_Inf / guile_Inf;
 #endif
@@ -692,7 +720,7 @@ SCM_PRIMITIVE_GENERIC (scm_abs, "abs", 1, 0, 0,
     {
       if (scm_is_false (scm_negative_p (SCM_FRACTION_NUMERATOR (x))))
        return x;
-      return scm_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), SCM_UNDEFINED),
+      return scm_i_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), SCM_UNDEFINED),
                             SCM_FRACTION_DENOMINATOR (x));
     }
   else
@@ -874,8 +902,9 @@ scm_modulo (SCM x, SCM y)
            scm_num_overflow (s_modulo);
          else
            {
-             /* FIXME: I think this may be a bug on some arches -- results
-                of % with negative second arg are undefined... */
+             /* C99 specifies that "%" is the remainder corresponding to a
+                 quotient rounded towards zero, and that's also traditional
+                 for machine division, so z here should be well defined.  */
              long z = xx % yy;
              long result;
 
@@ -1304,7 +1333,7 @@ SCM_DEFINE1 (scm_logior, "logior", scm_tc7_asubr,
            mpz_ior (SCM_I_BIG_MPZ (result_z), nn1_z, SCM_I_BIG_MPZ (n2));
            scm_remember_upto_here_1 (n2);
            mpz_clear (nn1_z);
-           return result_z;
+           return scm_i_normbig (result_z);
          }
        }
       else
@@ -1325,7 +1354,7 @@ SCM_DEFINE1 (scm_logior, "logior", scm_tc7_asubr,
                   SCM_I_BIG_MPZ (n1),
                   SCM_I_BIG_MPZ (n2));
          scm_remember_upto_here_2 (n1, n2);
-         return result_z;
+         return scm_i_normbig (result_z);
        }
       else
        SCM_WRONG_TYPE_ARG (SCM_ARG2, n2);
@@ -1412,8 +1441,12 @@ SCM_DEFINE1 (scm_logxor, "logxor", scm_tc7_asubr,
 
 SCM_DEFINE (scm_logtest, "logtest", 2, 0, 0,
             (SCM j, SCM k),
+           "Test whether @var{j} and @var{k} have any 1 bits in common.\n"
+           "This is equivalent to @code{(not (zero? (logand j k)))}, but\n"
+           "without actually calculating the @code{logand}, just testing\n"
+           "for non-zero.\n"
+           "\n"
            "@lisp\n"
-           "(logtest j k) @equiv{} (not (zero? (logand j k)))\n\n"
            "(logtest #b0100 #b1011) @result{} #f\n"
            "(logtest #b0100 #b0111) @result{} #t\n"
            "@end lisp")
@@ -1480,8 +1513,10 @@ SCM_DEFINE (scm_logtest, "logtest", 2, 0, 0,
 
 SCM_DEFINE (scm_logbit_p, "logbit?", 2, 0, 0,
             (SCM index, SCM j),
+           "Test whether bit number @var{index} in @var{j} is set.\n"
+           "@var{index} starts from 0 for the least significant bit.\n"
+           "\n"
            "@lisp\n"
-           "(logbit? index j) @equiv{} (logtest (integer-expt 2 index) j)\n\n"
            "(logbit? 0 #b1101) @result{} #t\n"
            "(logbit? 1 #b1101) @result{} #f\n"
            "(logbit? 2 #b1101) @result{} #t\n"
@@ -1663,14 +1698,18 @@ SCM_DEFINE (scm_modulo_expt, "modulo-expt", 3, 0, 0,
 
 SCM_DEFINE (scm_integer_expt, "integer-expt", 2, 0, 0,
             (SCM n, SCM k),
-           "Return @var{n} raised to the non-negative integer exponent\n"
-           "@var{k}.\n"
+           "Return @var{n} raised to the power @var{k}.  @var{k} must be an\n"
+           "exact integer, @var{n} can be any number.\n"
+           "\n"
+           "Negative @var{k} is supported, and results in @math{1/n^abs(k)}\n"
+           "in the usual way.  @math{@var{n}^0} is 1, as usual, and that\n"
+           "includes @math{0^0} is 1.\n"
            "\n"
            "@lisp\n"
-           "(integer-expt 2 5)\n"
-           "   @result{} 32\n"
-           "(integer-expt -3 3)\n"
-           "   @result{} -27\n"
+           "(integer-expt 2 5)   @result{} 32\n"
+           "(integer-expt -3 3)  @result{} -27\n"
+           "(integer-expt 5 -3)  @result{} 1/125\n"
+           "(integer-expt 0 0)   @result{} 1\n"
            "@end lisp")
 #define FUNC_NAME s_scm_integer_expt
 {
@@ -1693,22 +1732,6 @@ SCM_DEFINE (scm_integer_expt, "integer-expt", 2, 0, 0,
       scm_remember_upto_here_1 (k);
       i2_is_big = 1;
     }
-  else if (SCM_REALP (k))
-    {
-      double r = SCM_REAL_VALUE (k);
-      if (floor (r) != r)
-        SCM_WRONG_TYPE_ARG (2, k);
-      if ((r > SCM_MOST_POSITIVE_FIXNUM) || (r < SCM_MOST_NEGATIVE_FIXNUM))
-        {
-          z_i2 = scm_i_mkbig ();
-          mpz_set_d (SCM_I_BIG_MPZ (z_i2), r);
-          i2_is_big = 1;
-        }
-      else
-        {
-          i2 = r;
-        }
-    }
   else
     SCM_WRONG_TYPE_ARG (2, k);
   
@@ -1783,25 +1806,76 @@ SCM_DEFINE (scm_ash, "ash", 2, 0, 0,
   long bits_to_shift;
   bits_to_shift = scm_to_long (cnt);
 
-  if (bits_to_shift < 0)
+  if (SCM_I_INUMP (n))
     {
-      /* Shift right by abs(cnt) bits.  This is realized as a division
-         by div:=2^abs(cnt).  However, to guarantee the floor
-         rounding, negative values require some special treatment.
-      */
-      SCM div = scm_integer_expt (SCM_I_MAKINUM (2),
-                                  scm_from_long (-bits_to_shift));
+      long nn = SCM_I_INUM (n);
+
+      if (bits_to_shift > 0)
+        {
+          /* Left shift of bits_to_shift >= SCM_I_FIXNUM_BIT-1 will always
+             overflow a non-zero fixnum.  For smaller shifts we check the
+             bits going into positions above SCM_I_FIXNUM_BIT-1.  If they're
+             all 0s for nn>=0, or all 1s for nn<0 then there's no overflow.
+             Those bits are "nn >> (SCM_I_FIXNUM_BIT-1 -
+             bits_to_shift)".  */
+
+          if (nn == 0)
+            return n;
+
+          if (bits_to_shift < SCM_I_FIXNUM_BIT-1
+              && ((unsigned long)
+                  (SCM_SRS (nn, (SCM_I_FIXNUM_BIT-1 - bits_to_shift)) + 1)
+                  <= 1))
+            {
+              return SCM_I_MAKINUM (nn << bits_to_shift);
+            }
+          else
+            {
+              SCM result = scm_i_long2big (nn);
+              mpz_mul_2exp (SCM_I_BIG_MPZ (result), SCM_I_BIG_MPZ (result),
+                            bits_to_shift);
+              return result;
+            }
+        }
+      else
+        {
+          bits_to_shift = -bits_to_shift;
+          if (bits_to_shift >= SCM_LONG_BIT)
+            return (nn >= 0 ? SCM_I_MAKINUM (0) : SCM_I_MAKINUM(-1));
+          else
+            return SCM_I_MAKINUM (SCM_SRS (nn, bits_to_shift));
+        }
 
-      /* scm_quotient assumes its arguments are integers, but it's legal to (ash 1/2 -1) */
-      if (scm_is_false (scm_negative_p (n)))
-        return scm_quotient (n, div);
+    }
+  else if (SCM_BIGP (n))
+    {
+      SCM result;
+
+      if (bits_to_shift == 0)
+        return n;
+
+      result = scm_i_mkbig ();
+      if (bits_to_shift >= 0)
+        {
+          mpz_mul_2exp (SCM_I_BIG_MPZ (result), SCM_I_BIG_MPZ (n),
+                        bits_to_shift);
+          return result;
+        }
       else
-        return scm_sum (SCM_I_MAKINUM (-1L),
-                        scm_quotient (scm_sum (SCM_I_MAKINUM (1L), n), div));
+        {
+          /* GMP doesn't have an fdiv_q_2exp variant returning just a long, so
+             we have to allocate a bignum even if the result is going to be a
+             fixnum.  */
+          mpz_fdiv_q_2exp (SCM_I_BIG_MPZ (result), SCM_I_BIG_MPZ (n),
+                           -bits_to_shift);
+          return scm_i_normbig (result);
+        }
+
     }
   else
-    /* Shift left is done by multiplication with 2^CNT */
-    return scm_product (n, scm_integer_expt (SCM_I_MAKINUM (2), cnt));
+    {
+      SCM_WRONG_TYPE_ARG (SCM_ARG1, n);
+    }
 }
 #undef FUNC_NAME
 
@@ -2207,6 +2281,25 @@ idbl2str (double f, char *a, int radix)
   return ch;
 }
 
+
+static size_t
+icmplx2str (double real, double imag, char *str, int radix)
+{
+  size_t i;
+  
+  i = idbl2str (real, str, radix);
+  if (imag != 0.0)
+    {
+      /* Don't output a '+' for negative numbers or for Inf and
+        NaN.  They will provide their own sign. */
+      if (0 <= imag && !xisinf (imag) && !xisnan (imag))
+       str[i++] = '+';
+      i += idbl2str (imag, &str[i], radix);
+      str[i++] = 'i';
+    }
+  return i;
+}
+
 static size_t
 iflo2str (SCM flt, char *str, int radix)
 {
@@ -2214,45 +2307,43 @@ iflo2str (SCM flt, char *str, int radix)
   if (SCM_REALP (flt))
     i = idbl2str (SCM_REAL_VALUE (flt), str, radix);
   else
+    i = icmplx2str (SCM_COMPLEX_REAL (flt), SCM_COMPLEX_IMAG (flt),
+                   str, radix);
+  return i;
+}
+
+/* convert a scm_t_intmax to a string (unterminated).  returns the number of
+   characters in the result. 
+   rad is output base
+   p is destination: worst case (base 2) is SCM_INTBUFLEN  */
+size_t
+scm_iint2str (scm_t_intmax num, int rad, char *p)
+{
+  if (num < 0)
     {
-      i = idbl2str (SCM_COMPLEX_REAL (flt), str, radix);
-      if (SCM_COMPLEX_IMAG (flt) != 0.0)
-       {
-         double imag = SCM_COMPLEX_IMAG (flt);
-         /* Don't output a '+' for negative numbers or for Inf and
-            NaN.  They will provide their own sign. */
-         if (0 <= imag && !xisinf (imag) && !xisnan (imag))
-           str[i++] = '+';
-         i += idbl2str (imag, &str[i], radix);
-         str[i++] = 'i';
-       }
+      *p++ = '-';
+      return scm_iuint2str (-num, rad, p) + 1;
     }
-  return i;
+  else
+    return scm_iuint2str (num, rad, p);
 }
 
-/* convert a long to a string (unterminated).  returns the number of
+/* convert a scm_t_intmax to a string (unterminated).  returns the number of
    characters in the result. 
    rad is output base
    p is destination: worst case (base 2) is SCM_INTBUFLEN  */
 size_t
-scm_iint2str (long num, int rad, char *p)
+scm_iuint2str (scm_t_uintmax num, int rad, char *p)
 {
   size_t j = 1;
   size_t i;
-  unsigned long n = (num < 0) ? -num : num;
+  scm_t_uintmax n = num;
 
   for (n /= rad; n > 0; n /= rad)
     j++;
 
   i = j;
-  if (num < 0)
-    {
-      *p++ = '-';
-      j++;
-      n = -num;
-    }
-  else
-    n = num;
+  n = num;
   while (i--)
     {
       int d = n % rad;
@@ -2281,25 +2372,24 @@ SCM_DEFINE (scm_number_to_string, "number->string", 1, 1, 0,
     {
       char num_buf [SCM_INTBUFLEN];
       size_t length = scm_iint2str (SCM_I_INUM (n), base, num_buf);
-      return scm_mem2string (num_buf, length);
+      return scm_from_locale_stringn (num_buf, length);
     }
   else if (SCM_BIGP (n))
     {
       char *str = mpz_get_str (NULL, base, SCM_I_BIG_MPZ (n));
       scm_remember_upto_here_1 (n);
-      return scm_take0str (str);
+      return scm_take_locale_string (str);
     }
   else if (SCM_FRACTIONP (n))
     {
-      scm_i_fraction_reduce (n);
       return scm_string_append (scm_list_3 (scm_number_to_string (SCM_FRACTION_NUMERATOR (n), radix),
-                                           scm_mem2string ("/", 1), 
+                                           scm_from_locale_string ("/"), 
                                            scm_number_to_string (SCM_FRACTION_DENOMINATOR (n), radix)));
     }
   else if (SCM_INEXACTP (n))
     {
       char num_buf [FLOBUFLEN];
-       return scm_mem2string (num_buf, iflo2str (n, num_buf, base));
+      return scm_from_locale_stringn (num_buf, iflo2str (n, num_buf, base));
     }
   else
     SCM_WRONG_TYPE_ARG (1, n);
@@ -2318,6 +2408,13 @@ scm_print_real (SCM sexp, SCM port, scm_print_state *pstate SCM_UNUSED)
   return !0;
 }
 
+void
+scm_i_print_double (double val, SCM port)
+{
+  char num_buf[FLOBUFLEN];
+  scm_lfwrite (num_buf, idbl2str (val, num_buf, 10), port);
+}
+
 int
 scm_print_complex (SCM sexp, SCM port, scm_print_state *pstate SCM_UNUSED)
 
@@ -2327,13 +2424,19 @@ scm_print_complex (SCM sexp, SCM port, scm_print_state *pstate SCM_UNUSED)
   return !0;
 }
 
+void
+scm_i_print_complex (double real, double imag, SCM port)
+{
+  char num_buf[FLOBUFLEN];
+  scm_lfwrite (num_buf, icmplx2str (real, imag, num_buf, 10), port);
+}
+
 int
 scm_i_print_fraction (SCM sexp, SCM port, scm_print_state *pstate SCM_UNUSED)
 {
   SCM str;
-  scm_i_fraction_reduce (sexp);
   str = scm_number_to_string (sexp, SCM_UNDEFINED);
-  scm_lfwrite (SCM_STRING_CHARS (str), SCM_STRING_LENGTH (str), port);
+  scm_lfwrite (scm_i_string_chars (str), scm_i_string_length (str), port);
   scm_remember_upto_here_1 (str);
   return !0;
 }
@@ -2591,7 +2694,7 @@ mem2decimal_from_point (SCM result, const char* mem, size_t len,
          if (exponent > SCM_MAXEXP)
            {
              size_t exp_len = idx - start;
-             SCM exp_string = scm_mem2string (&mem[start], exp_len);
+             SCM exp_string = scm_from_locale_stringn (&mem[start], exp_len);
              SCM exp_num = scm_string_to_number (exp_string, SCM_UNDEFINED);
              scm_out_of_range ("string->number", exp_num);
            }
@@ -2684,7 +2787,7 @@ mem2ureal (const char* mem, size_t len, unsigned int *p_idx,
            return SCM_BOOL_F;
 
          /* both are int/big here, I assume */
-         result = scm_make_ratio (uinteger, divisor);
+         result = scm_i_make_ratio (uinteger, divisor);
        }
       else if (radix == 10)
        {
@@ -2856,7 +2959,8 @@ mem2complex (const char* mem, size_t len, unsigned int idx,
 enum t_radix {NO_RADIX=0, DUAL=2, OCT=8, DEC=10, HEX=16};
 
 SCM
-scm_i_mem2number (const char* mem, size_t len, unsigned int default_radix)
+scm_c_locale_stringn_to_number (const char* mem, size_t len,
+                               unsigned int default_radix)
 {
   unsigned int idx = 0;
   unsigned int radix = NO_RADIX;
@@ -2962,10 +3066,11 @@ SCM_DEFINE (scm_string_to_number, "string->number", 1, 1, 0,
   else
     base = scm_to_unsigned_integer (radix, 2, INT_MAX);
 
-  answer = scm_i_mem2number (SCM_STRING_CHARS (string),
-                            SCM_STRING_LENGTH (string),
-                            base);
-  return scm_return_first (answer, string);
+  answer = scm_c_locale_stringn_to_number (scm_i_string_chars (string),
+                                          scm_i_string_length (string),
+                                          base);
+  scm_remember_upto_here_1 (string);
+  return answer;
 }
 #undef FUNC_NAME
 
@@ -2973,23 +3078,6 @@ SCM_DEFINE (scm_string_to_number, "string->number", 1, 1, 0,
 /*** END strs->nums ***/
 
 
-SCM
-scm_make_complex (double x, double y)
-{
-  if (y == 0.0)
-    return scm_from_double (x);
-  else
-    {
-      SCM z;
-      SCM_NEWSMOB (z, scm_tc16_complex, scm_gc_malloc (sizeof (scm_t_complex),
-                                                      "complex"));
-      SCM_COMPLEX_REAL (z) = x;
-      SCM_COMPLEX_IMAG (z) = y;
-      return z;
-    }
-}
-
-
 SCM
 scm_bigequal (SCM x, SCM y)
 {
@@ -3014,8 +3102,6 @@ scm_complex_equalp (SCM x, SCM y)
 SCM
 scm_i_fraction_equalp (SCM x, SCM y)
 {
-  scm_i_fraction_reduce (x);
-  scm_i_fraction_reduce (y);
   if (scm_is_false (scm_equal_p (SCM_FRACTION_NUMERATOR (x),
                               SCM_FRACTION_NUMERATOR (y)))
       || scm_is_false (scm_equal_p (SCM_FRACTION_DENOMINATOR (x),
@@ -3026,26 +3112,30 @@ scm_i_fraction_equalp (SCM x, SCM y)
 }
 
 
-SCM_REGISTER_PROC (s_number_p, "number?", 1, 0, 0, scm_number_p);
-/* "Return @code{#t} if @var{x} is a number, @code{#f}\n"
- * "else.  Note that the sets of complex, real, rational and\n"
- * "integer values form subsets of the set of numbers, i. e. the\n"
- * "predicate will be fulfilled for any number."
- */
-SCM_DEFINE (scm_number_p, "complex?", 1, 0, 0, 
+SCM_DEFINE (scm_number_p, "number?", 1, 0, 0, 
+            (SCM x),
+           "Return @code{#t} if @var{x} is a number, @code{#f}\n"
+           "otherwise.")
+#define FUNC_NAME s_scm_number_p
+{
+  return scm_from_bool (SCM_NUMBERP (x));
+}
+#undef FUNC_NAME
+
+SCM_DEFINE (scm_complex_p, "complex?", 1, 0, 0, 
             (SCM x),
            "Return @code{#t} if @var{x} is a complex number, @code{#f}\n"
            "otherwise.  Note that the sets of real, rational and integer\n"
            "values form subsets of the set of complex numbers, i. e. the\n"
            "predicate will also be fulfilled if @var{x} is a real,\n"
            "rational or integer number.")
-#define FUNC_NAME s_scm_number_p
+#define FUNC_NAME s_scm_complex_p
 {
-  return scm_from_bool (SCM_NUMBERP (x));
+  /* all numbers are complex. */
+  return scm_number_p (x);
 }
 #undef FUNC_NAME
 
-
 SCM_DEFINE (scm_real_p, "real?", 1, 0, 0, 
             (SCM x),
            "Return @code{#t} if @var{x} is a real number, @code{#f}\n"
@@ -3084,7 +3174,6 @@ SCM_DEFINE (scm_rational_p, "rational?", 1, 0, 0,
 }
 #undef FUNC_NAME
 
-
 SCM_DEFINE (scm_integer_p, "integer?", 1, 0, 0, 
             (SCM x),
            "Return @code{#t} if @var{x} is an integer number, @code{#f}\n"
@@ -3103,6 +3192,7 @@ SCM_DEFINE (scm_integer_p, "integer?", 1, 0, 0,
   if (SCM_COMPLEXP (x))
     return SCM_BOOL_F;
   r = SCM_REAL_VALUE (x);
+  /* +/-inf passes r==floor(r), making those #t */
   if (r == floor (r))
     return SCM_BOOL_T;
   return SCM_BOOL_F;
@@ -3142,7 +3232,27 @@ scm_num_eq_p (SCM x, SCM y)
       else if (SCM_BIGP (y))
        return SCM_BOOL_F;
       else if (SCM_REALP (y))
-       return scm_from_bool ((double) xx == SCM_REAL_VALUE (y));
+        {
+          /* On a 32-bit system an inum fits a double, we can cast the inum
+             to a double and compare.
+
+             But on a 64-bit system an inum is bigger than a double and
+             casting it to a double (call that dxx) will round.  dxx is at
+             worst 1 bigger or smaller than xx, so if dxx==yy we know yy is
+             an integer and fits a long.  So we cast yy to a long and
+             compare with plain xx.
+
+             An alternative (for any size system actually) would be to check
+             yy is an integer (with floor) and is in range of an inum
+             (compare against appropriate powers of 2) then test
+             xx==(long)yy.  It's just a matter of which casts/comparisons
+             might be fastest or easiest for the cpu.  */
+
+          double yy = SCM_REAL_VALUE (y);
+          return scm_from_bool ((double) xx == yy
+                               && (DBL_MANT_DIG >= SCM_I_FIXNUM_BIT-1
+                                   || xx == (long) yy));
+        }
       else if (SCM_COMPLEXP (y))
        return scm_from_bool (((double) xx == SCM_COMPLEX_REAL (y))
                         && (0.0 == SCM_COMPLEX_IMAG (y)));
@@ -3188,8 +3298,15 @@ scm_num_eq_p (SCM x, SCM y)
     }
   else if (SCM_REALP (x))
     {
+      double xx = SCM_REAL_VALUE (x);
       if (SCM_I_INUMP (y))
-       return scm_from_bool (SCM_REAL_VALUE (x) == (double) SCM_I_INUM (y));
+        {
+          /* see comments with inum/real above */
+          long yy = SCM_I_INUM (y);
+          return scm_from_bool (xx == (double) yy
+                               && (DBL_MANT_DIG >= SCM_I_FIXNUM_BIT-1
+                                   || (long) xx == yy));
+        }
       else if (SCM_BIGP (y))
        {
          int cmp;
@@ -3835,16 +3952,16 @@ SCM_GPROC1 (s_sum, "+", scm_tc7_asubr, scm_sum, g_sum);
 SCM
 scm_sum (SCM x, SCM y)
 {
-  if (SCM_UNBNDP (y))
+  if (SCM_UNLIKELY (SCM_UNBNDP (y)))
     {
       if (SCM_NUMBERP (x)) return x;
       if (SCM_UNBNDP (x)) return SCM_INUM0;
       SCM_WTA_DISPATCH_1 (g_sum, x, SCM_ARG1, s_sum);
     }
 
-  if (SCM_I_INUMP (x))
+  if (SCM_LIKELY (SCM_I_INUMP (x)))
     {
-      if (SCM_I_INUMP (y))
+      if (SCM_LIKELY (SCM_I_INUMP (y)))
         {
           long xx = SCM_I_INUM (x);
           long yy = SCM_I_INUM (y);
@@ -3864,11 +3981,11 @@ scm_sum (SCM x, SCM y)
       else if (SCM_COMPLEXP (y))
         {
           long int xx = SCM_I_INUM (x);
-          return scm_make_complex (xx + SCM_COMPLEX_REAL (y),
+          return scm_c_make_rectangular (xx + SCM_COMPLEX_REAL (y),
                                    SCM_COMPLEX_IMAG (y));
         }
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (y), 
+       return scm_i_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (y), 
                                        scm_product (x, SCM_FRACTION_DENOMINATOR (y))),
                               SCM_FRACTION_DENOMINATOR (y));
       else
@@ -3930,10 +4047,10 @@ scm_sum (SCM x, SCM y)
            double real_part = (mpz_get_d (SCM_I_BIG_MPZ (x))
                                + SCM_COMPLEX_REAL (y));
            scm_remember_upto_here_1 (x);
-           return scm_make_complex (real_part, SCM_COMPLEX_IMAG (y));
+           return scm_c_make_rectangular (real_part, SCM_COMPLEX_IMAG (y));
          }
        else if (SCM_FRACTIONP (y))
-         return scm_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (y), 
+         return scm_i_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (y), 
                                          scm_product (x, SCM_FRACTION_DENOMINATOR (y))),
                                 SCM_FRACTION_DENOMINATOR (y));
        else
@@ -3952,7 +4069,7 @@ scm_sum (SCM x, SCM y)
       else if (SCM_REALP (y))
        return scm_from_double (SCM_REAL_VALUE (x) + SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_REAL_VALUE (x) + SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (SCM_REAL_VALUE (x) + SCM_COMPLEX_REAL (y),
                                 SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
        return scm_from_double (SCM_REAL_VALUE (x) + scm_i_fraction2double (y));
@@ -3962,23 +4079,23 @@ scm_sum (SCM x, SCM y)
   else if (SCM_COMPLEXP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) + SCM_I_INUM (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) + SCM_I_INUM (y),
                                 SCM_COMPLEX_IMAG (x));
       else if (SCM_BIGP (y))
        {
          double real_part = (mpz_get_d (SCM_I_BIG_MPZ (y))
                              + SCM_COMPLEX_REAL (x));
          scm_remember_upto_here_1 (y);
-         return scm_make_complex (real_part, SCM_COMPLEX_IMAG (x));
+         return scm_c_make_rectangular (real_part, SCM_COMPLEX_IMAG (x));
        }
       else if (SCM_REALP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) + SCM_REAL_VALUE (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) + SCM_REAL_VALUE (y),
                                 SCM_COMPLEX_IMAG (x));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) + SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) + SCM_COMPLEX_REAL (y),
                                 SCM_COMPLEX_IMAG (x) + SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) + scm_i_fraction2double (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) + scm_i_fraction2double (y),
                                 SCM_COMPLEX_IMAG (x));
       else
        SCM_WTA_DISPATCH_2 (g_sum, x, y, SCM_ARGn, s_sum);
@@ -3986,21 +4103,21 @@ scm_sum (SCM x, SCM y)
   else if (SCM_FRACTIONP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (x), 
+       return scm_i_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (x), 
                                        scm_product (y, SCM_FRACTION_DENOMINATOR (x))),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_BIGP (y))
-       return scm_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (x), 
+       return scm_i_make_ratio (scm_sum (SCM_FRACTION_NUMERATOR (x), 
                                        scm_product (y, SCM_FRACTION_DENOMINATOR (x))),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_REALP (y))
        return scm_from_double (SCM_REAL_VALUE (y) + scm_i_fraction2double (x));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (y) + scm_i_fraction2double (x),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (y) + scm_i_fraction2double (x),
                                 SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
        /* a/b + c/d = (ad + bc) / bd */
-       return scm_make_ratio (scm_sum (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_sum (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
                                        scm_product (SCM_FRACTION_NUMERATOR (y), SCM_FRACTION_DENOMINATOR (x))),
                               scm_product (SCM_FRACTION_DENOMINATOR (x), SCM_FRACTION_DENOMINATOR (y)));
       else
@@ -4011,6 +4128,16 @@ scm_sum (SCM x, SCM y)
 }
 
 
+SCM_DEFINE (scm_oneplus, "1+", 1, 0, 0, 
+            (SCM x),
+           "Return @math{@var{x}+1}.")
+#define FUNC_NAME s_scm_oneplus
+{
+  return scm_sum (x, SCM_I_MAKINUM (1));
+}
+#undef FUNC_NAME
+
+
 SCM_GPROC1 (s_difference, "-", scm_tc7_asubr, scm_difference, g_difference);
 /* If called with one argument @var{z1}, -@var{z1} returned. Otherwise
  * the sum of all but the first argument are subtracted from the first
@@ -4019,7 +4146,7 @@ SCM_GPROC1 (s_difference, "-", scm_tc7_asubr, scm_difference, g_difference);
 SCM
 scm_difference (SCM x, SCM y)
 {
-  if (SCM_UNBNDP (y))
+  if (SCM_UNLIKELY (SCM_UNBNDP (y)))
     {
       if (SCM_UNBNDP (x))
         SCM_WTA_DISPATCH_0 (g_difference, s_difference);
@@ -4033,23 +4160,24 @@ scm_difference (SCM x, SCM y)
               return scm_i_long2big (xx);
           }
         else if (SCM_BIGP (x))
-          /* FIXME: do we really need to normalize here? */
+          /* Must scm_i_normbig here because -SCM_MOST_NEGATIVE_FIXNUM is a
+             bignum, but negating that gives a fixnum.  */
           return scm_i_normbig (scm_i_clonebig (x, 0));
         else if (SCM_REALP (x))
           return scm_from_double (-SCM_REAL_VALUE (x));
         else if (SCM_COMPLEXP (x))
-          return scm_make_complex (-SCM_COMPLEX_REAL (x),
+          return scm_c_make_rectangular (-SCM_COMPLEX_REAL (x),
                                    -SCM_COMPLEX_IMAG (x));
        else if (SCM_FRACTIONP (x))
-         return scm_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), SCM_UNDEFINED),
+         return scm_i_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), SCM_UNDEFINED),
                                 SCM_FRACTION_DENOMINATOR (x));
         else
           SCM_WTA_DISPATCH_1 (g_difference, x, SCM_ARG1, s_difference);
     }
   
-  if (SCM_I_INUMP (x))
+  if (SCM_LIKELY (SCM_I_INUMP (x)))
     {
-      if (SCM_I_INUMP (y))
+      if (SCM_LIKELY (SCM_I_INUMP (y)))
        {
          long int xx = SCM_I_INUM (x);
          long int yy = SCM_I_INUM (y);
@@ -4096,12 +4224,12 @@ scm_difference (SCM x, SCM y)
       else if (SCM_COMPLEXP (y))
        {
          long int xx = SCM_I_INUM (x);
-         return scm_make_complex (xx - SCM_COMPLEX_REAL (y),
+         return scm_c_make_rectangular (xx - SCM_COMPLEX_REAL (y),
                                   - SCM_COMPLEX_IMAG (y));
        }
       else if (SCM_FRACTIONP (y))
        /* a - b/c = (ac - b) / c */
-       return scm_make_ratio (scm_difference (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_difference (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
                                               SCM_FRACTION_NUMERATOR (y)),
                               SCM_FRACTION_DENOMINATOR (y));
       else
@@ -4163,10 +4291,10 @@ scm_difference (SCM x, SCM y)
          double real_part = (mpz_get_d (SCM_I_BIG_MPZ (x))
                              - SCM_COMPLEX_REAL (y));
          scm_remember_upto_here_1 (x);
-         return scm_make_complex (real_part, - SCM_COMPLEX_IMAG (y));      
+         return scm_c_make_rectangular (real_part, - SCM_COMPLEX_IMAG (y));      
        }
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_difference (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_difference (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
                                               SCM_FRACTION_NUMERATOR (y)),
                               SCM_FRACTION_DENOMINATOR (y));
       else SCM_WTA_DISPATCH_2 (g_difference, x, y, SCM_ARGn, s_difference);
@@ -4184,7 +4312,7 @@ scm_difference (SCM x, SCM y)
       else if (SCM_REALP (y))
        return scm_from_double (SCM_REAL_VALUE (x) - SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_REAL_VALUE (x) - SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (SCM_REAL_VALUE (x) - SCM_COMPLEX_REAL (y),
                                 -SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
        return scm_from_double (SCM_REAL_VALUE (x) - scm_i_fraction2double (y));
@@ -4194,23 +4322,23 @@ scm_difference (SCM x, SCM y)
   else if (SCM_COMPLEXP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) - SCM_I_INUM (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) - SCM_I_INUM (y),
                                 SCM_COMPLEX_IMAG (x));
       else if (SCM_BIGP (y))
        {
          double real_part = (SCM_COMPLEX_REAL (x)
                              - mpz_get_d (SCM_I_BIG_MPZ (y)));
          scm_remember_upto_here_1 (x);
-         return scm_make_complex (real_part, SCM_COMPLEX_IMAG (y));      
+         return scm_c_make_rectangular (real_part, SCM_COMPLEX_IMAG (y));      
        }
       else if (SCM_REALP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) - SCM_REAL_VALUE (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) - SCM_REAL_VALUE (y),
                                 SCM_COMPLEX_IMAG (x));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) - SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) - SCM_COMPLEX_REAL (y),
                                 SCM_COMPLEX_IMAG (x) - SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
-       return scm_make_complex (SCM_COMPLEX_REAL (x) - scm_i_fraction2double (y),
+       return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) - scm_i_fraction2double (y),
                                 SCM_COMPLEX_IMAG (x));
       else
        SCM_WTA_DISPATCH_2 (g_difference, x, y, SCM_ARGn, s_difference);
@@ -4219,21 +4347,21 @@ scm_difference (SCM x, SCM y)
     {
       if (SCM_I_INUMP (y))
        /* a/b - c = (a - cb) / b */
-       return scm_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), 
+       return scm_i_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), 
                                               scm_product(y, SCM_FRACTION_DENOMINATOR (x))),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_BIGP (y))
-       return scm_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), 
+       return scm_i_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (x), 
                                               scm_product(y, SCM_FRACTION_DENOMINATOR (x))),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_REALP (y))
        return scm_from_double (scm_i_fraction2double (x) - SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (scm_i_fraction2double (x) - SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (scm_i_fraction2double (x) - SCM_COMPLEX_REAL (y),
                                 -SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
        /* a/b - c/d = (ad - bc) / bd */
-       return scm_make_ratio (scm_difference (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_difference (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
                                               scm_product (SCM_FRACTION_NUMERATOR (y), SCM_FRACTION_DENOMINATOR (x))),
                               scm_product (SCM_FRACTION_DENOMINATOR (x), SCM_FRACTION_DENOMINATOR (y)));
       else
@@ -4245,6 +4373,16 @@ scm_difference (SCM x, SCM y)
 #undef FUNC_NAME
 
 
+SCM_DEFINE (scm_oneminus, "1-", 1, 0, 0, 
+            (SCM x),
+           "Return @math{@var{x}-1}.")
+#define FUNC_NAME s_scm_oneminus
+{
+  return scm_difference (x, SCM_I_MAKINUM (1));
+}
+#undef FUNC_NAME
+
+
 SCM_GPROC1 (s_product, "*", scm_tc7_asubr, scm_product, g_product);
 /* "Return the product of all arguments.  If called without arguments,\n"
  * "1 is returned."
@@ -4252,7 +4390,7 @@ SCM_GPROC1 (s_product, "*", scm_tc7_asubr, scm_product, g_product);
 SCM
 scm_product (SCM x, SCM y)
 {
-  if (SCM_UNBNDP (y))
+  if (SCM_UNLIKELY (SCM_UNBNDP (y)))
     {
       if (SCM_UNBNDP (x))
        return SCM_I_MAKINUM (1L);
@@ -4262,7 +4400,7 @@ scm_product (SCM x, SCM y)
        SCM_WTA_DISPATCH_1 (g_product, x, SCM_ARG1, s_product);
     }
   
-  if (SCM_I_INUMP (x))
+  if (SCM_LIKELY (SCM_I_INUMP (x)))
     {
       long xx;
 
@@ -4275,7 +4413,7 @@ scm_product (SCM x, SCM y)
         case 1: return y; break;
        }
 
-      if (SCM_I_INUMP (y))
+      if (SCM_LIKELY (SCM_I_INUMP (y)))
        {
          long yy = SCM_I_INUM (y);
          long kk = xx * yy;
@@ -4299,10 +4437,10 @@ scm_product (SCM x, SCM y)
       else if (SCM_REALP (y))
        return scm_from_double (xx * SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (xx * SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (xx * SCM_COMPLEX_REAL (y),
                                 xx * SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_product (x, SCM_FRACTION_NUMERATOR (y)),
+       return scm_i_make_ratio (scm_product (x, SCM_FRACTION_NUMERATOR (y)),
                               SCM_FRACTION_DENOMINATOR (y));
       else
        SCM_WTA_DISPATCH_2 (g_product, x, y, SCM_ARGn, s_product);
@@ -4333,11 +4471,11 @@ scm_product (SCM x, SCM y)
        {
          double z = mpz_get_d (SCM_I_BIG_MPZ (x));
          scm_remember_upto_here_1 (x);
-         return scm_make_complex (z * SCM_COMPLEX_REAL (y),
+         return scm_c_make_rectangular (z * SCM_COMPLEX_REAL (y),
                                   z * SCM_COMPLEX_IMAG (y));
        }
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_product (x, SCM_FRACTION_NUMERATOR (y)),
+       return scm_i_make_ratio (scm_product (x, SCM_FRACTION_NUMERATOR (y)),
                               SCM_FRACTION_DENOMINATOR (y));
       else
        SCM_WTA_DISPATCH_2 (g_product, x, y, SCM_ARGn, s_product);
@@ -4345,7 +4483,12 @@ scm_product (SCM x, SCM y)
   else if (SCM_REALP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_from_double (SCM_I_INUM (y) * SCM_REAL_VALUE (x));
+        {
+          /* inexact*exact0 => exact 0, per R5RS "Exactness" section */
+          if (scm_is_eq (y, SCM_INUM0))
+            return y;
+          return scm_from_double (SCM_I_INUM (y) * SCM_REAL_VALUE (x));
+        }
       else if (SCM_BIGP (y))
        {
          double result = mpz_get_d (SCM_I_BIG_MPZ (y)) * SCM_REAL_VALUE (x);
@@ -4355,7 +4498,7 @@ scm_product (SCM x, SCM y)
       else if (SCM_REALP (y))
        return scm_from_double (SCM_REAL_VALUE (x) * SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
-       return scm_make_complex (SCM_REAL_VALUE (x) * SCM_COMPLEX_REAL (y),
+       return scm_c_make_rectangular (SCM_REAL_VALUE (x) * SCM_COMPLEX_REAL (y),
                                 SCM_REAL_VALUE (x) * SCM_COMPLEX_IMAG (y));
       else if (SCM_FRACTIONP (y))
        return scm_from_double (SCM_REAL_VALUE (x) * scm_i_fraction2double (y));
@@ -4365,21 +4508,26 @@ scm_product (SCM x, SCM y)
   else if (SCM_COMPLEXP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_make_complex (SCM_I_INUM (y) * SCM_COMPLEX_REAL (x),
-                                SCM_I_INUM (y) * SCM_COMPLEX_IMAG (x));
+        {
+          /* inexact*exact0 => exact 0, per R5RS "Exactness" section */
+          if (scm_is_eq (y, SCM_INUM0))
+            return y;
+          return scm_c_make_rectangular (SCM_I_INUM (y) * SCM_COMPLEX_REAL (x),
+                                         SCM_I_INUM (y) * SCM_COMPLEX_IMAG (x));
+        }
       else if (SCM_BIGP (y))
        {
          double z = mpz_get_d (SCM_I_BIG_MPZ (y));
          scm_remember_upto_here_1 (y);
-         return scm_make_complex (z * SCM_COMPLEX_REAL (x),
+         return scm_c_make_rectangular (z * SCM_COMPLEX_REAL (x),
                                   z * SCM_COMPLEX_IMAG (x));
        }
       else if (SCM_REALP (y))
-       return scm_make_complex (SCM_REAL_VALUE (y) * SCM_COMPLEX_REAL (x),
+       return scm_c_make_rectangular (SCM_REAL_VALUE (y) * SCM_COMPLEX_REAL (x),
                                 SCM_REAL_VALUE (y) * SCM_COMPLEX_IMAG (x));
       else if (SCM_COMPLEXP (y))
        {
-         return scm_make_complex (SCM_COMPLEX_REAL (x) * SCM_COMPLEX_REAL (y)
+         return scm_c_make_rectangular (SCM_COMPLEX_REAL (x) * SCM_COMPLEX_REAL (y)
                                   - SCM_COMPLEX_IMAG (x) * SCM_COMPLEX_IMAG (y),
                                   SCM_COMPLEX_REAL (x) * SCM_COMPLEX_IMAG (y)
                                   + SCM_COMPLEX_IMAG (x) * SCM_COMPLEX_REAL (y));
@@ -4387,7 +4535,7 @@ scm_product (SCM x, SCM y)
       else if (SCM_FRACTIONP (y))
        {
          double yy = scm_i_fraction2double (y);
-         return scm_make_complex (yy * SCM_COMPLEX_REAL (x),
+         return scm_c_make_rectangular (yy * SCM_COMPLEX_REAL (x),
                                   yy * SCM_COMPLEX_IMAG (x));
        }
       else
@@ -4396,22 +4544,22 @@ scm_product (SCM x, SCM y)
   else if (SCM_FRACTIONP (x))
     {
       if (SCM_I_INUMP (y))
-       return scm_make_ratio (scm_product (y, SCM_FRACTION_NUMERATOR (x)),
+       return scm_i_make_ratio (scm_product (y, SCM_FRACTION_NUMERATOR (x)),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_BIGP (y))
-       return scm_make_ratio (scm_product (y, SCM_FRACTION_NUMERATOR (x)),
+       return scm_i_make_ratio (scm_product (y, SCM_FRACTION_NUMERATOR (x)),
                               SCM_FRACTION_DENOMINATOR (x));
       else if (SCM_REALP (y))
        return scm_from_double (scm_i_fraction2double (x) * SCM_REAL_VALUE (y));
       else if (SCM_COMPLEXP (y))
        {
          double xx = scm_i_fraction2double (x);
-         return scm_make_complex (xx * SCM_COMPLEX_REAL (y),
+         return scm_c_make_rectangular (xx * SCM_COMPLEX_REAL (y),
                                   xx * SCM_COMPLEX_IMAG (y));
        }
       else if (SCM_FRACTIONP (y))
        /* a/b * c/d = ac / bd */
-       return scm_make_ratio (scm_product (SCM_FRACTION_NUMERATOR (x),
+       return scm_i_make_ratio (scm_product (SCM_FRACTION_NUMERATOR (x),
                                            SCM_FRACTION_NUMERATOR (y)),
                               scm_product (SCM_FRACTION_DENOMINATOR (x),
                                            SCM_FRACTION_DENOMINATOR (y)));
@@ -4465,7 +4613,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
 {
   double a;
 
-  if (SCM_UNBNDP (y))
+  if (SCM_UNLIKELY (SCM_UNBNDP (y)))
     {
       if (SCM_UNBNDP (x))
        SCM_WTA_DISPATCH_0 (g_divide, s_divide);
@@ -4482,14 +4630,14 @@ scm_i_divide (SCM x, SCM y, int inexact)
            {
              if (inexact)
                return scm_from_double (1.0 / (double) xx);
-             else return scm_make_ratio (SCM_I_MAKINUM(1), x);
+             else return scm_i_make_ratio (SCM_I_MAKINUM(1), x);
            }
        }
       else if (SCM_BIGP (x))
        {
          if (inexact)
            return scm_from_double (1.0 / scm_i_big2dbl (x));
-         else return scm_make_ratio (SCM_I_MAKINUM(1), x);
+         else return scm_i_make_ratio (SCM_I_MAKINUM(1), x);
        }
       else if (SCM_REALP (x))
        {
@@ -4505,30 +4653,30 @@ scm_i_divide (SCM x, SCM y, int inexact)
        {
          double r = SCM_COMPLEX_REAL (x);
          double i = SCM_COMPLEX_IMAG (x);
-         if (r <= i)
+         if (fabs(r) <= fabs(i))
            {
              double t = r / i;
              double d = i * (1.0 + t * t);
-             return scm_make_complex (t / d, -1.0 / d);
+             return scm_c_make_rectangular (t / d, -1.0 / d);
            }
          else
            {
              double t = i / r;
              double d = r * (1.0 + t * t);
-             return scm_make_complex (1.0 / d, -t / d);
+             return scm_c_make_rectangular (1.0 / d, -t / d);
            }
        }
       else if (SCM_FRACTIONP (x))
-       return scm_make_ratio (SCM_FRACTION_DENOMINATOR (x),
+       return scm_i_make_ratio (SCM_FRACTION_DENOMINATOR (x),
                               SCM_FRACTION_NUMERATOR (x));
       else
        SCM_WTA_DISPATCH_1 (g_divide, x, SCM_ARG1, s_divide);
     }
 
-  if (SCM_I_INUMP (x))
+  if (SCM_LIKELY (SCM_I_INUMP (x)))
     {
       long xx = SCM_I_INUM (x);
-      if (SCM_I_INUMP (y))
+      if (SCM_LIKELY (SCM_I_INUMP (y)))
        {
          long yy = SCM_I_INUM (y);
          if (yy == 0)
@@ -4543,7 +4691,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
            {
              if (inexact)
                return scm_from_double ((double) xx / (double) yy);
-             else return scm_make_ratio (x, y);
+             else return scm_i_make_ratio (x, y);
            }
          else
            {
@@ -4558,7 +4706,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
        {
          if (inexact)
            return scm_from_double ((double) xx / scm_i_big2dbl (y));
-         else return scm_make_ratio (x, y);
+         else return scm_i_make_ratio (x, y);
        }
       else if (SCM_REALP (y))
        {
@@ -4577,23 +4725,23 @@ scm_i_divide (SCM x, SCM y, int inexact)
          {
            double r = SCM_COMPLEX_REAL (y);
            double i = SCM_COMPLEX_IMAG (y);
-           if (r <= i)
+           if (fabs(r) <= fabs(i))
              {
                double t = r / i;
                double d = i * (1.0 + t * t);
-               return scm_make_complex ((a * t) / d,  -a / d);
+               return scm_c_make_rectangular ((a * t) / d,  -a / d);
              }
            else
              {
                double t = i / r;
                double d = r * (1.0 + t * t);
-               return scm_make_complex (a / d,  -(a * t) / d);
+               return scm_c_make_rectangular (a / d,  -(a * t) / d);
              }
          }
        }
       else if (SCM_FRACTIONP (y))
        /* a / b/c = ac / b */
-       return scm_make_ratio (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
                               SCM_FRACTION_NUMERATOR (y));
       else
        SCM_WTA_DISPATCH_2 (g_divide, x, y, SCM_ARGn, s_divide);
@@ -4641,7 +4789,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
                {
                  if (inexact)
                    return scm_from_double (scm_i_big2dbl (x) / (double) yy);
-                 else return scm_make_ratio (x, y);
+                 else return scm_i_make_ratio (x, y);
                }
            }
        }
@@ -4661,28 +4809,33 @@ scm_i_divide (SCM x, SCM y, int inexact)
          else
            {
              /* big_x / big_y */
-             int divisible_p = mpz_divisible_p (SCM_I_BIG_MPZ (x),
-                                                SCM_I_BIG_MPZ (y));
-             if (divisible_p)
-               {
-                 SCM result = scm_i_mkbig ();
-                 mpz_divexact (SCM_I_BIG_MPZ (result),
-                               SCM_I_BIG_MPZ (x),
-                               SCM_I_BIG_MPZ (y));
-                 scm_remember_upto_here_2 (x, y);
-                 return scm_i_normbig (result);
-               }
-             else
-               {
-                 if (inexact)
-                   {
-                     double dbx = mpz_get_d (SCM_I_BIG_MPZ (x));
-                     double dby = mpz_get_d (SCM_I_BIG_MPZ (y));
-                     scm_remember_upto_here_2 (x, y);
-                     return scm_from_double (dbx / dby);
-                   }
-                 else return scm_make_ratio (x, y);
-               }
+              if (inexact)
+                {
+                  /* It's easily possible for the ratio x/y to fit a double
+                     but one or both x and y be too big to fit a double,
+                     hence the use of mpq_get_d rather than converting and
+                     dividing.  */
+                  mpq_t q;
+                  *mpq_numref(q) = *SCM_I_BIG_MPZ (x);
+                  *mpq_denref(q) = *SCM_I_BIG_MPZ (y);
+                  return scm_from_double (mpq_get_d (q));
+                }
+              else
+                {
+                  int divisible_p = mpz_divisible_p (SCM_I_BIG_MPZ (x),
+                                                     SCM_I_BIG_MPZ (y));
+                  if (divisible_p)
+                    {
+                      SCM result = scm_i_mkbig ();
+                      mpz_divexact (SCM_I_BIG_MPZ (result),
+                                    SCM_I_BIG_MPZ (x),
+                                    SCM_I_BIG_MPZ (y));
+                      scm_remember_upto_here_2 (x, y);
+                      return scm_i_normbig (result);
+                    }
+                  else
+                    return scm_i_make_ratio (x, y);
+                }
            }
        }
       else if (SCM_REALP (y))
@@ -4701,7 +4854,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
          goto complex_div;
        }
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_product (x, SCM_FRACTION_DENOMINATOR (y)),
                               SCM_FRACTION_NUMERATOR (y));
       else
        SCM_WTA_DISPATCH_2 (g_divide, x, y, SCM_ARGn, s_divide);
@@ -4759,14 +4912,14 @@ scm_i_divide (SCM x, SCM y, int inexact)
 #endif
            {
              double d = yy;
-             return scm_make_complex (rx / d, ix / d);
+             return scm_c_make_rectangular (rx / d, ix / d);
            }
        }
       else if (SCM_BIGP (y))
        {
          double dby = mpz_get_d (SCM_I_BIG_MPZ (y));
          scm_remember_upto_here_1 (y);
-         return scm_make_complex (rx / dby, ix / dby);
+         return scm_c_make_rectangular (rx / dby, ix / dby);
        }
       else if (SCM_REALP (y))
        {
@@ -4776,29 +4929,29 @@ scm_i_divide (SCM x, SCM y, int inexact)
            scm_num_overflow (s_divide);
          else
 #endif
-           return scm_make_complex (rx / yy, ix / yy);
+           return scm_c_make_rectangular (rx / yy, ix / yy);
        }
       else if (SCM_COMPLEXP (y))
        {
          double ry = SCM_COMPLEX_REAL (y);
          double iy = SCM_COMPLEX_IMAG (y);
-         if (ry <= iy)
+         if (fabs(ry) <= fabs(iy))
            {
              double t = ry / iy;
              double d = iy * (1.0 + t * t);
-             return scm_make_complex ((rx * t + ix) / d, (ix * t - rx) / d);
+             return scm_c_make_rectangular ((rx * t + ix) / d, (ix * t - rx) / d);
            }
          else
            {
              double t = iy / ry;
              double d = ry * (1.0 + t * t);
-             return scm_make_complex ((rx + ix * t) / d, (ix - rx * t) / d);
+             return scm_c_make_rectangular ((rx + ix * t) / d, (ix - rx * t) / d);
            }
        }
       else if (SCM_FRACTIONP (y))
        {
          double yy = scm_i_fraction2double (y);
-         return scm_make_complex (rx / yy, ix / yy);
+         return scm_c_make_rectangular (rx / yy, ix / yy);
        }
       else
        SCM_WTA_DISPATCH_2 (g_divide, x, y, SCM_ARGn, s_divide);
@@ -4813,12 +4966,12 @@ scm_i_divide (SCM x, SCM y, int inexact)
            scm_num_overflow (s_divide);
          else
 #endif
-           return scm_make_ratio (SCM_FRACTION_NUMERATOR (x),
+           return scm_i_make_ratio (SCM_FRACTION_NUMERATOR (x),
                                   scm_product (SCM_FRACTION_DENOMINATOR (x), y));
        } 
       else if (SCM_BIGP (y)) 
        {
-         return scm_make_ratio (SCM_FRACTION_NUMERATOR (x),
+         return scm_i_make_ratio (SCM_FRACTION_NUMERATOR (x),
                                 scm_product (SCM_FRACTION_DENOMINATOR (x), y));
        } 
       else if (SCM_REALP (y)) 
@@ -4837,7 +4990,7 @@ scm_i_divide (SCM x, SCM y, int inexact)
          goto complex_div;
        } 
       else if (SCM_FRACTIONP (y))
-       return scm_make_ratio (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
+       return scm_i_make_ratio (scm_product (SCM_FRACTION_NUMERATOR (x), SCM_FRACTION_DENOMINATOR (y)),
                               scm_product (SCM_FRACTION_NUMERATOR (y), SCM_FRACTION_DENOMINATOR (x)));
       else 
        SCM_WTA_DISPATCH_2 (g_divide, x, y, SCM_ARGn, s_divide);
@@ -4904,13 +5057,8 @@ SCM_GPROC1 (s_atanh, "$atanh", scm_tc7_dsubr, (SCM (*)()) atanh, g_atanh);
  */
 
 
-/* XXX - eventually, we should remove this definition of scm_round and
-   rename scm_round_number to scm_round.  Likewise for scm_truncate
-   and scm_truncate_number.
- */
-
 double
-scm_truncate (double x)
+scm_c_truncate (double x)
 {
 #if HAVE_TRUNC
   return trunc (x);
@@ -4921,10 +5069,10 @@ scm_truncate (double x)
 #endif
 }
 
-/* scm_round is done using floor(x+0.5) to round to nearest and with
-   half-way case (ie. when x is an integer plus 0.5) going upwards.  Then
-   half-way cases are identified and adjusted down if the round-upwards
-   didn't give the desired even integer.
+/* scm_c_round is done using floor(x+0.5) to round to nearest and with
+   half-way case (ie. when x is an integer plus 0.5) going upwards.
+   Then half-way cases are identified and adjusted down if the
+   round-upwards didn't give the desired even integer.
 
    "plus_half == result" identifies a half-way case.  If plus_half, which is
    x + 0.5, is an integer then x must be an integer plus 0.5.
@@ -4948,7 +5096,7 @@ scm_truncate (double x)
    an 0.5 to be represented, and hence added without a bad rounding.  */
 
 double
-scm_round (double x)
+scm_c_round (double x)
 {
   double plus_half, result;
 
@@ -4957,7 +5105,7 @@ scm_round (double x)
 
   plus_half = x + 0.5;
   result = floor (plus_half);
-  /* Adjust so that the scm_round is towards even.  */
+  /* Adjust so that the rounding is towards even.  */
   return ((plus_half == result && plus_half / 2 != floor (plus_half / 2))
          ? result - 1
          : result);
@@ -4987,7 +5135,7 @@ SCM_DEFINE (scm_round_number, "round", 1, 0, 0,
   if (SCM_I_INUMP (x) || SCM_BIGP (x))
     return x;
   else if (SCM_REALP (x))
-    return scm_from_double (scm_round (SCM_REAL_VALUE (x)));
+    return scm_from_double (scm_c_round (SCM_REAL_VALUE (x)));
   else
     {
       /* OPTIMIZE-ME: Fraction case could be done more efficiently by a
@@ -4995,7 +5143,7 @@ SCM_DEFINE (scm_round_number, "round", 1, 0, 0,
          the rounding should go.  */
       SCM plus_half = scm_sum (x, exactly_one_half);
       SCM result = scm_floor (plus_half);
-      /* Adjust so that the scm_round is towards even.  */
+      /* Adjust so that the rounding is towards even.  */
       if (scm_is_true (scm_num_eq_p (plus_half, result))
           && scm_is_true (scm_odd_p (result)))
         return scm_difference (result, SCM_I_MAKINUM (1));
@@ -5172,20 +5320,47 @@ SCM_DEFINE (scm_sys_atan2, "$atan2", 2, 0, 0,
 }
 #undef FUNC_NAME
 
+SCM
+scm_c_make_rectangular (double re, double im)
+{
+  if (im == 0.0)
+    return scm_from_double (re);
+  else
+    {
+      SCM z;
+      SCM_NEWSMOB (z, scm_tc16_complex,
+                  scm_gc_malloc_pointerless (sizeof (scm_t_complex),
+                                             "complex"));
+      SCM_COMPLEX_REAL (z) = re;
+      SCM_COMPLEX_IMAG (z) = im;
+      return z;
+    }
+}
 
 SCM_DEFINE (scm_make_rectangular, "make-rectangular", 2, 0, 0,
-            (SCM real, SCM imaginary),
-           "Return a complex number constructed of the given @var{real} and\n"
-           "@var{imaginary} parts.")
+            (SCM real_part, SCM imaginary_part),
+           "Return a complex number constructed of the given @var{real-part} "
+           "and @var{imaginary-part} parts.")
 #define FUNC_NAME s_scm_make_rectangular
 {
   struct dpair xy;
-  scm_two_doubles (real, imaginary, FUNC_NAME, &xy);
-  return scm_make_complex (xy.x, xy.y);
+  scm_two_doubles (real_part, imaginary_part, FUNC_NAME, &xy);
+  return scm_c_make_rectangular (xy.x, xy.y);
 }
 #undef FUNC_NAME
 
-
+SCM
+scm_c_make_polar (double mag, double ang)
+{
+  double s, c;
+#if HAVE_SINCOS
+  sincos (ang, &s, &c);
+#else
+  s = sin (ang);
+  c = cos (ang);
+#endif
+  return scm_c_make_rectangular (mag * c, mag * s);
+}
 
 SCM_DEFINE (scm_make_polar, "make-polar", 2, 0, 0,
             (SCM x, SCM y),
@@ -5193,15 +5368,8 @@ SCM_DEFINE (scm_make_polar, "make-polar", 2, 0, 0,
 #define FUNC_NAME s_scm_make_polar
 {
   struct dpair xy;
-  double s, c;
   scm_two_doubles (x, y, FUNC_NAME, &xy);
-#if HAVE_SINCOS
-  sincos (xy.y, &s, &c);
-#else
-  s = sin (xy.y);
-  c = cos (xy.y);
-#endif
-  return scm_make_complex (xy.x * c, xy.x * s);
+  return scm_c_make_polar (xy.x, xy.y);
 }
 #undef FUNC_NAME
 
@@ -5258,10 +5426,7 @@ scm_numerator (SCM z)
   else if (SCM_BIGP (z))
     return z;
   else if (SCM_FRACTIONP (z))
-    {
-      scm_i_fraction_reduce (z);
-      return SCM_FRACTION_NUMERATOR (z);
-    }
+    return SCM_FRACTION_NUMERATOR (z);
   else if (SCM_REALP (z))
     return scm_exact_to_inexact (scm_numerator (scm_inexact_to_exact (z)));
   else
@@ -5280,10 +5445,7 @@ scm_denominator (SCM z)
   else if (SCM_BIGP (z)) 
     return SCM_I_MAKINUM (1);
   else if (SCM_FRACTIONP (z))
-    {
-      scm_i_fraction_reduce (z);
-      return SCM_FRACTION_DENOMINATOR (z);
-    }
+    return SCM_FRACTION_DENOMINATOR (z);
   else if (SCM_REALP (z))
     return scm_exact_to_inexact (scm_denominator (scm_inexact_to_exact (z)));
   else
@@ -5324,7 +5486,7 @@ scm_magnitude (SCM z)
     {
       if (scm_is_false (scm_negative_p (SCM_FRACTION_NUMERATOR (z))))
        return z;
-      return scm_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (z), SCM_UNDEFINED),
+      return scm_i_make_ratio (scm_difference (SCM_FRACTION_NUMERATOR (z), SCM_UNDEFINED),
                             SCM_FRACTION_DENOMINATOR (z));
     }
   else
@@ -5417,10 +5579,10 @@ SCM_DEFINE (scm_inexact_to_exact, "inexact->exact", 1, 0, 0,
          
          mpq_init (frac);
          mpq_set_d (frac, SCM_REAL_VALUE (z));
-         q = scm_make_ratio (scm_i_mpz2num (mpq_numref (frac)),
+         q = scm_i_make_ratio (scm_i_mpz2num (mpq_numref (frac)),
                              scm_i_mpz2num (mpq_denref (frac)));
 
-         /* When scm_make_ratio throws, we leak the memory allocated
+         /* When scm_i_make_ratio throws, we leak the memory allocated
             for frac...
           */
          mpq_clear (frac);
@@ -5435,8 +5597,18 @@ SCM_DEFINE (scm_inexact_to_exact, "inexact->exact", 1, 0, 0,
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_rationalize, "rationalize", 2, 0, 0, 
-            (SCM x, SCM err),
-           "Return an exact number that is within @var{err} of @var{x}.")
+            (SCM x, SCM eps),
+           "Returns the @emph{simplest} rational number differing\n"
+           "from @var{x} by no more than @var{eps}.\n"
+           "\n"
+           "As required by @acronym{R5RS}, @code{rationalize} only returns an\n"
+           "exact result when both its arguments are exact.  Thus, you might need\n"
+           "to use @code{inexact->exact} on the arguments.\n"
+           "\n"
+           "@lisp\n"
+           "(rationalize (inexact->exact 1.2) 1/100)\n"
+           "@result{} 6/5\n"
+           "@end lisp")
 #define FUNC_NAME s_scm_rationalize
 {
   if (SCM_I_INUMP (x))
@@ -5468,7 +5640,7 @@ SCM_DEFINE (scm_rationalize, "rationalize", 2, 0, 0,
         converges after less than a dozen iterations.
       */
 
-      err = scm_abs (err);
+      eps = scm_abs (eps);
       while (++i < 1000000)
        {
          a = scm_sum (scm_product (a1, tt), a2);    /* a = a1*tt + a2 */
@@ -5476,11 +5648,11 @@ SCM_DEFINE (scm_rationalize, "rationalize", 2, 0, 0,
          if (scm_is_false (scm_zero_p (b)) &&         /* b != 0 */
              scm_is_false 
              (scm_gr_p (scm_abs (scm_difference (ex, scm_divide (a, b))),
-                        err)))                      /* abs(x-a/b) <= err */
+                        eps)))                      /* abs(x-a/b) <= eps */
            {
              SCM res = scm_sum (int_part, scm_divide (a, b));
              if (scm_is_false (scm_exact_p (x))
-                 || scm_is_false (scm_exact_p (err)))
+                 || scm_is_false (scm_exact_p (eps)))
                return scm_exact_to_inexact (res);
              else
                return res;
@@ -5605,6 +5777,16 @@ scm_is_unsigned_integer (SCM val, scm_t_uintmax min, scm_t_uintmax max)
     return 0;
 }
 
+static void
+scm_i_range_error (SCM bad_val, SCM min, SCM max)
+{
+  scm_error (scm_out_of_range_key,
+            NULL,
+            "Value out of range ~S to ~S: ~S",
+             scm_list_3 (min, max, bad_val),
+             scm_list_1 (bad_val));
+}
+
 #define TYPE                     scm_t_intmax
 #define TYPE_MIN                 min
 #define TYPE_MAX                 max
@@ -5689,6 +5871,23 @@ scm_is_unsigned_integer (SCM val, scm_t_uintmax min, scm_t_uintmax max)
 
 #endif
 
+void
+scm_to_mpz (SCM val, mpz_t rop)
+{
+  if (SCM_I_INUMP (val))
+    mpz_set_si (rop, SCM_I_INUM (val));
+  else if (SCM_BIGP (val))
+    mpz_set (rop, SCM_I_BIG_MPZ (val));
+  else
+    scm_wrong_type_arg_msg (NULL, 0, val, "exact integer");
+}
+
+SCM
+scm_from_mpz (mpz_t val)
+{
+  return scm_i_mpz2num (val);
+}
+
 int
 scm_is_real (SCM val)
 {
@@ -5713,7 +5912,7 @@ scm_to_double (SCM val)
   else if (SCM_REALP (val))
     return SCM_REAL_VALUE (val);
   else
-    scm_wrong_type_arg (NULL, 0, val);
+    scm_wrong_type_arg_msg (NULL, 0, val, "real number");
 }
 
 SCM
@@ -5758,6 +5957,195 @@ scm_num2double (SCM num, unsigned long int pos, const char *s_caller)
 
 #endif
 
+int
+scm_is_complex (SCM val)
+{
+  return scm_is_true (scm_complex_p (val));
+}
+
+double
+scm_c_real_part (SCM z)
+{
+  if (SCM_COMPLEXP (z))
+    return SCM_COMPLEX_REAL (z);
+  else
+    {
+      /* Use the scm_real_part to get proper error checking and
+        dispatching.
+      */
+      return scm_to_double (scm_real_part (z)); 
+    }
+}
+
+double
+scm_c_imag_part (SCM z)
+{
+  if (SCM_COMPLEXP (z))
+    return SCM_COMPLEX_IMAG (z);
+  else
+    {
+      /* Use the scm_imag_part to get proper error checking and
+        dispatching.  The result will almost always be 0.0, but not
+        always.
+      */
+      return scm_to_double (scm_imag_part (z));
+    }
+}
+
+double
+scm_c_magnitude (SCM z)
+{
+  return scm_to_double (scm_magnitude (z));
+}
+
+double
+scm_c_angle (SCM z)
+{
+  return scm_to_double (scm_angle (z));
+}
+
+int
+scm_is_number (SCM z)
+{
+  return scm_is_true (scm_number_p (z));
+}
+
+
+/* In the following functions we dispatch to the real-arg funcs like log()
+   when we know the arg is real, instead of just handing everything to
+   clog() for instance.  This is in case clog() doesn't optimize for a
+   real-only case, and because we have to test SCM_COMPLEXP anyway so may as
+   well use it to go straight to the applicable C func.  */
+
+SCM_DEFINE (scm_log, "log", 1, 0, 0,
+            (SCM z),
+           "Return the natural logarithm of @var{z}.")
+#define FUNC_NAME s_scm_log
+{
+  if (SCM_COMPLEXP (z))
+    {
+#if HAVE_COMPLEX_DOUBLE && HAVE_CLOG && defined (SCM_COMPLEX_VALUE)
+      return scm_from_complex_double (clog (SCM_COMPLEX_VALUE (z)));
+#else
+      double re = SCM_COMPLEX_REAL (z);
+      double im = SCM_COMPLEX_IMAG (z);
+      return scm_c_make_rectangular (log (hypot (re, im)),
+                                     atan2 (im, re));
+#endif
+    }
+  else
+    {
+      /* ENHANCE-ME: When z is a bignum the logarithm will fit a double
+         although the value itself overflows.  */
+      double re = scm_to_double (z);
+      double l = log (fabs (re));
+      if (re >= 0.0)
+        return scm_from_double (l);
+      else
+        return scm_c_make_rectangular (l, M_PI);
+    }
+}
+#undef FUNC_NAME
+
+
+SCM_DEFINE (scm_log10, "log10", 1, 0, 0,
+            (SCM z),
+           "Return the base 10 logarithm of @var{z}.")
+#define FUNC_NAME s_scm_log10
+{
+  if (SCM_COMPLEXP (z))
+    {
+      /* Mingw has clog() but not clog10().  (Maybe it'd be worth using
+         clog() and a multiply by M_LOG10E, rather than the fallback
+         log10+hypot+atan2.)  */
+#if HAVE_COMPLEX_DOUBLE && HAVE_CLOG10 && defined (SCM_COMPLEX_VALUE)
+      return scm_from_complex_double (clog10 (SCM_COMPLEX_VALUE (z)));
+#else
+      double re = SCM_COMPLEX_REAL (z);
+      double im = SCM_COMPLEX_IMAG (z);
+      return scm_c_make_rectangular (log10 (hypot (re, im)),
+                                     M_LOG10E * atan2 (im, re));
+#endif
+    }
+  else
+    {
+      /* ENHANCE-ME: When z is a bignum the logarithm will fit a double
+         although the value itself overflows.  */
+      double re = scm_to_double (z);
+      double l = log10 (fabs (re));
+      if (re >= 0.0)
+        return scm_from_double (l);
+      else
+        return scm_c_make_rectangular (l, M_LOG10E * M_PI);
+    }
+}
+#undef FUNC_NAME
+
+
+SCM_DEFINE (scm_exp, "exp", 1, 0, 0,
+            (SCM z),
+           "Return @math{e} to the power of @var{z}, where @math{e} is the\n"
+           "base of natural logarithms (2.71828@dots{}).")
+#define FUNC_NAME s_scm_exp
+{
+  if (SCM_COMPLEXP (z))
+    {
+#if HAVE_COMPLEX_DOUBLE && HAVE_CEXP && defined (SCM_COMPLEX_VALUE)
+      return scm_from_complex_double (cexp (SCM_COMPLEX_VALUE (z)));
+#else
+      return scm_c_make_polar (exp (SCM_COMPLEX_REAL (z)),
+                               SCM_COMPLEX_IMAG (z));
+#endif
+    }
+  else
+    {
+      /* When z is a negative bignum the conversion to double overflows,
+         giving -infinity, but that's ok, the exp is still 0.0.  */
+      return scm_from_double (exp (scm_to_double (z)));
+    }
+}
+#undef FUNC_NAME
+
+
+SCM_DEFINE (scm_sqrt, "sqrt", 1, 0, 0,
+            (SCM x),
+           "Return the square root of @var{z}.  Of the two possible roots\n"
+           "(positive and negative), the one with the a positive real part\n"
+           "is returned, or if that's zero then a positive imaginary part.\n"
+           "Thus,\n"
+           "\n"
+           "@example\n"
+           "(sqrt 9.0)       @result{} 3.0\n"
+           "(sqrt -9.0)      @result{} 0.0+3.0i\n"
+           "(sqrt 1.0+1.0i)  @result{} 1.09868411346781+0.455089860562227i\n"
+           "(sqrt -1.0-1.0i) @result{} 0.455089860562227-1.09868411346781i\n"
+           "@end example")
+#define FUNC_NAME s_scm_sqrt
+{
+  if (SCM_COMPLEXP (x))
+    {
+#if HAVE_COMPLEX_DOUBLE && HAVE_USABLE_CSQRT && defined (SCM_COMPLEX_VALUE)
+      return scm_from_complex_double (csqrt (SCM_COMPLEX_VALUE (x)));
+#else
+      double re = SCM_COMPLEX_REAL (x);
+      double im = SCM_COMPLEX_IMAG (x);
+      return scm_c_make_polar (sqrt (hypot (re, im)),
+                               0.5 * atan2 (im, re));
+#endif
+    }
+  else
+    {
+      double xx = scm_to_double (x);
+      if (xx < 0)
+        return scm_c_make_rectangular (0.0, sqrt (-xx));
+      else
+        return scm_from_double (sqrt (xx));
+    }
+}
+#undef FUNC_NAME
+
+
+
 void
 scm_init_numbers ()
 {
@@ -5789,10 +6177,6 @@ scm_init_numbers ()
       scm_dblprec[10-2] = (DBL_DIG > 20) ? 20 : DBL_DIG;
 #endif
 
-#ifdef GUILE_DEBUG
-  check_sanity ();
-#endif
-
   exactly_one_half = scm_permanent_object (scm_divide (SCM_I_MAKINUM (1),
                                                       SCM_I_MAKINUM (2)));
 #include "libguile/numbers.x"