[EMAIL PROTECTED] (Ludovic Courtès) writes: > I believe it *really* makes sense to turn `scm_is_pair ()' into a macro.
Yep. But what about an inline function? There is some machinery in inline.h for this and we already use it for scm_cell, for example. > +#define scm_is_pair(__obj) (SCM_I_CONSP (__obj)) SCM_I_CONSP is not a safe macro, it expands its argument twice so we shouldn't use it just like this. (What is the point of the double underscores?) (It is true that I didn't care about performance at all when introducing scm_is_pair and your approach is the exactly right one: profile and identify the real bottlenecks.) -- GPG: D5D4E405 - 2F9B BCCC 8527 692A 04E3 331E FAF8 226A D5D4 E405 _______________________________________________ Guile-devel mailing list Guile-devel@gnu.org http://lists.gnu.org/mailman/listinfo/guile-devel
