On Sat 29 Jan 2011 09:20, Mark H Weaver <[email protected]> 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/
