From 5546688fb6798d37a770f366c13dedfa7e4cb417 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Sat, 21 Nov 2020 14:16:22 +0100 Subject: [PATCH] patch 8.2.2029: Coverity warns for not checking return value Problem: Coverity warns for not checking return value. Solution: Check that u_save_cursor() returns OK. --- src/ops.c | 7 ++++--- src/version.c | 2 ++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/ops.c b/src/ops.c index 417bb1621..b83394e01 100644 --- a/src/ops.c +++ b/src/ops.c @@ -3871,9 +3871,10 @@ do_pending_operator(cmdarg_T *cap, int old_col, int gui_yank) else { (void)op_delete(oap); - if (oap->motion_type == MLINE && has_format_option(FO_AUTO)) - u_save_cursor(); // cursor line wasn't saved yet - auto_format(FALSE, TRUE); + // save cursor line for undo if it wasn't saved yet + if (oap->motion_type == MLINE && has_format_option(FO_AUTO) + && u_save_cursor() == OK) + auto_format(FALSE, TRUE); } break; diff --git a/src/version.c b/src/version.c index 0583f64b9..8e9d8103c 100644 --- a/src/version.c +++ b/src/version.c @@ -750,6 +750,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 2029, /**/ 2028, /**/