Patch 8.2.0829

8 views
Skip to first unread message

Bram Moolenaar

unread,
May 26, 2020, 2:22:43 PM5/26/20
to vim...@googlegroups.com

Patch 8.2.0829
Problem: filter() may give misleading error message.
Solution: Also mention Blob as an allowed argument.
Files: src/list.c, src/testdir/test_filter_map.vim


*** ../vim-8.2.0828/src/list.c 2020-05-14 22:41:10.229637563 +0200
--- src/list.c 2020-05-26 20:16:34.288543540 +0200
***************
*** 1768,1774 ****
}
else
{
! semsg(_(e_listdictarg), ermsg);
return;
}

--- 1768,1774 ----
}
else
{
! semsg(_(e_listdictblobarg), ermsg);
return;
}

*** ../vim-8.2.0828/src/testdir/test_filter_map.vim 2020-04-25 15:24:40.551354115 +0200
--- src/testdir/test_filter_map.vim 2020-05-26 20:21:12.959726456 +0200
***************
*** 92,99 ****
func Test_map_filter_fails()
call assert_fails('call map([1], "42 +")', 'E15:')
call assert_fails('call filter([1], "42 +")', 'E15:')
! call assert_fails("let l = map('abc', '\"> \" . v:val')", 'E712:')
! call assert_fails("let l = filter('abc', '\"> \" . v:val')", 'E712:')
call assert_fails("let l = filter([1, 2, 3], '{}')", 'E728:')
call assert_fails("let l = filter({'k' : 10}, '{}')", 'E728:')
call assert_fails("let l = filter([1, 2], {})", 'E731:')
--- 92,99 ----
func Test_map_filter_fails()
call assert_fails('call map([1], "42 +")', 'E15:')
call assert_fails('call filter([1], "42 +")', 'E15:')
! call assert_fails("let l = map('abc', '\"> \" . v:val')", 'E896:')
! call assert_fails("let l = filter('abc', '\"> \" . v:val')", 'E896:')
call assert_fails("let l = filter([1, 2, 3], '{}')", 'E728:')
call assert_fails("let l = filter({'k' : 10}, '{}')", 'E728:')
call assert_fails("let l = filter([1, 2], {})", 'E731:')
*** ../vim-8.2.0828/src/version.c 2020-05-26 20:09:06.910102525 +0200
--- src/version.c 2020-05-26 20:17:30.040351991 +0200
***************
*** 748,749 ****
--- 748,751 ----
{ /* Add new patch number below this line */
+ /**/
+ 829,
/**/

--
hundred-and-one symptoms of being an internet addict:
186. You overstay in the office so you can have more time surfing the net.

/// Bram Moolenaar -- Br...@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 ///
Reply all
Reply to author
Forward
0 new messages