bug fix in SRFI 121 implementation
Arthur A. Gleckler 12 Aug 2016 01:32 UTC
Kevin Wortman has contributed a fix to a bug in the
reference implementation for SRFI 121, and has done minor
refactoring, too. John Cowan, one of the authors, has
approved this change, so I've published it on
<srfi.schemers.org>.
The diffs are here:
<https://github.com/scheme-requests-for-implementation/srfi-121/pull/10/files>
Kevin's description is here:
| Commit cd3f56f fixed a bug in generator->list, where the
| procedure could consume the wrong number of elements from
| the generator. It turns out that generator->reverse-list
| has the same bug. This patch fixes that.
>
| While we're at it, we replaced each of these problematic
| loops with a one-liner that reuses other procedures. The
| old loops were not tail-recursive, so they allocated O(n)
| activation records. The new code avoids that so should
| perform better.
| Commit Summary
>
| * fix overconsumption in generator->reverse-list, as in
| generator->list
>
| File Changes
>
| * M generators/generators-impl.scm (28)
Thanks, Kevin.
Regards,
SRFI Editor