[darcs-users] darcs patch: Add Show/Show2 instances for :/\: (and 3 more)

Ian Lynagh igloo at earth.li
Sun Jun 1 16:30:41 UTC 2008


On Sat, May 31, 2008 at 03:51:39AM -0700, David Roundy wrote:
> On Sat, May 31, 2008 at 3:21 AM, David Roundy <droundy at darcs.net> wrote:
> > On Fri, May 30, 2008 at 03:08:49PM +0000, E.Y.Kow at brighton.ac.uk wrote:
> >> I thought I might re-send Ian's non-conflictor patches.
> >>
> >> Wed Apr 30 01:11:12 BST 2008  Ian Lynagh <igloo at earth.li>
> >>   * Add Show/Show2 instances for :/\:
> >>
> > I've applied these first two.
> 
> The first patch caused an error in compiling the unit tests, since we
> already define this instance elsewhere, so it's not being pushed.

It would be nice to move the instance to the module defining the type,
partly to avoid orphan modules[1], but mostly just so that it's always
available when working with ghci on a subset of the modules.


[1] http://www.haskell.org/ghc/docs/latest/html/users_guide/separate-compilation.html#orphan-modules


Thanks
Ian



More information about the darcs-users mailing list