<div dir="ltr">Hi all,<div>I almost finished the new autosubpatching feature, only leaving undo/redo support. </div><div>I have several questions regarding this topic.</div><div>All the actions performed by the feature can be partitioned into 2+ basic actions that already have undo support: 1 cut + 1 paste + N connects (or 1 cut + 1 paste + 1 special paste, if we slightly change the "dopaste" function, probably not the best option).</div><div>The question is if it's worth it to implement a new undo type for the feature or instead would be better implement a new special undo type that implements a multi-step undo by having a private queue of basic undo steps and executing all of them in a row. This special undo type could be used not only in this feature but somewhere else in the future.</div><div><br></div><div>What do you think?</div><div>Especially to Ico, who implemented the infinite undo.</div><div><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><font size="2" face="arial, helvetica, sans-serif">Guillem Bartrina Moreno</font></div></div></div></div></div></div></div></div>