[darcs-devel] darcs patch: (Re)upgrade createPS and generatePS to D... (and 8 more)

[Don Stewart]

droundy:
> On Wed, Feb 06, 2008 at 08:46:44AM -0800, Don Stewart wrote:
> > droundy:
> > > On Wed, Feb 06, 2008 at 04:42:02PM +0000, Eric Kow wrote:
> > > > Maybe we can egg on the bytestring people and get them to submit
> > > > patches taking this further down (for example, they could improve our
> > > > between newlines and nth newline stuff).
> > > 
> > > That'd be nice.
> > 
> > If you can get darcs working with bytestrings, I'm happy to write
> > bytestring-optimised variants of things you need, to squeeze out more 
> > performance.
> 
> Eric's already got darcs working with bytestrings, so you could already
> take a look.  But as I mentioned to Eric, it'd be best first to eliminate
> use of [PackedString] where we can.  That'll affect where our bottlenecks
> are.

Yeah, [PackedString] could be fruitfully replaced with Chunk !ByteString (Chunk ..)
if you still need a list like thing, to save one indirection as well
(specialising, and unboxing, you can save a bit of space and a good
percent of time too).

-- Don