Opened 6 years ago

Closed 6 years ago

#2769 closed proposal (fixed)

Export mapAccumR from Data.Map, Data.IntMap

Reported by: Deewiant Owned by:
Priority: normal Milestone: Not GHC
Component: libraries (other) Version: 6.11
Keywords: containers Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Test Case:
Blocked By: Blocking:
Related Tickets: Differential Revisions:

Description

For some reason, the mapAccumR function isn't exported from Data.Map or Data.IntMap: it's just commented out. Attached is a patch which corrects that.

Attachments (2)

export-mapAccumR.patch (3.3 KB) - added by Deewiant 6 years ago.
export-mapAccumRWithKey.patch (4.5 KB) - added by Deewiant 6 years ago.
Revised

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by Deewiant

comment:2 Changed 6 years ago by igloo

  • difficulty set to Unknown
  • Milestone set to Not GHC

comment:3 Changed 6 years ago by Deewiant

I think I've let this sit for long enough now.

Discussion was short, with only one response: Ross Paterson suggested that the name be mapAccumRWithKey instead of just mapAccumR as the latter is available through Traversable with a different type.

I agree with this change and attached is the revised patch.

Changed 6 years ago by Deewiant

Revised

comment:4 Changed 6 years ago by igloo

  • Resolution set to fixed
  • Status changed from new to closed

export-mapAccumRWithKey.patch applied, thanks!

Note: See TracTickets for help on using tickets.