[darcs-devel] [patch1182] Implementation of the command 'darcs undo'.

Ganesh Sittampalam bugs at darcs.net
Tue Jul 29 20:36:50 UTC 2014


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

Some brief initial comments:

- watch out for spurious whitespace changes

- at some point please merge undoFun into finalizeRepositoryChanges as 
discussed before - it'll mean a tedious initial pass through the 
codebase marking everything as not undoable by default, but it should 
help to shake out any tricky cases and also make sure that nothing is 
missed. It'll also be good for future maintainability as people won't be 
able to forget undo.

----------
status: needs-screening -> in-discussion

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


More information about the darcs-devel mailing list