ClojureScript

Add clojure.core/shuffle implementation

Details

  • Type: Enhancement Enhancement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Patch:
    Code and Test

Description

I added a simple implementation of clojure.core/shuffle, which uses goog.array's Fisher-Yates for its implementation. Included in the patch is a test to make sure it works.

Attachments

  1. Text File
    0001-Add-clojure.core-shuffle-and-a-test.patch
    20/May/12 4:24 PM
    2 kB
    Evan Mezeske
  2. Text File
    shuffle.patch
    17/May/12 6:45 PM
    1 kB
    Evan Mezeske
  3. Text File
    shuffle.v2.patch
    17/May/12 7:11 PM
    1 kB
    Evan Mezeske

Activity

Descending order - Click to sort in ascending order
Hide
Permalink
Evan Mezeske added a comment -

Use git format-patch instead of git diff, to hopefully provide the right patch format.

Show
Evan Mezeske added a comment - Use git format-patch instead of git diff, to hopefully provide the right patch format.
Hide
Permalink
David Nolen added a comment -

The patch is not correctly formatted with attribution.

Show
David Nolen added a comment - The patch is not correctly formatted with attribution.
Hide
Permalink
Evan Mezeske added a comment -

Return a vector like Clojure.

Show
Evan Mezeske added a comment - Return a vector like Clojure.
Hide
Permalink
David Nolen added a comment -

Why don't you return a vector like Clojure does?

Show
David Nolen added a comment - Why don't you return a vector like Clojure does?

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4#649-r158309) | Report a problem
Powered by a free Atlassian JIRA open source license for Clojure. Try JIRA - bug tracking software for your team.