patch 8.2.1638: leaking memory when popup filter function can't be called

Problem:    Leaking memory when popup filter function can't be called.
Solution:   Don't return too soon.

patch 8.2.1637: Vim9: :put ={expr} does not work inside :def function

Problem:    Vim9: :put ={expr} does not work inside :def function.
Solution:   Add ISN_PUT. (closes #6397)

patch 8.2.1636: get stuck if a popup filter causes an error

Problem:    Get stuck if a popup filter causes an error.
Solution:   Check whether the function can be called and does not cause an
            error.  (closes #6902)

patch 8.2.1635: no digraph for 0x2022 BULLET

Problem:    No digraph for 0x2022 BULLET.
Solution:   Use "oo". (Hans Ginzel, closes #6904)

Update runtime files.

patch 8.2.1634: loop to handle keys for the command line is too long

Problem:    Loop to handle keys for the command line is too long.
Solution:   Move a few more parts to separate functions. (Yegappan Lakshmanan,
            closes #6895)

patch 8.2.1633: some error messages are internal but do not use iemsg()

Problem:    Some error messages are internal but do not use iemsg().
Solution:   Use iemsg(). (Dominique Pellé, closes #6894)

patch 8.2.1632: not checking the context of test_fails()

Problem:    Not checking the context of test_fails().
Solution:   Add the line number and context arguments.  Give error if
            assert_fails() argument types are wrong.

patch 8.2.1631: test_fails() does not check the context of the line number

Problem:    test_fails() does not check the context of the line number.
Solution:   Use another argument to specify the context of the line number.

patch 8.2.1630: terminal test fails

Problem:    Terminal test fails.
Solution:   Correct argument to term_start().  Correct error number.

patch 8.2.1629: test fails without terminal feature

Problem:    Test fails without terminal feature.
Solution:   Check for terminal feature.

patch 8.2.1628: Vim9: cannot pass "true" to timer_paused()

Problem:    Vim9: cannot pass "true" to timer_paused().
Solution:   Use tv_get_bool(). (closes #6891)

patch 8.2.1627: Vim9: cannot pass "true" to submatch/term_gettty/term_start

Problem:    Vim9: cannot pass "true" to submatch(), term_gettty() and
            term_start()
Solution:   Use tv_get_bool_chk(). (closes #6888, closes #6890, closes #6889)

patch 8.2.1626: test for strchars() fails with different error number

Problem:    Test for strchars() fails with different error number.
Solution:   Adjust the error number.

patch 8.2.1625: compiler warning for use of fptr_T

Problem:    Compiler warning for use of fptr_T.
Solution:   Make the type less strict.

patch 8.2.1624: Vim9: cannot pass "true" to split(), str2nr() and strchars()

Problem:    Vim9: cannot pass "true" to split(), str2nr() and strchars().
Solution:   Use tv_get_bool_chk(). (closes #6884, closes #6885, closes #6886)

patch 8.2.1623: Vim9: using :call where it is not needed

Problem:    Vim9: using :call where it is not needed.
Solution:   Remove :call. (closes #6892)

patch 8.2.1622: loop to handle keys for the command line is too long

Problem:    Loop to handle keys for the command line is too long.
Solution:   Move code to functions. (Yegappan Lakshmanan, closes #6880)

patch 8.2.1621: crash when using submatch(0, 1) in substitute()

Problem:    Crash when using submatch(0, 1) in substitute().
Solution:   Increment reference count. (closes #6887)

patch 8.2.1620: searchcount() test fails

Problem:    searchcount() test fails.
Solution:   Restore default flag value.

patch 8.2.1619: Vim9: cannot pass "true" to spellsuggest()

Problem:    Vim9: cannot pass "true" to spellsuggest().
Solution:   Use tv_get_bool_chk(). (closes #6883)

patch 8.2.1618: Vim9: cannot pass "true" to setloclist()

Problem:    Vim9: cannot pass "true" to setloclist().
Solution:   Use dict_get_bool(). (closes #6882)

patch 8.2.1617: Vim9: cannot pass "true" to win_splitmove()

Problem:    Vim9: cannot pass "true" to win_splitmove().
Solution:   Use dict_get_bool(). (closes #6862)  Alphabetize test functions.

patch 8.2.1616: Vim9: cannot pass "true" to synID()

Problem:    Vim9: cannot pass "true" to synID().
Solution:   Use tv_get_bool_chk(). (closes #6860)

patch 8.2.1615: Vim9: cannot pass "true" to searchdecl()

Problem:    Vim9: cannot pass "true" to searchdecl().
Solution:   use tv_get_bool_chk(). (closes #6881)

patch 8.2.1614: Vim9: cannot pass "true" to searchcount()

Problem:    Vim9: cannot pass "true" to searchcount().
Solution:   Use tv_get_bool_chk(). (closes #6854)

patch 8.2.1613: Vim9: cannot pass "true" to prop_type_add()

Problem:    Vim9: cannot pass "true" to prop_type_add().
Solution:   Use tv_get_bool(). (closes #6850)

patch 8.2.1612: Vim9: cannot pass "true" to prop_remove()

Problem:    Vim9: cannot pass "true" to prop_remove().
Solution:   Use dict_get_bool(). (closes #6853)

patch 8.2.1611: Vim9: cannot pass "true" to nr2char()

Problem:    Vim9: cannot pass "true" to nr2char().
Solution:   use tv_get_bool_chk(). (closes #6878)

patch 8.2.1610: Vim9: cannot pass "true" to list2str() and str2list()

Problem:    Vim9: cannot pass "true" to list2str() and str2list().
Solution:   Use tv_get_bool_chk(). (closes #6877)

patch 8.2.1609: Vim9: test fails when build without +channel

Problem:    Vim9: test fails when build without +channel.
Solution:   Add check for +channel. (closes #6879)

patch 8.2.1608: Vim9: getchar() test fails with GUI

Problem:    Vim9: getchar() test fails with GUI.
Solution:   Avoid that getchar(0) gets stuck on K_IGNORE.

patch 8.2.1607: Vim9: getchar() test fails on MS-Windows

Problem:    Vim9: getchar() test fails on MS-Windows.
Solution:   First consume any available input.

patch 8.2.1606: Vim9: cannot use "true" with has()

Problem:    Vim9: cannot use "true" with has().
Solution:   Use tv_get_bool(). (closes #6876)

patch 8.2.1605: default maintainer on github is wrong

Problem:    Defyyyyyyyyyyyyyyyyyyyyyyyub is wrong.
Solution:   Use Bram's account.

patch 8.2.1604: Vim9: cannot use "true" with getcompletion()

Problem:    Vim9: cannot use "true" with getcompletion().
Solution:   use tv_get_bool_chk(). (closes #6875)

patch 8.2.1603: Vim9: cannot use "true" with getchar()

Problem:    Vim9: cannot use "true" with getchar().
Solution:   use tv_get_bool_chk(). (closes #6874)

patch 8.2.1602: Vim9: cannot use 'true" with getbufinfo()

Problem:    Vim9: cannot use 'true" with getbufinfo().
Solution:   Use dict_get_bool(). (closes #6873)

patch 8.2.1601: Vim9: cannot use 'true" with garbagecollect()

Problem:    Vim9: cannot use 'true" with garbagecollect().
Solution:   Use tv_get_bool(). (closes #6871)

patch 8.2.1600: Vim9: cannot use "true" with deepcopy()

Problem:    Vim9: cannot use "true" with deepcopy().
Solution:   Use tv_get_bool_chk(). (closes #6867)

patch 8.2.1599: missing line end when skipping a long line with :cgetfile

Problem:    Missing line end when skipping a long line with :cgetfile.
Solution:   Fix off-by-one error. (closes #6870)

patch 8.2.1598: starting a hidden terminal resizes the current window

Problem:    Starting a hidden terminal resizes the current window.
Solution:   Do not resize the current window for a hidden terminal.
            (closes #6872)

patch 8.2.1597: the channel source file is too big

Problem:    The channel source file is too big.
Solution:   Move job related code to a new source file.

patch 8.2.1596: using win_screenpos('.') in tests works but is wrong

Problem:    Using win_screenpos('.') in tests works but is wrong.
Solution:   Use win_screenpos(0).

patch 8.2.1595: cannot easily see what Vim sends to the terminal

Problem:    Cannot easily see what Vim sends to the terminal.
Solution:   Write output to the channel log if it contains terminal control
            sequences.  Avoid warnings for tputs() argument.

patch 8.2.1594: pull requests on github do not notify a maintainer

Problem:    Pull requests on github do not notify a maintainer.
Solution:   Add a CODEOWNERS file with a few initial entries.

patch 8.2.1593: tests do not check the error number properly

Problem:    Tests do not check the error number properly.0
Solution:   Add a colon after the error number. (closes #6869)

patch 8.2.1592: Vim9: passing "true" to char2nr() fails

Problem:    Vim9: passing "true" to char2nr() fails.
Solution:   Use tv_get_bool_chk(). (closes #6865)

patch 8.2.1591: using winheight('.') in tests works but is wrong

Problem:    Using winheight('.') in tests works but is wrong.
Solution:   Use winheight(0). (issue #6863)

patch 8.2.1590: Vim9: bufnr() doesn't take "true" argument

Problem:    Vim9: bufnr() doesn't take "true" argument.
Solution:   use tv_get_bool_chk(). (closes #6863)

patch 8.2.1589: term_start() options for size are overruled by 'termwinsize'

Problem:    Term_start() options for size are overruled by 'termwinsize'.
            (Sergey Vlasov)
Solution:   Set 'termwinsize' to the specified size.

patch 8.2.1588: cannot read back the prompt of a prompt buffer

Problem:    Cannot read back the prompt of a prompt buffer.
Solution:   Add prompt_getprompt(). (Ben Jackson, closes #6851)

patch 8.2.1587: loop for handling keys for the command line is too long

Problem:    Loop for handling keys for the command line is too long.
Solution:   Move wild menu handling to separate functions. (Yegappan
            Lakshmanan, closes #6856)

patch 8.2.1586: :resize command not fully tested

Problem:    :resize command not fully tested.
Solution:   Add a couple of tests. (Dominique Pellé, closes #6857)

patch 8.2.1585: messages in globals.h not translated

Problem:    Messages in globals.h not translated, xgettext on MS-Windows not
            fully supported.
Solution:   Add globals.h to list of input files.  Update MS-Windows makefiles
            to improve message translations. (Ken Takata, closes #6858)

patch 8.2.1584: Vim9: cannot use "true" for "skipstart" in prop_find()

Problem:    Vim9: cannot use "true" for "skipstart" in prop_find().
Solution:   Use dict_get_bool() instead of tv_get_number(). (closes #6852)

patch 8.2.1583: MS-Windows: cannot easily measure code coverage

Problem:    MS-Windows: cannot easily measure code coverage.
Solution:   Add the COVERAGE option. (Ken Takata, closes #6842)

patch 8.2.1582: the channel log does not show typed text

Problem:    The channel log does not show typed text.
Solution:   Add raw typed text to the log file.

patch 8.2.1581: using line() for global popup window doesn't work

Problem:    Using line() for global popup window doesn't work.
Solution:   Set tabpage to "curtab". (closes #6847)

patch 8.2.1580: wildmenu does not work properly

Problem:    Wildmenu does not work properly.
Solution:   Do not call may_do_incsearch_highlighting() if completion is in
            progress.

patch 8.2.1579: reports from asan are not optimal

Problem:    Reports from asan are not optimal.
Solution:   Use clang with ubsan. (James McCoy, closes #6811)

patch 8.2.1578: Vim9: popup_clear() does not take "true" as argument

Problem:    Vim9: popup_clear() does not take "true" as argument.
Solution:   Use tv_get_bool(). (closes #6826)

patch 8.2.1577: Vim9: hasmapto()/mapcheck()/maparg() do nottake "true" arg

Problem:    Vim9: hasmapto(), mapcheck() and maparg() do not take "true" as
            argument.
Solution:   Use tv_get_bool(). (closes #6822, closes #6824)

patch 8.2.1576: Vim9: index() does not take "true" as argument

Problem:    Vim9: index() does not take "true" as argument.
Solution:   Use tv_get_bool_chk(). (closes #6823)

patch 8.2.1575: Vim9: globpath() doesnot take "true" as argument

Problem:    Vim9: globpath() doesnot take "true" as argument.
Solution:   Use tv_get_bool_chk(). (closes #6821)

patch 8.2.1574: Vim9: glob() doesnot take "true" as argument

Problem:    Vim9: glob() doesnot take "true" as argument.
Solution:   Use tv_get_bool_chk(). (closes #6821)

patch 8.2.1573: Vim9: getreg() does not take "true" as argument

Problem:    Vim9: getreg() does not take "true" as argument.
Solution:   Use tv_get_bool_chk().  (closes #6820)

patch 8.2.1572: Vim9: expand() does not take "true" as argument

Problem:    Vim9: expand() does not take "true" as argument.
Solution:   Use tv_get_bool_chk().  (closes #6819)

patch 8.2.1571: Vim9: count() third argument cannot be "true"

Problem:    Vim9: count() third argument cannot be "true".
Solution:   use tv_get_bool_chk(). (closes #6818)

patch 8.2.1570: configure check for dirfd() does not work on HPUX

Problem:    Configure check for dirfd() does not work on HPUX. (Michael Osipov)
Solution:   Use AC_TRY_LINK instead of AC_TRY_COMPILE. (closes #6838)

patch 8.2.1569: Vim9: fixes not tested; failure in getchangelist()

Problem:    Vim9: fixes for functions not tested; failure in getchangelist().
Solution:   Add tests. (closes #6813, closes #6815, closes #6817)

patch 8.2.1568: prop_find() skips properties in the same line

Problem:    prop_find() skips properties in the same line if "skipstart" is
            used.
Solution:   Use "continue" instead of "break". (closes #6840)

patch 8.2.1567: no example to use ubsan with clang

Problem:    No example to use ubsan with clang.
Solution:   Add example commands. (Dominique Pellé, issue #6811)

patch 8.2.1566: not all Bazel files are recognized

Problem:    Not all Bazel files are recognized.
Solution:   Add *.bazel and *.BUILD. (closes #6836)

patch 8.2.1565: spellfile test sometimes fails

Problem:    Spellfile test sometimes fails.
Solution:   Check running into the end of the file.

patch 8.2.1564: a few remaining errors from ubsan

Problem:    A few remaining errors from ubsan.
Solution:   Avoid the warnings. (Dominique Pellé, closes #6837)

patch 8.2.1563: Vim9: error when using '%" with setbufvar() r getbufvar()

Problem:    Vim9: error when using '%" with setbufvar() or getbufvar().
Solution:   Use tv_get_buf_from_arg(). (closes #6816)

patch 8.2.1562: Vim9: error when using "%" where a buffer is expected

Problem:    Vim9: error when using "%" where a buffer is expected.
Solution:   Add tv_get_buf_from_arg(). (closes #6814)

patch 8.2.1561: using NULL pointers in fold code

Problem:    Using NULL pointers in fold code.
Solution:   Avoid using a NULL pointer. (Dominique Pellé, closes #6831,
            closes #6831)

patch 8.2.1560: using NULL pointers in some code

Problem:    Using NULL pointers in some code. (James McCoy)
Solution:   Avoid adding to a NULL pointer.  Use byte as unsigned.

patch 8.2.1559: s390x tests work again

Problem:    s390x tests work again.
Solution:   re-enable s390x tests. (James McCoy, closes #6829)

patch 8.2.1558: signs test fails

Problem:    Signs test fails.
Solution:   Add missing change to sign.c.

patch 8.2.1557: crash in :vimgrep when started as "vim -n"

Problem:    Crash in :vimgrep when started as "vim -n". (Raul Segura)
Solution:   Check mfp pointer. (Yegappan Lakshmanan, closes #6827)

patch 8.2.1556: cursorline highlighting always overrules sign highlighting

Problem:    Cursorline highlighting always overrules sign highlighting.
Solution:   Combine the highlighting, use the priority to decide how.
            (closes #6812)

patch 8.2.1555: not all tests are executed on Github Actions

Problem:    Not all tests are executed on Github Actions.
Solution:   Copy "src" to "src2" earlier. Recognize "src2" in a couple more
            places.  Add two tests to the list of flaky tests. (Ken Takata,
            closes #6798)

patch 8.2.1554: crash in normal test

Problem:    Crash in normal test.
Solution:   Skip adjusting marks if there are no folds.

patch 8.2.1553: crash in edit test

Problem:    Crash in edit test.
Solution:   Avoid using invalid pointer.

patch 8.2.1552: warnings from asan with clang-11

Problem:    Warnings from asan with clang-11. (James McCoy)
Solution:   Avoid using a NULL pointer. (issue #6811)

patch 8.2.1551: Vim9: error for argument type does not mention the number

Problem:    Vim9: error for argument type does not mention the number.
Solution:   Pass the argument number to where the error is given.

patch 8.2.1550: Vim9: bufname('%') gives an error

Problem:    Vim9: bufname('%') gives an error.
Solution:   Only give an error for wrong argument type. (closes #6807)

patch 8.2.1549: "r" fails if 'esckeys' is off and modifyOtherKeys is used

Problem:    The "r" command fails for keys with modifiers if 'esckeys' is off
            and modifyOtherKeys is used. (Lauri Tirkkonen)
Solution:   Temporarily disable bracketed paste and modifyOtherKeys if
            'esckeys' is off. (closes #6809)

patch 8.2.1548: cannot move position of "%%" in message translations

Problem:    Cannot move position of "%%" in message translations. (Emir Sarı)
Solution:   Improve the check script.

patch 8.2.1547: various comment problems

Problem:    Various comment problems.
Solution:   Update comments.

patch 8.2.1546: build rule for Vim.app is unused

Problem:    Build rule for Vim.app is unused.
Solution:   Delete the related build rules.

patch 8.2.1545: ch_logfile() is unclear about closing when forking

Problem:    ch_logfile() is unclear about closing when forking.
Solution:   Adjust the log messages.

Update runtime files.

patch 8.2.1544: cannot translate messages in a Vim script

Problem:    Cannot translate messages in a Vim script.
Solution:   Add gettext().  Try it out for a few messages in the options
            window.

patch 8.2.1543: Vim9: test with invalid SID is skipped in the GUI

Problem:    Vim9: test with invalid SID is skipped in the GUI.
Solution:   Read the CTRL-C that feedkeys() put in typeahead.

patch 8.2.1542: Vim9: test with invalid SID does not work in the GUI

Problem:    Vim9: test with invalid SID does not work in the GUI.
Solution:   Skip the test in the GUI.

patch 8.2.1541: Vim9: cannot find function reference for s:Func

Problem:    Vim9: cannot find function reference for s:Func.
Solution:   Recognize <SNR> prefix. (closes #6805)