Ticket #1259 (closed bug: invalid)

Opened 6 years ago

Last modified 4 years ago

Accessing undefined value in DiffArray returns misleading error message

Reported by: guest Owned by:
Priority: normal Milestone: 6.12 branch
Component: libraries/base Version: 6.6
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Difficulty: Easy (less than 1 hour)
Test Case: Blocked By:
Blocking: Related Tickets:

Description (last modified by igloo) (diff) 

Prelude> :m + Data.Array.Diff
Prelude Data.Array.Diff> :t array
array :: (Ix i, IArray a e) => (i, i) -> [(i, e)] -> a i e
Prelude Data.Array.Diff> array (1,1) [] :: DiffArray Int Int
array (1,1) [(1,*** Exception: MArray: undefined array element

Since DiffArray? isn't a MArray(it implements IArray), this is misleading. It should either say IArray or better DiffArray?: undefined array element.

Change History

Changed 6 years ago by igloo

  • description modified (diff)
  • milestone set to 6.1

I think the right way to fix this is to wait until we have extensible exceptions, so MArray throws a structured exception. DiffArray? can then s/MArray/DiffArray/ without having to duplicate all the checks.

As such, it's probably something for 6.10.

Changed 5 years ago by simonmar

  • architecture changed from Multiple to Unknown/Multiple

Changed 5 years ago by simonmar

  • os changed from Unknown to Unknown/Multiple

Changed 4 years ago by igloo

  • priority changed from low to normal
  • milestone changed from 6.10 branch to 6.12 branch

Raising priority now that we have extensible exceptions.

Changed 4 years ago by igloo

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

diffarray is now in its own package. Ticket moved to here:  http://trac.haskell.org/diffarray/ticket/1

Changed 4 years ago by simonmar

  • difficulty changed from Easy (1 hr) to Easy (less than 1 hour)
Note: See TracTickets for help on using tickets.