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

[David Roundy]

On Sun, Jun 01, 2008 at 05:30:41PM +0100, Ian Lynagh wrote:
> 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.

I'd rather keep debugging code separate from code we use in darcs itself,
just to avoid bloat.
-- 
David Roundy
Department of Physics
Oregon State University