Opened 8 years ago

Closed 7 years ago

#34 closed defect (fixed)

Ambiguous show instance.

Reported by: benl 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 7 years ago by benl

  • Milestone changed from 3.0.0 to 3.2.0

comment:2 Changed 7 years ago by benl

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

Fixed as we just used deriving Show for manifest arrays.

Note: See TracTickets for help on using tickets.