[darcs-devel] [issue1568] bytestring for Darcs.Patch.FileName

[Eric Kow]

New submission from Eric Kow <kowey at darcs.net>:

Paraphrased from http://wiki.darcs.net/HowToHelp:

8. Try changing `FileName <FileName>`_ to store filenames as bytestrings
   rather than String. This should speed parsing somewhat, and reduce
   memory usage. We'd still need to convert to String for IO functions,
   but those are relatively isolated, so I think this'd still be a win
   (particularly as the filename starts out as a packed string in the
   patch files.

Sounds like something you'd need to benchmark.  Proceed with caution, as there
may be some compatibility/encodings stuff to contend with

----------
messages: 8502
nosy: darcs-devel, dmitry.kurochkin, kowey, thorkilnaur
priority: feature
status: need-implementation
title: bytestring for Darcs.Patch.FileName
topic: Performance

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/issue1568>
__________________________________