patch 8.2.0429: no warning when test checks for option that never exists

Problem:    No warning when test checks for option that never exists.
Solution:   In tests check that the option can exist.

patch 8.2.0428: buffer name may leak

Problem:    Buffer name may leak.
Solution:   Free the buffer name before overwriting it.

patch 8.2.0427: it is not possible to check for a typo in a feature name

Problem:    It is not possible to check for a typo in a feature name.
Solution:   Add an extra argument to has().

patch 8.2.0426: some errors were not tested for

Problem:    Some errors were not tested for.
Solution:   Add tests. (Dominique Pelle, closes #5824)

patch 8.2.0425: code for modeless selection not sufficiently tested

Problem:    Code for modeless selection not sufficiently tested.
Solution:   Add tests.  Move mouse code functionality to a common script file.
            (Yegappan Lakshmanan, closes #5821)

patch 8.2.0424: checking for wrong return value

Problem:    Checking for wrong return value. (Tom)
Solution:   Invert the check and fix the test.

patch 8.2.0423: in some environments a few tests are expected to fail

Problem:    In some environments a few tests are expected to fail.
Solution:   Add $TEST_MAY_FAIL to list tests that should not cause make to
            fail.

patch 8.2.0422: crash when passing popup window to win_splitmove()

Problem:    Crash when passing popup window to win_splitmove(). (john Devin)
Solution:   Disallow moving a popup window. (closes #5816)

patch 8.2.0421: interrupting with CTRL-C does not always work

Problem:    Interrupting with CTRL-C does not always work.
Solution:   Recognize CTRL-C while modifyOtherKeys is set.

patch 8.2.0420: Vim9: cannot interrupt a loop with CTRL-C

Problem:    Vim9: cannot interrupt a loop with CTRL-C.
Solution:   Check for CTRL-C once in a while.  Doesn't fully work yet.

patch 8.2.0419: various memory leaks in Vim9 script code

Problem:    Various memory leaks in Vim9 script code.
Solution:   Fix the leaks. (Ozaki Kiichi, closes #5814)

patch 8.2.0418: code in eval.c not sufficiently covered by tests

Problem:    Code in eval.c not sufficiently covered by tests.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5815)

patch 8.2.0417: Travis CI config can be improved

Problem:    Travis CI config can be improved.
Solution:   Remove COVERAGE variable. Add load-snd-dummy script. add "-i NONE"
            to avoid messages about viminfo. (Ozaki Kiichi, closes #5813)

Update runtime files

patch 8.2.0416: test leaves file behind

Problem:    Test leaves file behind.
Solution:   Delete the file.

patch 8.2.0415: bsdl filetype is not detected

Problem:    Bsdl filetype is not detected.
Solution:   Add an entry in the filetype list. (Daniel Kho, closes #5810)

patch 8.2.0414: channel connect_waittime() test is flaky

Problem:    Channel connect_waittime() test is flaky.
Solution:   Set the test_is_flaky flag.  Use test_is_flaky for more tests.

patch 8.2.0413: buffer menu does not handle special buffers properly

Problem:    Buffer menu does not handle special buffers properly.
Solution:   Keep a dictionary with buffer names to reliably keep track of
            entries.
            Also trigger BufFilePre and BufFilePost for command-line and
            terminal buffers when the name changes.

patch 8.2.0412: MS-Windows: cannot use vimtutor from the start menu

Problem:    MS-Windows: cannot use vimtutor from the start menu.
Solution:   Better check for writable directory. Use the right path for the
            executable. (Wu Yongwei, closes #5774, closes #5756)

patch 8.2.0411: Mac: breakcheck is using a value from the stone ages

Problem:    Mac: breakcheck is using a value from the stone ages.
Solution:   Delete BREAKCHECK_SKIP from the Mac header file. (Ben Jackson)

patch 8.2.0410: channel test fails too often on slow Mac

Problem:    Channel test fails too often on slow Mac.
Solution:   Increase waiting time to 10 seconds.

patch 8.2.0409: search test leaves file behind

Problem:    Search test leaves file behind.
Solution:   Delete the file.  Also use Check commands.

patch 8.2.0408: delete() commented out for testing

Problem:    Delete() commented out for testing.
Solution:   Undo commenting-out.

patch 8.2.0407: no early check if :find and :sfind have an argument

Problem:    No early check if :find and :sfind have an argument.
Solution:   Add EX_NEEDARG.

patch 8.2.0406: FileReadCmd event not well tested

Problem:    FileReadCmd event not well tested.
Solution:   Add a test.

patch 8.2.0405: MSVC: build fails with some combination of features

Problem:    MSVC: build fails with some combination of features.
Solution:   Enable CHANNEL if TERMINAL is enabled. (Mike Williams)

patch 8.2.0404: writefile() error does not give a hint

Problem:    Writefile() error does not give a hint.
Solution:   Add remark about first argument.

patch 8.2.0403: when 'buftype' is "nofile" there is no overwrite check

Problem:    When 'buftype' is "nofile" there is no overwrite check.
Solution:   Also check for existing file when 'buftype' is set.
            (closes #5807)

patch 8.2.0402: setting local instead of global flag

Problem:    Setting local instead of global flag.
Solution:   Prepend "g:" to "test_is_flaky".

patch 8.2.0401: not enough test coverage for evalvars.c

Problem:    Not enough test coverage for evalvars.c.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5804)

patch 8.2.0400: not all tests using a terminal are in the list of flaky tests

Problem:    Not all tests using a terminal are in the list of flaky tests.
Solution:   Introduce the test_is_flaky flag.

patch 8.2.0399: various memory leaks

Problem:    Various memory leaks.
Solution:   Avoid the leaks. (Ozaki Kiichi, closes #5803)

patch 8.2.0398: profile test fails when two functions take same time

Problem:    Profile test fails when two functions take same time.
Solution:   Add a short sleep in once function. (closes #5797)

patch 8.2.0397: delayed screen update when using undo from Insert mode

Problem:    Delayed screen update when using undo from Insert mode.
Solution:   Update w_topline and cursor shape before sleeping. (closes #5790)

patch 8.2.0396: cmdexpand.c insufficiently tested

Problem:    Cmdexpand.c insufficiently tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5789)

patch 8.2.0395: build fails with FEAT_EVAL but without FEAT_MENU

Problem:    Build fails with FEAT_EVAL but without FEAT_MENU.
Solution:   Add #ifdef. (John Marriott)

patch 8.2.0394: Coverity complains about using NULL pointer

Problem:    Coverity complains about using NULL pointer.
Solution:   Use empty string when option value is NULL.

patch 8.2.0393: Coverity warns for not using return value

Problem:    Coverity warns for not using return value.
Solution:   Add (void).

patch 8.2.0392: Coverity warns for using array index out of range

Problem:    Coverity warns for using array index out of range.
Solution:   Add extra "if" to avoid warning.

patch 8.2.0391: CI test coverage dropped

Problem:    CI test coverage dropped.
Solution:   Set $DISPLAY also for non-GUI builds. (James McCoy, closes #5788)

patch 8.2.0390: terminal postponed scrollback test is flaky

Problem:    Terminal postponed scrollback test is flaky.
Solution:   Add delay in between sending keys.  Rename dump files.

patch 8.2.0389: delayed redraw when shifting text from Insert mode

Problem:    Delayed redraw when shifting text from Insert mode.
Solution:   Use msg_attr_keep() instead of msg(). (closes #5782)

patch 8.2.0388: printmbcharset option not tested

Problem:    Printmbcharset option not tested.
Solution:   Add a test.  Enable PostScript for AppVeyor build. (Dominique
            Pelle, closes #5783)

patch 8.2.0387: error for possible NULL argument to qsort()

Problem:    Error for possible NULL argument to qsort().
Solution:   Don't call qsort() when there is nothing to sort. (Dominique
            Pelle, closes #5780)

patch 8.2.0386: part from unfinished patch got included

Problem:    Part from unfinished patch got included.
Solution:   Undo that part.

patch 8.2.0385: menu functionality insufficiently tested

Problem:    Menu functionality insufficiently tested.
Solution:   Add tests.  Add menu_info(). (Yegappan Lakshmanan, closes #5760)

patch 8.2.0384: Travis CI has warnings

Problem:    Travis CI has warnings.
Solution:   Avoid warnings, clean up the config. (Ozaki Kiichi, closes #5779)

patch 8.2.0383: wrong feature check causes test not to be run

Problem:    Wrong feature check causes test not to be run.
Solution:   Use CheckFunction instead of CheckFeature. (Ozaki Kiichi,
            closes #5781)

patch 8.2.0382: some tests fail when run under valgrind

Problem:    Some tests fail when run under valgrind.
Solution:   Increase timeouts.

patch 8.2.0381: using freed memory with :lvimgrep and autocommand

Problem:    Using freed memory with :lvimgrep and autocommand. (extracted from
            POC by Dominique Pelle)
Solution:   Avoid deleting a dummy buffer used in a window. (closes #5777)

patch 8.2.0380: tiny popup when creating a terminal popup without minwidth

Problem:    Tiny popup when creating a terminal popup without minwidth.
Solution:   Use a default mininum size of 5 lines of 20 characters.

Update runtime files

patch 8.2.0379: gcc warns for ambiguous else

Problem:    Gcc warns for ambiguous else.
Solution:   Add braces. (Dominique Pelle, closes #5778)

patch 8.2.0378: prop_find() does not find all props

Problem:    prop_find() does not find all props.
Solution:   Check being in the start line. (Axel Forsman, closes #5776)

patch 8.2.0377: no CI test for a big-endian system

Problem:    No CI test for a big-endian system.
Solution:   Test with s390x. (James McCoy, closes #5772)

patch 8.2.0376: nasty callback test fails on some systems

Problem:    Nasty callback test fails on some systems.
Solution:   Increase the sleep time.

patch 8.2.0375: Coverity warning for not using return value

Problem:    Coverity warning for not using return value.
Solution:   Move error message to separate function.

patch 8.2.0374: using wrong printf directive for jump location

Problem:    Using wrong printf directive for jump location.
Solution:   Change "%lld" to "%d". (James McCoy, closes #5773)

patch 8.2.0373: type of term_sendkeys() is unknown

Problem:    Type of term_sendkeys() is unknown.
Solution:   Just return zero. (closes #5762)

patch 8.2.0372: prop_find() may not find text property at start of the line

Problem:    Prop_find() may not find text property at start of the line.
Solution:   Adjust the loop to find properties. (Axel Forsman, closes #5761,
            closes #5663)

patch 8.2.0371: crash with combination of terminal popup and autocmd

Problem:    Crash with combination of terminal popup and autocmd.
Solution:   Disallow closing a popup that is the current window.  Add a check
            that the current buffer is valid. (closes #5754)

patch 8.2.0370: the typebuf_was_filled flag is sometimes not reset

Problem:    The typebuf_was_filled flag is sometimes not reset, which may
            cause a hang.
Solution:   Make sure typebuf_was_filled is reset when the typeahead buffer is
            empty.

patch 8.2.0369: various Normal mode commands not fully tested

Problem:    Various Normal mode commands not fully tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5751)

patch 8.2.0368: Vim9: import that redefines local variable does not fail

Problem:    Vim9: import that redefines local variable does not fail.
Solution:   Check for already defined symbols.

patch 8.2.0367: can use :pedit in a popup window

Problem:    Can use :pedit in a popup window.
Solution:   Disallow it.

patch 8.2.0366: hardcopy command not tested enough

Problem:    Hardcopy command not tested enough.
Solution:   Add tests for printing. (Dominique Pelle, closes #5748)

patch 8.2.0365: tag kind can't be a multi-byte character

Problem:    Tag kind can't be a multi-byte character. (Marcin Szamotulski)
Solution:   Recognize multi-byte character. (closes #5724)

patch 8.2.0364: printf test failing on Haiku

Problem:    Printf test failing on Haiku.
Solution:   Make a difference between int and short. (Dominique Pelle,
            closes #5749)

patch 8.2.0363: some Normal mode commands not tested

Problem:    Some Normal mode commands not tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5746)

patch 8.2.0362: MS-Windows: channel test fails if grep is not available

Problem:    MS-Windows: channel test fails if grep is not available.
Solution:   Use another command. (Ken Takata, closes #5739)

patch 8.2.0361: internal error when using "0" for a callback

Problem:    Internal error when using "0" for a callback.
Solution:   Give a normal error. (closes #5743)

patch 8.2.0360: yaml files are only recognized by the file extension

Problem:    Yaml files are only recognized by the file extension.
Solution:   Check for a line starting with "%YAML". (Jason Franklin)

patch 8.2.0359: popup_atcursor() may hang

Problem:    popup_atcursor() may hang. (Yasuhiro Matsumoto)
Solution:   Take the decoration into account. (closes #5728)

patch 8.2.0358: insufficient testing for indent.c

Problem:    Insufficient testing for indent.c.
Solution:   Add indent tests. (Yegappan Lakshmanan, closes #5736)

patch 8.2.0357: cannot delete a text property matching both id and type

Problem:    Cannot delete a text property matching both id and type. (Axel
            Forsman)
Solution:   Add the "both" argument.

patch 8.2.0356: MS-Windows: feedkeys() with VIMDLL cannot handle CSI

Problem:    MS-Windows: feedkeys() with VIMDLL cannot handle CSI correctly.
Solution:   Modify mch_inchar() to encode CSI bytes. (Ozaki Kiichi, Ken
            Takata, closes #5726)

patch 8.2.0355: Vim9: str_val is confusing, it's a number

Problem:    Vim9: str_val is confusing, it's a number
Solution:   Rename to stnr_val.

patch 8.2.0354: Python 3.9 does not define _Py_DEC_REFTOTAL

Problem:    Python 3.9 does not define _Py_DEC_REFTOTAL. (Zdenek Dohnal)
Solution:   Remove it, it was only for debugging.

patch 8.2.0353: Vim9: while loop not tested

Problem:    Vim9: while loop not tested.
Solution:   Add test with "while", "break" and "continue"

patch 8.2.0352: FreeBSD: test for sourcing utf-8 is skipped

Problem:    FreeBSD: test for sourcing utf-8 is skipped.
Solution:   Run the matchadd_conceal test separately to avoid that setting
            'term' to "ansi" causes problems for other tests. (Ozaki Kiichi,
            closes #5721)

patch 8.2.0351: terminal in popup test is still a bit flaky

Problem:    Terminal in popup test is still a bit flaky.
Solution:   Clear and redraw before opening the popup.

patch 8.2.0350: Vim9: expression tests don't use recognized constants

Problem:    Vim9: expression tests don't use recognized constants.
Solution:   Recognize "true" and "false" as constants.  Make skipping work for
            assignment and expression evaluation.

patch 8.2.0349: Vim9: constant expression not well tested

Problem:    Vim9: constant expression not well tested.
Solution:   Add tests for "if" with constant expression.

patch 8.2.0348: Vim9: not all code tested

Problem:    Vim9: not all code tested.
Solution:   Add a few more tests. fix using "b:" in literal dictionary.

patch 8.2.0347: various code not covered by tests

Problem:    Various code not covered by tests.
Solution:   Add more test coverage. (Yegappan Lakshmanan, closes #5720)

patch 8.2.0346: Vim9: finding common list type not tested

Problem:    Vim9: finding common list type not tested.
Solution:   Add more tests.  Fix listing function.  Fix overwriting type.

patch 8.2.0345: compiler warning when building without the float feature

Problem:    Compiler warning when building without the float feature.
Solution:   Add #ifdef. (John Marriott)

patch 8.2.0344: ":def" not skipped properly

Problem:    ":def" not skipped properly.
Solution:   Add CMD_def to list of commands the require evaluation even when
            not being executed.

Update runtime files

patch 8.2.0343: Vim9: using wrong instruction, limited test coverage

Problem:    Vim9: using wrong instruction, limited test coverage.
Solution:   Use ISN_PUSHJOB.  Add a few more tests.

patch 8.2.0342: some code in ex_getln.c not covered by tests

Problem:    Some code in ex_getln.c not covered by tests.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5717)

patch 8.2.0341: using ":for" in Vim9 script gives an error

Problem:    Using ":for" in Vim9 script gives an error.
Solution:   Pass the LET_NO_COMMAND flag. (closes #5715)

patch 8.2.0340: Vim9: function and partial types not tested

Problem:    Vim9: function and partial types not tested.
Solution:   Support more for partial, add tests.

patch 8.2.0339: Vim9: function return type may depend on arguments

Problem:    Vim9: function return type may depend on arguments.
Solution:   Instead of a fixed return type use a function to figure out the
            return type.

patch 8.2.0338: build failure without the channel feature

Problem:    Build failure without the channel feature.
Solution:   Add #ifdef

patch 8.2.0337: build fails on a few systems

Problem:    Build fails on a few systems.
Solution:   Use vim_snprintf() instead of snprintf().

patch 8.2.0336: Vim9: insufficient test coverage for compiling

Problem:    Vim9: insufficient test coverage for compiling.
Solution:   Add more tests.

patch 8.2.0335: no completion for :disassemble

Problem:    No completion for :disassemble.
Solution:   Make completion work.  Also complete script-local functions if the
            name starts with "s:".

patch 8.2.0334: abort called when using test_void()

Problem:    Abort called when using test_void(). (Dominique Pelle)
Solution:   Only give an error, don't abort.

patch 8.2.0333: terminal in popup test is flaky

Problem:    Terminal in popup test is flaky.
Solution:   Make sure redraw is done before opening the popup.