Re: [PATCH] First batch of numerics changes

[Andy Wingo]

On Sat 29 Jan 2011 09:20, Mark H Weaver <address@hidden> writes:

> @@ -728,7 +728,7 @@ SCM_PRIMITIVE_GENERIC (scm_abs, "abs", 1, 0, 0,
>                      "Return the absolute value of @var{x}.")
>  #define FUNC_NAME
>  {
> -  if (SCM_I_INUMP (x))
> +  if (SCM_LIKELY (SCM_I_INUMP (x)))
>      {
>        scm_t_inum xx = SCM_I_INUM (x);
>        if (xx >= 0)

Is this really likely the case?  Probably not.  I'm going to hold off on
this patch, unless you really want it in.

I applied the other four in your series, though not including the div0
one.

Thanks for the patches!

Andy
-- 
http://wingolog.org/