patch 8.2.0457: Test_quotestar() often fails when run under valgrind

Problem:    Test_quotestar() often fails when run under valgrind.
Solution:   Wait longer for the GUI to start.

patch 8.2.0456: Test_confirm_cmd is flaky

Problem:    Test_confirm_cmd is flaky.
Solution:   Add a term_wait() call. (closes #5854)

patch 8.2.0455: cannot set the highlight group for a specific terminal

Problem:    Cannot set the highlight group for a specific terminal.
Solution:   Add the "highlight" option to term_start(). (closes #5818)

patch 8.2.0454: some tests fail when the system is slow

Problem:    Some tests fail when the system is slow.
Solution:   Make the run number global, use in the test to increase the
            waiting time. (closes #5841)

patch 8.2.0453: trailing space in job_start() command causes empty argument

Problem:    Trailing space in job_start() command causes empty argument.
Solution:   Ignore trailing space. (closes #5851)

patch 8.2.0452: channel_parse_messages() fails when called recursively

Problem:    channel_parse_messages() fails when called recursively.
Solution:   Return for a recursive call. (closes #5835)

patch 8.2.0451: Win32: double-width character displayed incorrectly

Problem:    Win32: double-width character displayed incorrectly.
Solution:   First move the cursor to the first column. (Nobuhiro Takasaki,
            closes #5848)

patch 8.2.0450: not enough testing for restricted mode and function calls

Problem:    Not enough testing for restricted mode and function calls.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5847)

patch 8.2.0449: Vim9: crash if return type is invalid

Problem:    Vim9: crash if return type is invalid. (Yegappan Lakshmanan)
Solution:   Always return some type, not NULL.

patch 8.2.0448: various functions not properly tested

Problem:    Various functions not properly tested.
Solution:   Add more tests, especially for failures. (Yegappan Lakshmanan,
            closes #5843)

patch 8.2.0447: terminal scroll tests fails on some systems

Problem:    Terminal scroll tests fails on some systems.
Solution:   Remove the fixed 100msec wait for Win32.  Add a loop to wait until
            scrolling has finished. (James McCoy, closes #5842)

patch 8.2.0446: listener with undo of deleting all lines not tested

Problem:    Listener with undo of deleting all lines not tested.
Solution:   Add a test.

patch 8.2.0445: png and xpm files not in MS-Windows zip file

Problem:    Png and xpm files not in MS-Windows zip file.
Solution:   Move files to shared between Unix and Windows target.

patch 8.2.0444: swap file test fails on some systems

Problem:    Swap file test fails on some systems.
Solution:   Preserve the swap file. Send NL terminated keys.

patch 8.2.0443: clipboard code is spread out

Problem:    Clipboard code is spread out.
Solution:   Move clipboard code to its own file. (Yegappan Lakshmanan,
            closes #5827)

patch 8.2.0442: channel contents might be used after being freed

Problem:    Channel contents might be used after being freed.
Solution:   Reset the job channel before freeing the channel.

patch 8.2.0441: terminal noblock test is still failing on BSD

Problem:    Terminal noblock test is still failing on BSD.
Solution:   Reduce the amount of text.

patch 8.2.0440: terminal noblock test is still very flaky on BSD

Problem:    Terminal noblock test is still very flaky on BSD.
Solution:   Increase the waiting time.

patch 8.2.0439: :disassemble has minor flaws

Problem:    :disassemble has minor flaws.
Solution:   Format the code.  Use (int) instead of (char) for %c.
            (also by James McCoy, closes #5831)

patch 8.2.0438: terminal noblock test is very flaky on BSD

Problem:    Terminal noblock test is very flaky on BSD.
Solution:   Change WaitFor() to WaitForAssert() to be able to see why it
            failed.  Add a short wait in between sending keys.

patch 8.2.0437: MS-Windows installer contains old stuff

Problem:    MS-Windows installer contains old stuff.
Solution:   Rely on Windows NT. (Ken Takata, closes #5832)

patch 8.2.0436: no warnings for incorrect printf arguments

Problem:    No warnings for incorrect printf arguments.
Solution:   Fix attribute in declaration.  Fix uncovered mistakes. (Dominique
            Pelle, closes #5834)

patch 8.2.0435: channel contents might be freed twice

Problem:    Channel contents might be freed twice.
Solution:   Call either channel_free_channel() or channel_free(), not both.
            (Nobuhiro Takasaki, closes #5835)

patch 8.2.0434: MS-Windows with VTP: Normal color not working

Problem:    MS-Windows with VTP: Normal color not working.
Solution:   After changing the Normal color update the VTP console color.
            (Nobuhiro Takasaki, closes #5836)

patch 8.2.0433: INT signal not properly tested

Problem:    INT signal not properly tested.
Solution:   Add a test.  Also clean up some unnecessary lines. (Dominique
            Pelle, closes #5828)

patch 8.2.0432: a few tests fail in a huge terminal

Problem:    A few tests fail in a huge terminal.
Solution:   Make the tests pass. (Dominique Pelle, closes #5829)

patch 8.2.0431: some compilers don't support using \e for Esc

Problem:    Some compilers don't support using \e for Esc. (Yegappan
            Lakshmanan)
Solution:   use \033 instead.

patch 8.2.0430: window creation failure not properly tested

Problem:    Window creation failure not properly tested.
Solution:   Improve the test. (Yegappan Lakshmanan, closes #5826)

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)