srfi 134 errors
Issue #228
resolved
I recently opened a pull request on the official srfi 134 reference implementation, as there are two errors. These errors are also in Sagittarius.
In your file: sagittarius-scheme / sitelib / srfi / %3a134 / ideque.scm
line 301 should be: (check (- lenf n) (drop f n) lenr r) to keep the size of f correct
line 306 should be: (unless (<= 0 n (ideque-length dq)) so take/drop can use all elements in the deque
There are additional test cases in the attached file, to highlight and check the fix.
(Sorry for not making a pull request, but I seem to be having problems with Mercurial.)
Comments (3)
-
repo owner -
repo owner -
assigned issue to
-
assigned issue to
-
repo owner - changed status to resolved
Adapting upstream fix of SRFI-134 (thanks to Peter Lane). Fixes
#228→ <<cset 87d42a326e26>>
- Log in to comment
Thank you for the report!
I'll adapt the fix asap.