To: vim_dev@googlegroups.com Subject: Patch 8.2.2029 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 8.2.2029 Problem: Coverity warns for not checking return value. Solution: Check that u_save_cursor() returns OK. Files: src/ops.c *** ../vim-8.2.2028/src/ops.c 2020-11-18 11:34:30.178191415 +0100 --- src/ops.c 2020-11-21 14:15:17.860940587 +0100 *************** *** 3871,3879 **** 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); } break; --- 3871,3880 ---- else { (void)op_delete(oap); ! // 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; *** ../vim-8.2.2028/src/version.c 2020-11-21 14:03:39.851158161 +0100 --- src/version.c 2020-11-21 14:16:00.984817194 +0100 *************** *** 752,753 **** --- 752,755 ---- { /* Add new patch number below this line */ + /**/ + 2029, /**/ -- Facepalm statement #9: "Did you see, there is now even a hobbit book" /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org ///