@scresto09 commented on this pull request.
@@ -162,14 +163,14 @@ void cmd_del_word_left(CmdContext *c, CmdParams *p)
void cmd_undo(CmdContext *c, CmdParams *p) { gint i; - gint pos = SSM(p->sci, SCI_GETCURRENTPOS, 0, 0); + start_undo(c); for (i = 0; i < p->num; i++) { if (!SSM(p->sci, SCI_CANUNDO, 0, 0)) break; SSM(p->sci, SCI_UNDO, 0, 0); }
OK, I tried to refactor the code [63b7e25](https://github.com/geany/geany-plugins/pull/1328/commits/63b7e25242e485448c8...). But `excmd_undo` cannot simply call `cmd_undo` because of the use of `CmdParams` parameter. So I moved this code into a new `undo_apply` function which is called by `cmd_undo` and `excmd_undo`.