Range issues

Description

Problem
There are a number of bugs with Range which occur when the step size is 0 or where negative.

Examples

Environment

None

Attachments

2

Activity

Show:

Mike Fikes November 20, 2017 at 1:39 AM

Patch no longer applies; needs re-baseline.

Thomas Mulvaney November 15, 2016 at 2:24 PM

Updated patch with performance tweaks.

  • Added the ^boolean annotation to the `some-range?` helper.

  • Removed calls to methods of Range where possible.

  • Improved 2-arity reduces performance over master significantly by replacing the call to ci-reduce.

David Nolen November 10, 2016 at 10:37 PM

This patch is headed in the right direction but it needs to be more vigilant about performance. I'm more than happy to talk it over via IRC or Slack. Thanks!

Details

Assignee

Reporter

Labels

Patch

Code and Test

Priority

Created November 10, 2016 at 2:01 PM
Updated November 20, 2017 at 1:39 AM