Clojure

sort changes its argument, if a Java array

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: Release 1.3, Release 1.4
  • Fix Version/s: Backlog, Release 1.5
  • Component/s: None
  • Labels:
    None
  • Environment:
    java version "1.6.0_22"
    OpenJDK Runtime Environment (IcedTea6 1.10.4) (6b22-1.10.4-0ubuntu1~11.04.1)
    (clojure-version)
    "1.4.0-alpha2"
  • Patch:
    Code
  • Approval:
    Ok

Description

user> (let [a (to-array [32 11])]
(prn (seq a))
(sort a)
(prn (seq a)))
(32 11)
(11 32)
nil

Where the second line printed ought to be the same as the first.

Activity

Hide
Andy Fingerhut added a comment -

clj-892-clarify-sort-sort-by-doc-strings-patch1.txt of Mar 26, 2012 applies cleanly to latest master on that date. Only doc string changes, to make it clear that by sort and sort-by will modify Java arrays given as arguments.

Show
Andy Fingerhut added a comment - clj-892-clarify-sort-sort-by-doc-strings-patch1.txt of Mar 26, 2012 applies cleanly to latest master on that date. Only doc string changes, to make it clear that by sort and sort-by will modify Java arrays given as arguments.
Hide
Stuart Halloway added a comment -

This is an enhancement request, since the docs for sort make no promise one way or the other.

For performance, I prefer the current behavior, so another possibility is a clarifying doc string.

Show
Stuart Halloway added a comment - This is an enhancement request, since the docs for sort make no promise one way or the other. For performance, I prefer the current behavior, so another possibility is a clarifying doc string.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: