Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

`range` reducer

Description

Rich mentioned in IRC today he'd welcome a reducer implementation of clojure.core/range. Now that I've figured out how to do iterate, I figure I'll knock out range as well by the end of the night. Just opening the issue early to announce my intentions to anyone else interested in doing it.

Environment

None

Attachments

2

Activity

Show:

Ghadi Shayban June 30, 2016 at 3:17 AM

We should close this too. The only thing not covered in 1.7 is the foldable aspect.

Ghadi Shayban January 9, 2016 at 11:31 PM

range is now reducible.

Alan Malloy August 19, 2012 at 1:18 AM

As requested, I have split up the large patch on this issue into four smaller tickets. The other three are: CLJ-1045, CLJ-1046, and CLJ-992.

CLJ-1045 contains the implementation of fold-by-halves, and as such this patch cannot be applied until CLJ-1045 is accepted. This ticket does not depend on the other two, but there will be minor merge conflicts if this is merged before them.

Alan Malloy August 19, 2012 at 12:19 AM

Okay, I should be able to split it up over the weekend. I'll also see about converting fold-by-halves into a function that is used by Range/Vector, rather than a function that gets extended onto them.

Rich Hickey August 14, 2012 at 8:07 PM

I prefer not to see the use of extend like this for new types. Perhaps this code is too DRY? Also, it does a lot in one patch which makes it hard to parse and accept. This adds Range, switches impl of vector folds etc. Can it be broken up into separate tickets that do each step that builds on the previous, e.g. one ticket could be: capture vector fold impl for reuse by similar things.

Details

Assignee

Reporter

Labels

Patch

Code and Test

Priority

Affects versions

Created May 11, 2012 at 3:47 AM
Updated May 15, 2017 at 9:55 PM