[Q] Change forms like (delq nil (mapcar ...)) to (mapcan ...).
Stephen J. Turnbull
stephen at xemacs.org
Fri Sep 17 02:08:35 EDT 2010
Aidan Kehoe writes:
> Replace the O2N (delq nil (mapcar (lambda (W) (and X Y)) Z)) with
> the ON (mapcan (lambda (W) (and X (list Y))) Z) in these files.
This can't work as written, can it? If X is nil, if Y is nil, it will
wrap nil in a list and the nil will not be deleted. I guess from a
quick look at the patch that it's expected that if X is non-nil, Y
will be non-nil, but are you sure?
I'm also not convinced that this is necessarily faster if (length Z)
is small, as the mapcan version involves extra consing.
More information about the XEmacs-Patches