From c9e474b5ce8e902777ef718705f95696cda7d8e2 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Tue, 26 May 2009 10:50:53 +0000 Subject: [PATCH] [7.2.189] Possible hang for deleting auto-indent. (Dominique Pelle) Problem: Possible hang for deleting auto-indent. (Dominique Pelle) Solution: Make sure the position is not beyond the end of the line. Patch 7.2.189 --- src/edit.c | 11 ++++++++--- src/version.c | 2 ++ 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/edit.c b/src/edit.c index 4cb727c..c1935c2 100644 --- a/src/edit.c +++ b/src/edit.c @@ -6420,13 +6420,17 @@ stop_insert(end_insert_pos, esc) /* If we just did an auto-indent, remove the white space from the end * of the line, and put the cursor back. - * Do this when ESC was used or moving the cursor up/down. */ + * Do this when ESC was used or moving the cursor up/down. + * Check for the old position still being valid, just in case the text + * got changed unexpectedly. */ if (did_ai && (esc || (vim_strchr(p_cpo, CPO_INDENT) == NULL - && curwin->w_cursor.lnum != end_insert_pos->lnum))) + && curwin->w_cursor.lnum != end_insert_pos->lnum)) + && end_insert_pos->lnum <= curbuf->b_ml.ml_line_count) { pos_T tpos = curwin->w_cursor; curwin->w_cursor = *end_insert_pos; + check_cursor_col(); /* make sure it is not past the line */ for (;;) { if (gchar_cursor() == NUL && curwin->w_cursor.col > 0) @@ -6434,7 +6438,8 @@ stop_insert(end_insert_pos, esc) cc = gchar_cursor(); if (!vim_iswhite(cc)) break; - (void)del_char(TRUE); + if (del_char(TRUE) == FAIL) + break; /* should not happen */ } if (curwin->w_cursor.lnum != tpos.lnum) curwin->w_cursor = tpos; diff --git a/src/version.c b/src/version.c index f9860ec..d744a02 100644 --- a/src/version.c +++ b/src/version.c @@ -677,6 +677,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 189, +/**/ 188, /**/ 187, -- 2.11.4.GIT