commit 8088ae95bbed2085c5fb196850c4e4b8df55c989
author Bram Moolenaar <Bram@vim.org> Mon Jun 20 11:38:17 2022 +0100
committer Bram Moolenaar <Bram@vim.org> Mon Jun 20 11:38:17 2022 +0100
tree c1507cdeb643ca125e72d4a1158c2b7ac018c6fa
parent 509ce0383105d5b516ef2d489adb7c554d3c8f81

patch 8.2.5138: various small issues

Problem:    Various small issues.
Solution:   Various small improvments.

src/filepath.c

diff --git a/src/filepath.c b/src/filepath.c
index 654d03a..854d823 100644
--- a/src/filepath.c
+++ b/src/filepath.c
@@ -1314,7 +1314,7 @@
 	if (rettv->v_type == VAR_STRING)
 	    rettv->vval.v_string = ExpandOne(&xpc, tv_get_string(&argvars[0]),
 						     NULL, options, WILD_ALL);
-	else if (rettv_list_alloc(rettv) != FAIL)
+	else if (rettv_list_alloc(rettv) == OK)
 	{
 	  int i;
 
@@ -1395,7 +1395,7 @@
 	globpath(tv_get_string(&argvars[0]), file, &ga, flags);
 	if (rettv->v_type == VAR_STRING)
 	    rettv->vval.v_string = ga_concat_strings(&ga, "\n");
-	else if (rettv_list_alloc(rettv) != FAIL)
+	else if (rettv_list_alloc(rettv) == OK)
 	    for (i = 0; i < ga.ga_len; ++i)
 		list_append_string(rettv->vval.v_list,
 					    ((char_u **)(ga.ga_data))[i], -1);

src/job.c

diff --git a/src/job.c b/src/job.c
index 7dbfc7d..259d49c 100644
--- a/src/job.c
+++ b/src/job.c
@@ -222,7 +222,8 @@
 		opt->jo_io_buf[part] = tv_get_number(item);
 		if (opt->jo_io_buf[part] <= 0)
 		{
-		    semsg(_(e_invalid_value_for_argument_str_str), hi->hi_key, tv_get_string(item));
+		    semsg(_(e_invalid_value_for_argument_str_str),
+					      hi->hi_key, tv_get_string(item));
 		    return FAIL;
 		}
 		if (buflist_findnr(opt->jo_io_buf[part]) == NULL)

src/mark.c

diff --git a/src/mark.c b/src/mark.c
index e7addbe..ade5a10 100644
--- a/src/mark.c
+++ b/src/mark.c
@@ -1484,7 +1484,7 @@
 {
     buf_T	*buf = NULL;
 
-    if (rettv_list_alloc(rettv) != OK)
+    if (rettv_list_alloc(rettv) == FAIL)
 	return;
 
     if (in_vim9script() && check_for_opt_buffer_arg(argvars, 0) == FAIL)

src/move.c

diff --git a/src/move.c b/src/move.c
index 5865af8..c2e2712 100644
--- a/src/move.c
+++ b/src/move.c
@@ -315,7 +315,7 @@
 	    if (curwin->w_cursor.lnum < curwin->w_botline)
 	    {
 	      if (((long)curwin->w_cursor.lnum
-					     >= (long)curwin->w_botline - *so_ptr
+					   >= (long)curwin->w_botline - *so_ptr
 #ifdef FEAT_FOLDING
 			|| hasAnyFolding(curwin)
 #endif
@@ -378,7 +378,7 @@
 		else
 #endif
 		    line_count = curwin->w_cursor.lnum - curwin->w_botline
-								   + 1 + *so_ptr;
+								 + 1 + *so_ptr;
 		if (line_count <= curwin->w_height + 1)
 		    scroll_cursor_bot(scrolljump_value(), FALSE);
 		else
@@ -1986,7 +1986,7 @@
 
 /*
  * Recompute topline to put the cursor at the bottom of the window.
- * Scroll at least "min_scroll" lines.
+ * When scrolling scroll at least "min_scroll" lines.
  * If "set_topbot" is TRUE, set topline and botline first (for "zb").
  * This is messy stuff!!!
  */

src/popupwin.c

diff --git a/src/popupwin.c b/src/popupwin.c
index bb889d5..dd87705 100644
--- a/src/popupwin.c
+++ b/src/popupwin.c
@@ -2983,7 +2983,7 @@
     win_T	*wp;
     tabpage_T	*tp;
 
-    if (rettv_list_alloc(rettv) != OK)
+    if (rettv_list_alloc(rettv) == FAIL)
 	return;
     FOR_ALL_POPUPWINS(wp)
 	list_append_number(rettv->vval.v_list, wp->w_id);

src/version.c

diff --git a/src/version.c b/src/version.c
index 584d9e1..0422787 100644
--- a/src/version.c
+++ b/src/version.c
@@ -735,6 +735,8 @@
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    5138,
+/**/
     5137,
 /**/
     5136,