On Tue, Jan 01, 2008 at 03:30:46PM -0500, [EMAIL PROTECTED] wrote:
> Tue Jan  1 15:04:47 EST 2008  Mark Stosberg <[EMAIL PROTECTED]>
>   * refactor: replace recursive indent with more concise find-and-replace 
> syntax

Applied.  Thanks, Mark!

Although I'd rather not use Text.Regex for this (I have a longstanding fear
of the Regex libraries, ever since a traumatic encounter with a segfaulting
regex... back in ghc 4.x or something), less code does seem better.  Your
code is less efficient, but for an error message that won't matter.  I
suspect we could get something even more elegant with lines
and unlines... hmmmm:

indent = unlines . map ("    "++) . lines

Okay, I've gone ahead and implemented this change.  But thanks for finding
the less-than-elegant code!
-- 
David Roundy
Department of Physics
Oregon State University
_______________________________________________
darcs-devel mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-devel

Reply via email to