Mercurial > vim
changeset 23509:18f3dc6974af v8.2.2297
patch 8.2.2297: Vim9: cannot set 'number' to a boolean value
Commit: https://github.com/vim/vim/commit/0ea04408653be843e37d03ce574f5d2eae682339
Author: Bram Moolenaar <Bram@vim.org>
Date: Mon Jan 4 13:37:54 2021 +0100
patch 8.2.2297: Vim9: cannot set 'number' to a boolean value
Problem: Vim9: cannot set 'number' to a boolean value.
Solution: Use tv_get_bool(). (closes https://github.com/vim/vim/issues/7615)
author | Bram Moolenaar <Bram@vim.org> |
---|---|
date | Mon, 04 Jan 2021 13:45:04 +0100 |
parents | 66c2449bef2e |
children | ecdd0d05edd8 |
files | src/evalvars.c src/testdir/test_vim9_assign.vim src/version.c |
diffstat | 3 files changed, 19 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/src/evalvars.c +++ b/src/evalvars.c @@ -1370,8 +1370,14 @@ ex_let_one( || opt_type == gov_hidden_bool || opt_type == gov_hidden_number) && (tv->v_type != VAR_STRING || !in_vim9script())) - // number, possibly hidden - n = (long)tv_get_number(tv); + { + if (opt_type == gov_bool || opt_type == gov_hidden_bool) + // bool, possibly hidden + n = (long)tv_get_bool(tv); + else + // number, possibly hidden + n = (long)tv_get_number(tv); + } // Avoid setting a string option to the text "v:false" or similar. // In Vim9 script also don't convert a number to string.
--- a/src/testdir/test_vim9_assign.vim +++ b/src/testdir/test_vim9_assign.vim @@ -161,6 +161,15 @@ def Test_assignment() assert_equal(6, &g:ts) &g:ts += 2 assert_equal(8, &g:ts) + + &number = true + assert_equal(true, &number) + &number = 0 + assert_equal(false, &number) + &number = 1 + assert_equal(true, &number) + &number = false + assert_equal(false, &number) END CheckDefAndScriptSuccess(lines)