[darcs-devel] [patch1917] fix lazy reading of inventories for apply and obliterate

Ganesh Sittampalam bugs at darcs.net
Fri Sep 20 19:41:49 UTC 2019


Ganesh Sittampalam <ganesh at earth.li> added the comment:

> I am curious: would you consider keeping the original order of arguments
> and instead using flip the better solution in this case? Or would you
> rather give a name to the intermediate PatchSet?

I think I generally find pointfree style more attractive when I'm
writing code than when I'm reading it :-) So from the perspective of a
reviewer now I'd probably say "give it a name". But I think a flip would
have been fine too. And even the actual code is not worth changing again
now.

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


More information about the darcs-devel mailing list