[implementations-list] [PATCH] Remove unused undo-list-munging code.

Štěpán Němec stepnem at gmail.com
Fri May 21 22:42:42 CEST 2010


On Fri, May 21, 2010 at 08:22:22PM +0000, Vegard Øye wrote:
> > While looking at the code, I noticed that all of this is currently
> > unused. Is that just an oversight, or are you actually planning to
> > use it in the future?
> 
> The purpose of `vimpulse-connect-undos' is to collapse a single "edit"
> like "cwfoo" to a single undo. Right now it's more fine-grained:
> "cwfoo<Esc>u" undos to the insertion of "foo", so that another "u" is
> needed to undo the deletion of the original word.

Well, the purpose is clear, it's just that the code is not used
anywhere.
 
> The reason this code lies dormant is that although it's relevant, I'm
> unsure how it goes together with undo-tree.el.

Yeah. I'm unsure even about how adding `vimpulse' to buffer-undo-list
goes together with standard Emacs undo. Maybe Frank could comment -- I
assume the original code comes from him?

Štěpán

P.S.: Maybe you could at least add a code comment in similar cases? Am I
the only one who finds it weird to have unused code lying around like
this?



More information about the implementations-list mailing list