Ticket #4526 (closed proposal: fixed)

Opened 2 years ago

Last modified 2 years ago

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 Difficulty:
Test Case: Blocked By:
Blocking: Related Tickets:

Description

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

Attachments

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

Change History

Changed 2 years ago by mitar

Changed 2 years ago by mitar

I attached a patch which works for me.

Changed 2 years ago by mitar

  • type changed from feature request to proposal

Changed 2 years ago by mitar

I am attaching Bas van Dijk's darcs patch.

Changed 2 years ago by mitar

Changed 2 years ago by basvandijk

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

Changed 2 years ago by basvandijk

  • cc v.dijk.bas@… added

 Thread on the libraries list.

Changed 2 years ago by mitar

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

Changed 2 years ago by basvandijk

Darcs patch that adds Eq instances for QSem and QSemN

Changed 2 years ago by igloo

  • milestone set to Not GHC

Changed 2 years ago by mitar

  • status changed from new to patch

There were no objections.

Changed 2 years ago by simonmar

  • priority changed from normal to high
  • milestone changed from Not GHC to 7.2.1

Changed 2 years ago by igloo

  • status changed from patch to closed
  • resolution set to fixed

Applied, thanks!

Note: See TracTickets for help on using tickets.