Opened 7 years ago

Closed 15 months ago

#1298 closed bug (invalid)

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 (5)

comment:1 Changed 7 years ago by igloo

  • Milestone set to _|_

comment:2 Changed 6 years ago by simonmar

  • Architecture changed from Unknown to Unknown/Multiple

comment:3 Changed 6 years ago by simonmar

  • Operating System changed from Unknown to Unknown/Multiple

comment:4 Changed 15 months ago by morabbin

  • Type of failure set to None/Unknown

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

comment:5 Changed 15 months ago by igloo

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

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.