Stack overflow regression for sequence xform applied to eduction

Description

There is a regression in the ability to use sequence to apply transducers to eductions that was introduced with 1.9.562.

In 1.9.542 and earlier,

produced ("1").

Minimal repro:

Environment

None

Activity

Show:
Mike Fikes
July 18, 2017, 1:36 PM

CLJS-2034:

António Monteiro
July 18, 2017, 6:08 PM

Attached patch with fix and test.

Mike Fikes
July 18, 2017, 6:37 PM

I don't know if this is a bug or not, but with 1.9.542

and after the patch:

I wonder if there is some public code path where this would show up (as opposed to directly calling iter).

António Monteiro
July 18, 2017, 6:57 PM

`(iter (eduction [1 2 3]))` is now a TransformerIterator, which doesn't have a print method.

For comparison with Clojure:

David Nolen
July 24, 2017, 6:12 AM
Completed

Assignee

David Nolen

Reporter

Mike Fikes

Approval

Accepted

Patch

Code and Test

Affects versions

Priority

Critical