Patch 8.2.0558

6 views
Skip to first unread message

Bram Moolenaar

unread,
Apr 12, 2020, 12:02:47 PM4/12/20
to vim...@googlegroups.com

Patch 8.2.0558
Problem: Vim9: dict code not covered by tests.
Solution: Remove dead code, adjust test case.
Files: src/vim9compile.c, src/testdir/test_vim9_expr.vim


*** ../vim-8.2.0557/src/vim9compile.c 2020-04-12 17:11:24.087578053 +0200
--- src/vim9compile.c 2020-04-12 17:58:20.388304868 +0200
***************
*** 2788,2798 ****
*arg = skipwhite(*arg + 1);
}

- if (**arg != '}')
- {
- semsg(_(e_missing_dict_end), *arg);
- goto failret;
- }
*arg = *arg + 1;

// Allow for following comment, after at least one space.
--- 2788,2793 ----
*** ../vim-8.2.0557/src/testdir/test_vim9_expr.vim 2020-04-12 16:38:54.775938935 +0200
--- src/testdir/test_vim9_expr.vim 2020-04-12 18:01:12.087711622 +0200
***************
*** 767,773 ****

call CheckDefFailure("let x = #{8: 8}", 'E1014:')
call CheckDefFailure("let x = #{xxx}", 'E720:')
! call CheckDefFailure("let x = #{xxx: 1", 'E723:')
call CheckDefFailure("let x = #{xxx: 1,", 'E723:')
call CheckDefFailure("let x = {'a': xxx}", 'E1001:')
call CheckDefFailure("let x = {xxx: 8}", 'E1001:')
--- 767,773 ----

call CheckDefFailure("let x = #{8: 8}", 'E1014:')
call CheckDefFailure("let x = #{xxx}", 'E720:')
! call CheckDefFailureMult(["let x = #{xxx: 1", "let y = 2"], 'E722:')
call CheckDefFailure("let x = #{xxx: 1,", 'E723:')
call CheckDefFailure("let x = {'a': xxx}", 'E1001:')
call CheckDefFailure("let x = {xxx: 8}", 'E1001:')
*** ../vim-8.2.0557/src/version.c 2020-04-12 17:52:49.429492390 +0200
--- src/version.c 2020-04-12 18:01:43.983602623 +0200
***************
*** 740,741 ****
--- 740,743 ----
{ /* Add new patch number below this line */
+ /**/
+ 558,
/**/

--
Mushrooms always grow in damp places and so they look like umbrellas.

/// 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