patch 8.2.0517: Vim9: cannot separate "func" and "func(): void"

Problem:    Vim9: cannot separate "func" and "func(): void".
Solution:   Use VAR_ANY for "any" and VAR_UNKNOWN for "no type".

patch 8.2.0516: client-server code is spread out

Problem:    Client-server code is spread out.
Solution:   Move client-server code to a new file. (Yegappan Lakshmanan,
            closes #5885)

patch 8.2.0515: some compilers cannot add to "void *"

Problem:    Some compilers cannot add to "void *".
Solution:   Cast to "char *".

patch 8.2.0514: several global functions are used in only one file

Problem:    Several global functions are used in only one file.
Solution:   Make the functions static. (Yegappan Lakshmanan, closes #5884)

patch 8.2.0513: reading past allocate memory when using varargs

Problem:    Reading past allocate memory when using varargs.
Solution:   Fix copying function argument types.

patch 8.2.0512: Vim9: no optional arguments in func type

Problem:    Vim9: no optional arguments in func type.
Solution:   Check for question mark after type.  Find function reference
            without function().

patch 8.2.0511: Cscope code not fully tested

Problem:    Cscope code not fully tested.
Solution:   Add more test cases. (Dominique Pelle, closes #5886)

patch 8.2.0510: Coverity complains about using uninitialized variable

Problem:    Coverity complains about using uninitialized variable.
Solution:   Assign a value to "scol".  Move code inside NULL check.

patch 8.2.0509: various code is not properly tested.

Problem:    various code is not properly tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5871)

patch 8.2.0508: Vim9: func and partial types not done yet

Problem:    Vim9: func and partial types not done yet
Solution:   Fill in details about func declaration, drop a separate partial
            declaration.

patch 8.2.0507: getbufvar() may get the wrong dictionary

Problem:    Getbufvar() may get the wrong dictionary. (David le Blanc)
Solution:   Check for empty name. (closes #5878)

patch 8.2.0506: Coverity complains about ignoring return value

Problem:    Coverity complains about ignoring return value.
Solution:   Add (void).

patch 8.2.0505: term_getty() not sufficiently tested

Problem:    term_getty() not sufficiently tested.
Solution:   Add more asserts. (Dominique Pelle, closes #5877)

patch 8.2.0504: Vim9: leaking scope memory when compilation fails

Problem:    Vim9: leaking scope memory when compilation fails.
Solution:   Cleanup the scope list.

patch 8.2.0503: Vim9: some code is not tested

Problem:    Vim9: some code is not tested.
Solution:   Add tests.  Fix uncovered problems.

patch 8.2.0502: Vim9: some code is not tested

Problem:    Vim9: some code is not tested.
Solution:   Add more tests.  Fix uncovered problems.

patch 8.2.0501: Vim9: script test fails when channel feature is missing

Problem:    Vim9: script test fails when channel feature is missing.
Solution:   Add a has() condition.

patch 8.2.0500: using the same loop in many places

Problem:    Using the same loop in many places.
Solution:   Define more FOR_ALL macros. (Yegappan Lakshmanan, closes #5339)

patch 8.2.0499: calling a lambda is slower than evaluating a string

Problem:    Calling a lambda is slower than evaluating a string.
Solution:   Make calling a lambda faster. (Ken Takata, closes #5727)

patch 8.2.0498: Coverity complains about uninitialized field

Problem:    Coverity complains about uninitialized field.
Solution:   Initialize the whole typval_T.

patch 8.2.0497: too verbose output from the asan build in Travis

Problem:    Too verbose output from the asan build in Travis.
Solution:   Filter out suppression messages. (Ozaki Kiichi, closes #5874)

patch 8.2.0496: Vim9: disassemble test fails

Problem:    Vim9: disassemble test fails.
Solution:   Separate test cases with recognized constant expressions.

patch 8.2.0495: Vim9: some code not tested

Problem:    Vim9: some code not tested.
Solution:   Add more tests.  Support more const expressions.

patch 8.2.0494: Vim9: asan error

Problem:    Vim9: asan error.
Solution:   Only get the type when there is one.

patch 8.2.0493: Vim9: some error messages not tested

Problem:    Vim9: some error messages not tested.
Solution:   Add more tests.  Fix uncovered bugs.

patch 8.2.0492: Vim9: some error messages not tested

Problem:    Vim9: some error messages not tested.
Solution:   Add more tests.  Remove dead code.  Fix uncovered bugs.

patch 8.2.0491: cannot recognize a <script> mapping using maparg()

Problem:    Cannot recognize a <script> mapping using maparg().
Solution:   Add the "script" key. (closes #5873)

patch 8.2.0490: Win32: VTP doesn't respect 'restorescreen'

Problem:    Win32: VTP doesn't respect 'restorescreen'.
Solution:   Use escape codes to switch to alternate screen. (Nobuhiro
            Takasaki, closes #5872)

patch 8.2.0489: Vim9: memory leaks

Problem:    Vim9: memory leaks.
Solution:   Free memory in the right place.  Add hints for using asan.

patch 8.2.0488: Vim9: compiling can break when using a lambda inside :def

Problem:    Vim9: Compiling can break when using a lambda inside :def.
Solution:   Do not keep a pointer to the dfunc_T for longer time.

patch 8.2.0487: Vim9: compiling not sufficiently tested

Problem:    Vim9: compiling not sufficiently tested.
Solution:   Add more tests.  Fix bug with PCALL.

patch 8.2.0486: Vim9: some code and error messages not tested

Problem:    Vim9: some code and error messages not tested.
Solution:   Add more tests.

patch 8.2.0485: Vim9 script test fails

Problem:    Vim9 script test fails.
Solution:   Stricter condition for adding new local variable.

patch 8.2.0484: Vim9: some error messages not tested

Problem:    Vim9: some error messages not tested.
Solution:   Add more tests.

patch 8.2.0483: Vim9: "let x = x + 1" does not give an error

Problem:    Vim9: "let x = x + 1" does not give an error.
Solution:   Hide the variable when compiling the expression.

patch 8.2.0482: channel and sandbox code not sufficiently tested

Problem:    Channel and sandbox code not sufficiently tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5855)

patch 8.2.0481: Travis is still using trusty

Problem:    Travis is still using trusty.
Solution:   Adjust config to use bionic. (Ozaki Kiichi, closes #5868)

patch 8.2.0480: Vim9: some code is not tested

Problem:    Vim9: some code is not tested.
Solution:   Add more tests.

patch 8.2.0479: unloading shared libraries on exit has no purpose

Problem:    Unloading shared libraries on exit has no purpose.
Solution:   Do not unload shared libraries on exit.

patch 8.2.0478: new buffers are not added to the Buffers menu

Problem:    New buffers are not added to the Buffers menu.
Solution:   Turn number into string. (Yee Cheng Chin, closes #5864)

patch 8.2.0477: Vim9: error messages not tested

Problem:    Vim9: error messages not tested.
Solution:   Add more tests.

patch 8.2.0476: terminal nasty callback test fails sometimes

Problem:    Terminal nasty callback test fails sometimes.
Solution:   use term_wait() instead of a sleep. (Yee Cheng Chin,closes #5865)

patch 8.2.0475: channel out_cb test still fails sometimes on Mac

Problem:    Channel out_cb test still fails sometimes on Mac.
Solution:   Use an ever longer timeout.

patch 8.2.0474: cannot use :write when using a plugin with BufWriteCmd

Problem:    Cannot use :write when using a plugin with BufWriteCmd.
Solution:   Reset BF_NOTEDITED after BufWriteCmd. (closes #5807)

patch 8.2.0473: variables declared in an outer scope

Problem:    Variables declared in an outer scope.
Solution:   Decleare variables only in the scope where they are used.

patch 8.2.0472: terminal highlight name is set twice, leaking memory

Problem:    Terminal highlight name is set twice, leaking memory.
Solution:   Delete one.

patch 8.2.0471: missing change to compile_list()

Problem:    Missing change to compile_list().
Solution:   Add error message.

patch 8.2.0470: Test_confirm_cmd_cancel() can fail on a slow system

Problem:    Test_confirm_cmd_cancel() can fail on a slow system.
Solution:   Use WaitForAssert(). (Ozaki Kiichi, closes #5861)

patch 8.2.0469: Vim9: no error for missing ] after list

Problem:    Vim9: no error for missing ] after list.
Solution:   Add error message. Add more tests.

patch 8.2.0468: GUI: pixel dust with some fonts and characters

Problem:    GUI: pixel dust with some fonts and characters.
Solution:   Always redraw the character before the cursor. (Nir Lichtman,
            closes #5549, closes #5856)

patch 8.2.0467: Vim9: some errors are not tested

Problem:    Vim9: some errors are not tested
Solution:   Add more tests.  Fix that Vim9 script flag is not reset.

patch 8.2.0466: not parsing messages recursively breaks the govim plugin

Problem:    Not parsing messages recursively breaks the govim plugin.
Solution:   When called recursively do handle messages but do not close
            channels.

patch 8.2.0465: Vim9: dead code and wrong return type

Problem:    Vim9: dead code and wrong return type.
Solution:   Remove dead code.  Fix return type.  Add more tests.

Correct list of patch numbers

patch 8.2.0464: typos and other small problems

Problem:    Typos and other small problems.
Solution:   Fix the typos.  Add missing files to the distribution.

patch 8.2.0464: typos and other small problems

Problem:    Typos and other small problems.
Solution:   Fix the typos.  Add missing file to distribution.

Update runtime files

patch 8.2.0462: previewwindow test fails on some systems

Problem:    Previewwindow test fails on some systems. (James McCoy)
Solution:   Wait a bit after sending the "o". (closes #5849)

patch 8.2.0461: confirm test fails on amd64 system

Problem:    Confirm test fails on amd64 system. (Alimar Riesebieter)
Solution:   Add an extra WaitForAssert(). (Dominique Pelle)

patch 8.2.0460: build failure because of wrong feature name

Problem:    Build failure because of wrong feature name.
Solution:   Correct feature name.

patch 8.2.0459: cannot check if a function name is correct

Problem:    Cannot check if a function name is correct.
Solution:   Add "?funcname" to exists().

patch 8.2.0458: missing feature check in test function

Problem:    Missing feature check in test function.
Solution:   Add check commands.

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.