diff options
| author | zeertzjq <zeertzjq@outlook.com> | 2022-04-01 11:40:45 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-04-01 11:40:45 +0800 |
| commit | 973e91007ce3f343f7aeed8d30d70a8616ab6bb5 (patch) | |
| tree | afa42fdbb9fb0c3b7f0d305c2b7407cb178b322f | |
| parent | 8a6cf51a710585fee7edb9d1357791da30f31f44 (diff) | |
| download | rneovim-973e91007ce3f343f7aeed8d30d70a8616ab6bb5.tar.gz rneovim-973e91007ce3f343f7aeed8d30d70a8616ab6bb5.tar.bz2 rneovim-973e91007ce3f343f7aeed8d30d70a8616ab6bb5.zip | |
refactor: remove redundant check for w_p_cole (#17944)
No longer needed after #17890.
| -rw-r--r-- | src/nvim/edit.c | 12 | ||||
| -rw-r--r-- | src/nvim/normal.c | 7 |
2 files changed, 7 insertions, 12 deletions
diff --git a/src/nvim/edit.c b/src/nvim/edit.c index 815d57121b..a2687ada73 100644 --- a/src/nvim/edit.c +++ b/src/nvim/edit.c @@ -1486,7 +1486,7 @@ static void ins_redraw(bool ready) // Trigger CursorMoved if the cursor moved. Not when the popup menu is // visible, the command might delete it. - if (ready && (has_event(EVENT_CURSORMOVEDI) || curwin->w_p_cole > 0) + if (ready && has_event(EVENT_CURSORMOVEDI) && !equalpos(curwin->w_last_cursormoved, curwin->w_cursor) && !pum_visible()) { // Need to update the screen first, to make sure syntax @@ -1496,12 +1496,10 @@ static void ins_redraw(bool ready) if (syntax_present(curwin) && must_redraw) { update_screen(0); } - if (has_event(EVENT_CURSORMOVEDI)) { - // Make sure curswant is correct, an autocommand may call - // getcurpos() - update_curswant(); - ins_apply_autocmds(EVENT_CURSORMOVEDI); - } + // Make sure curswant is correct, an autocommand may call + // getcurpos() + update_curswant(); + ins_apply_autocmds(EVENT_CURSORMOVEDI); curwin->w_last_cursormoved = curwin->w_cursor; } diff --git a/src/nvim/normal.c b/src/nvim/normal.c index 72e80952ff..8332b8bbd6 100644 --- a/src/nvim/normal.c +++ b/src/nvim/normal.c @@ -1233,12 +1233,9 @@ static void normal_check_window_scrolled(NormalState *s) static void normal_check_cursor_moved(NormalState *s) { // Trigger CursorMoved if the cursor moved. - if (!finish_op && (has_event(EVENT_CURSORMOVED) || curwin->w_p_cole > 0) + if (!finish_op && has_event(EVENT_CURSORMOVED) && !equalpos(curwin->w_last_cursormoved, curwin->w_cursor)) { - if (has_event(EVENT_CURSORMOVED)) { - apply_autocmds(EVENT_CURSORMOVED, NULL, NULL, false, curbuf); - } - + apply_autocmds(EVENT_CURSORMOVED, NULL, NULL, false, curbuf); curwin->w_last_cursormoved = curwin->w_cursor; } } |