[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