diff --git a/libguile/ChangeLog b/libguile/ChangeLog index 6942746ad..5ee017ac1 100644 --- a/libguile/ChangeLog +++ b/libguile/ChangeLog @@ -1,3 +1,27 @@ +Thu Jan 6 09:54:33 2000 Dirk Herrmann --gjb applied + + * scm_validate.h: Remove some redundant NIMP tests. + + * alist.c: minimize scope of the tmp variables, and initialize + them when declared. The strange SCM_NIMP tests are replaced by + SCM_CONSP tests that more closely reflect the intended semantics. + However, we don't get a performance penalty here, because the + SCM_CONSP test was performed by the ALISTCELL test anyway. * The + extremely ugly use of ASRTGO macros was removed: The calls to + ASRTGO were not encapsulated by "#ifndef SCM_RECKLESS", but got a + label parameter that only exists when SCM_RECKLESS is not defined. + This works, because ASRTGO itself is defined in a way that it only + makes use of the label parameter if SCM_RECKLESS is not defined + (shudder!). Does guile make at all use of the possibility to + define SCM_RECKLESS? * Codesize is likely to be reduced, since + instead of two calls to SCM_ASSERT performed by the ALISTCELL test + we now only get one test. + + * list.c: Use SCM_NNULLP, not SCM_NIMP as appropriate. Also use + SCM_NULLP instead of SCM_IMP. Drop use of "register" keyword on + some variables in `list?'. Fix `reverse' and `reverse!' + primitives to handle improper lists better. + Wed Jan 5 11:24:53 2000 Greg J. Badros * *.[ch]: Whitespace changes -- added space after SCM_VALIDATE_*