Clojure

reduce-kv on sorted maps should reduce, in sorted order

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Approval:
    Ok

Description

Currently reduces, but not in sorted order.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Alan Malloy added a comment -

Stuart, your patch doesn't check for Reduced after calling f on the current node; I think you need to move that check along with the call to f.

Show
Alan Malloy added a comment - Stuart, your patch doesn't check for Reduced after calling f on the current node; I think you need to move that check along with the call to f.
Hide
Permalink
Alan Malloy added a comment -

CLJ-1041 is a related issue, and its patch also includes a commit that I think would fix your patch.

Show
Alan Malloy added a comment - CLJ-1041 is a related issue, and its patch also includes a commit that I think would fix your patch.

People

Vote (0)
Watch (2)

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.