Ticket #1298 (closed bug: invalid)

Opened 6 years ago

Last modified 5 months ago

Data.Complex.phase does unnecessary pattern match

Reported by: guest Owned by:
Priority: normal Milestone: _|_
Component: Prelude Version: 6.7
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty: Unknown
Test Case: Blocked By:
Blocking: Related Tickets:

Description

The Data.Complex.phase function contains this line: phase (0 :+ 0) = 0 -- SLPJ July 97 from John Peterson

This is not necessary since atan2 (invoked on the following line) is defined such that it returns 0 in this case already.

-- Lennart

Change History

Changed 6 years ago by igloo

  • milestone set to _|_

Changed 5 years ago by simonmar

  • architecture changed from Unknown to Unknown/Multiple

Changed 5 years ago by simonmar

  • os changed from Unknown to Unknown/Multiple

Changed 5 months ago by morabbin

  • failure set to None/Unknown

Since it's not GHC-specific (see Ian's comment above), close as invalid?

Changed 5 months ago by igloo

  • status changed from new to closed
  • resolution set to invalid

Yeah, I'll close this. Anyone interested in seeing it happen, please propose it on the list as per the policy.

Note: See TracTickets for help on using tickets.