Opened 10 years ago

Closed 10 years ago

#34 closed defect (fixed)

Ambiguous show instance.

Reported by: Ben Lippmeier Owned by:
Priority: normal Milestone: 3.2.0
Component: library Version: 2.1.0.1
Keywords: Cc:

Description

Grigory Sarnitskiy reports:

Hello! I'm using repa 2.1.1.5, so maybe this issue is gone already.

Nevertheless, repa's Show instance is a bit ambiguous, e.g. show of Maybe (Array DIM1 Int)

"Just Array (Z :. 1) [...]"

Wouldn't it be better to have

"Just (Array (Z :. 1) [...])"

just like elsewhere in Haskell? For example

show $ Just $ Just 5
"Just (Just 5)"

Change History (2)

comment:1 Changed 10 years ago by Ben Lippmeier

Milestone: 3.0.03.2.0

comment:2 Changed 10 years ago by Ben Lippmeier

Resolution: fixed
Status: newclosed

Fixed as we just used deriving Show for manifest arrays.

Note: See TracTickets for help on using tickets.