Opened 9 years ago

Closed 9 years ago

#4526 closed proposal (fixed)

Make Chan an instance of Eq

Reported by: mitar Owned by:
Priority: high Milestone: 7.4.1
Component: libraries/base Version: 7.0.1
Keywords: Cc: mmitar@…, v.dijk.bas@…
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

Chan is not an instance of Eq, but it could be easily, as MVars it uses are.

Attachments (4)

chan-eq.patch (793 bytes) - added by mitar 9 years ago.
derived_Eq_instance_for_Chan.dpatch (57.0 KB) - added by mitar 9 years ago.
derived_Eq_instance_for_Chan_with_doc.dpatch (57.4 KB) - added by basvandijk 9 years ago.
Adds "deriving Eq" to Chan & a little note in the doc of dupChan
derived_Eq_instance_for_QSem_and_QSemN.dpatch (57.5 KB) - added by basvandijk 9 years ago.
Darcs patch that adds Eq instances for QSem and QSemN

Download all attachments as: .zip

Change History (13)

Changed 9 years ago by mitar

Attachment: chan-eq.patch added

comment:1 Changed 9 years ago by mitar

I attached a patch which works for me.

comment:2 Changed 9 years ago by mitar

Type: feature requestproposal

comment:3 Changed 9 years ago by mitar

I am attaching Bas van Dijk's darcs patch.

Changed 9 years ago by mitar

Changed 9 years ago by basvandijk

Adds "deriving Eq" to Chan & a little note in the doc of dupChan

comment:4 Changed 9 years ago by basvandijk

Cc: v.dijk.bas@… added

Thread on the libraries list.

comment:5 Changed 9 years ago by mitar

I am proposing a week to discus things. Starting now.

Changed 9 years ago by basvandijk

Darcs patch that adds Eq instances for QSem and QSemN

comment:6 Changed 9 years ago by igloo

Milestone: Not GHC

comment:7 Changed 9 years ago by mitar

Status: newpatch

There were no objections.

comment:8 Changed 9 years ago by simonmar

Milestone: Not GHC7.2.1
Priority: normalhigh

comment:9 Changed 9 years ago by igloo

Resolution: fixed
Status: patchclosed

Applied, thanks!

Note: See TracTickets for help on using tickets.