Opened 8 years ago

Closed 8 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: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

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 8 years ago.
export-mapAccumRWithKey.patch (4.5 KB) - added by Deewiant 8 years ago.
Revised

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by Deewiant

Attachment: export-mapAccumR.patch added

comment:2 Changed 8 years ago by igloo

difficulty: Unknown
Milestone: Not GHC

comment:3 Changed 8 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 8 years ago by Deewiant

Revised

comment:4 Changed 8 years ago by igloo

Resolution: fixed
Status: newclosed

export-mapAccumRWithKey.patch applied, thanks!

Note: See TracTickets for help on using tickets.