[darcs-devel] [patch2180] remove debugMessage for withNamedTemp (and 9 more)

[Ben Franksen]

Ben Franksen <ben.franksen at online.de> added the comment:

>> * remove debugMessage for withNamedTemp
> 
> Fine though I'd favour just deleting rather than commenting out
> unwanted things.

In general I agree. Can we make an exception to this rule for code that
is only there for debugging? It makes it easier to restore the debug
code in case it will be needed again, and it is not a great problem if
such code has bit-rotted in the meantime.

>> * use +<<+ instead of  +<+ reverseFL
> 
> Fine though I'm not actually sure this is an improvement, it doesn't 
> have a complexity advantage and is another operator to remember.> But
> it was there before and we could debate removing it and +>>+ entirely
> in a separate patch.

I will respond to this in a separate message to the list then.

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