commit 4997f2a605e15261c4f825494324371f271ef6a8
author Bram Moolenaar <Bram@vim.org> Thu Oct 13 14:00:45 2022 +0100
committer Bram Moolenaar <Bram@vim.org> Thu Oct 13 14:00:45 2022 +0100
tree cb74cc4914c7bf84073a5d48c5de2c87b1733ad3
parent 17822c507c03d509037c9ee5eee5cfbb201b3f01

patch 9.0.0740: prop_add_list() gives multiple errors for invalid argument

Problem:    prop_add_list() gives multiple errors for invalid argument.
Solution:   Only give one error message.

src/textprop.c

diff --git a/src/textprop.c b/src/textprop.c
index 4153e3f..9256368 100644
--- a/src/textprop.c
+++ b/src/textprop.c
@@ -353,6 +353,7 @@
     linenr_T	end_lnum;
     colnr_T	end_col;
     int		error = FALSE;
+    int		prev_did_emsg = did_emsg;
 
     if (check_for_dict_arg(argvars, 0) == FAIL
 	    || check_for_list_arg(argvars, 1) == FAIL)
@@ -389,13 +390,17 @@
 
 	pos_list = li->li_tv.vval.v_list;
 	start_lnum = list_find_nr(pos_list, 0L, &error);
-	start_col = list_find_nr(pos_list, 1L, &error);
-	end_lnum = list_find_nr(pos_list, 2L, &error);
-	end_col = list_find_nr(pos_list, 3L, &error);
+	if (!error)
+	    start_col = list_find_nr(pos_list, 1L, &error);
+	if (!error)
+	    end_lnum = list_find_nr(pos_list, 2L, &error);
+	if (!error)
+	    end_col = list_find_nr(pos_list, 3L, &error);
 	if (error || start_lnum <= 0 || start_col <= 0
-		|| end_lnum <= 0 || end_col <= 0)
+		  || end_lnum <= 0 || end_col <= 0)
 	{
-	    emsg(_(e_invalid_argument));
+	    if (prev_did_emsg == did_emsg)
+		emsg(_(e_invalid_argument));
 	    return;
 	}
 	if (prop_add_one(buf, type_name, id, NULL, 0, 0, start_lnum, end_lnum,