patch 8.2.5157: MS-Windows GUI: CTRL-key combinations do not always work

Problem:    MS-Windows GUI: CTRL-key combinations do not always work.
Solution:   Handle special key combinations better. (closes #10613,
            closes #10602, closes #10579)

patch 8.2.5156: search timeout test often fails with FreeBSD

Problem:    Search timeout test often fails with FreeBSD.
Solution:   Double the maximum time.

patch 8.2.5155: in diff mode windows may get out of sync

Problem:    In diff mode windows may get out of sync. (Gary Johnson)
Solution:   Avoid that the other window scrolls for 'cursorbind'.

Update runtime files

patch 8.2.5154: still mentioning version8, some cosmetic issues

Problem:    Still mentioning version8, some cosmetic issues.
Solution:   Prefer mentioning version9, cosmetic improvements.

patch 8.2.5153: "make uninstall" does not remove colors/lists

Problem:    "make uninstall" does not remove colors/lists.
Solution:   Add a line to the Makefile. (closes #10609)

Update runtime files

patch 8.2.5152: search() gets stuck with "c" and skip evaluates to true

Problem:    search() gets stuck with "c" and skip evaluates to true.
Solution:   Reset the SEARCH_START option. (closes #10608)

patch 8.2.5151: reading beyond the end of the line with lisp indenting

Problem:    Reading beyond the end of the line with lisp indenting.
Solution:   Avoid going over the NUL at the end of the line.

patch 8.2.5150: read past the end of the first line with ":0;'{"

Problem:    Read past the end of the first line with ":0;'{".
Solution:   When on line zero check the column is valid for line one.

patch 8.2.5149: cannot build without the +eval feature

Problem:    Cannot build without the +eval feature. (Tony Mechelynck)
Solution:   Add #ifdefs.

patch 8.2.5148: invalid memory access when using expression on command line

Problem:    Invalid memory access when using an expression on the command line.
Solution:   Make sure the position does not go negative.

patch 8.2.5147: flaky test always fails on retry

Problem:    Flaky test always fails on retry.
Solution:   Delete the created function.

patch 8.2.5146: memory leak when substitute expression nests

Problem:    Memory leak when substitute expression nests.
Solution:   Use an array of expression results.

patch 8.2.5145: exit test causes spurious valgrind reports

Problem:    Exit test causes spurious valgrind reports.
Solution:   Skip test.  Add CheckNotValgrind.

patch 8.2.5144: with 'lazyredraw' set completion menu may be wrong

Problem:    With 'lazyredraw' set completion menu may be displayed wrong.
Solution:   When the popup menu is visible do not insert a screen line.
            (closes #106010)

patch 8.2.5143: some tests fail when using valgrind

Problem:    Some tests fail when using valgrind. Spurious leak reports.
Solution:   Use WaitForAssert().  Avoid failing fork/exec.  Skip tests where a
            job is killed when running valgrind.

patch 8.2.5142: startup test fails if there is a status bar

Problem:    Startup test fails if there is a status bar at the top of the
            screen. (Ernie Rael)
Solution:   Use a larger vertical offset in the test.

patch 8.2.5141: using "volatile int" in a signal handler might be wrong

Problem:    Using "volatile int" in a signal handler might be wrong.
Solution:   Use "volatile sig_atomic_t".

patch 8.2.5140: seachpair timeout test is flaky

Problem:    Seachpair timeout test is flaky.
Solution:   Mark the test as flaky so it is retried.

patch 8.2.5139: TIME_WITH_SYS_TIME is no longer supported by autoconf

Problem:    TIME_WITH_SYS_TIME is no longer supported by autoconf.
Solution:   Always include time.h.

patch 8.2.5138: various small issues

Problem:    Various small issues.
Solution:   Various small improvments.

patch 8.2.5137: cannot build without the +channel feature

Problem:    Cannot build without the +channel feature. (Dominique Pellé)
Solution:   Add #ifdef around ch_log() calls. (closes #10598)

Update runtime files

patch 8.2.5136: debugger test fails when run with valgrind

Problem:    Debugger test fails when run with valgrind.
Solution:   Wait longer when using valgrind.

patch 8.2.5135: running configure gives warnings for main() return type

Problem:    Running configure gives warnings for main() return type.
Solution:   Specify "int" return type.  Avoid a few more warnings.

patch 8.2.5134: function has confusing name

Problem:    Function has confusing name.
Solution:   Rename tgetent_error() to invoke_tgetent().

patch 8.2.5133: MacOS: build fails

Problem:    MacOS: build fails.
Solution:   Remove "#if 0" from timer_delete().

patch 8.2.5132: :mkview test doesn't test much

Problem:    :mkview test doesn't test much.
Solution:   Save the view with the folds closed. (James McCoy, closes #10596)

patch 8.2.5131: timeout implementation is not optimal

Problem:    Timeout implementation is not optimal.
Solution:   Further improvements for timeouts.  Add a test for searchpair()
            timeout. (partly by Paul Ollis)

patch 8.2.5130: edit test for mode message fails when using valgrind

Problem:    Edit test for mode message fails when using valgrind.
Solution:   Use WaitForAssert().  Run beep test later.

patch 8.2.5129: timeout handling is not optimal

Problem:    Timeout handling is not optimal.
Solution:   Avoid setting timeout_flag twice.  Adjust the pointer when
            stopping the regexp timeout.  Adjust variable name.

patch 8.2.5128: syntax disabled when using synID() in searchpair() skip expr

Problem:    Syntax highlighting disabled when using synID() in searchpair()
            skip expression and it times out. (Jaehwang Jung)
Solution:   Add the redrawtime_limit_set flag. (closes #10562)

patch 8.2.5127: using assert_true() does not show value on failure

Problem:    Using assert_true() does not show value on failure.
Solution:   Use assert_inrange(). (closes #10593)

patch 8.2.5126: substitute may overrun destination buffer

Problem:    Substitute may overrun destination buffer.
Solution:   Disallow switching buffers in a substitute expression.

patch 8.2.5125: MS-Windows: warnings from MinGW compiler

Problem:    MS-Windows: warnings from MinGW compyler.
Solution:   Use "volatile". (Yasuhiro Matsumoto, closes #10589)  Initialize
            variable.

patch 8.2.5124: when syntax timeout test fails it does not show the time

Problem:    When syntax timeout test fails it does not show the time.
Solution:   Use assert_inrange().

patch 8.2.5123: using invalid index when looking for spell suggestions

Problem:    Using invalid index when looking for spell suggestions.
Solution:   Do not decrement the index when it is zero.

patch 8.2.5122: lisp indenting my run over the end of the line

Problem:    Lisp indenting my run over the end of the line.
Solution:   Check for NUL earlier.

patch 8.2.5121: interrupt test sometimes fails

Problem:    Interrupt test sometimes fails.
Solution:   Use a different file name.

patch 8.2.5120: searching for quotes may go over the end of the line

Problem:    Searching for quotes may go over the end of the line.
Solution:   Check for running into the NUL.

patch 8.2.5119: CI uses cache v2

Problem:    CI uses cache v2.
Solution:   Use cache v3. (closes #10588)

patch 8.2.5118: MS-Windows: sending a message to another Vim may hang

Problem:    MS-Windows: sending a message to another Vim may hang if that Vim
            is halted.
Solution:   Add a timeout to serverSendToVim(). (Ken Takata, closes #10585)

patch 8.2.5117: crash when calling a Lua callback from a :def function

Problem:    Crash when calling a Lua callback from a :def function. (Bohdan
            Makohin)
Solution:   Handle FC_CFUNC in call_user_func_check(). (closes #10587)

patch 8.2.5116: "limit" option of matchfuzzy() not always respected

Problem:    "limit" option of matchfuzzy() not always respected.
Solution:   Remove "else". (Kazuyuki Miyagi, closes #10586)

Update runtime files

patch 8.2.5115: search timeout is overrun with some patterns

Problem:    Search timeout is overrun with some patterns.
Solution:   Check for timeout in more places.  Make the flag volatile and
            atomic.  Use assert_inrange() to see what happened.

patch 8.2.5114: time limit on searchpair() does not work properly

Problem:    Time limit on searchpair() does not work properly.
Solution:   Set the time limit once instead of for each regexp. (closes #10562)

patch 8.2.5113: timer becomes invalid after fork/exec, :gui gives errors

Problem:    Timer becomes invalid after fork/exec, :gui gives errors. (Gabriel
            Dupras)
Solution:   Delete the timer befor forking. (closes #10584)

patch 8.2.5112: gui test hangs on MS-Windows

Problem:    Gui test hangs on MS-Windows.
Solution:   Use "!start" to start Vim.

patch 8.2.5111: no test for --gui-dialog-file

Problem:    No test for --gui-dialog-file.
Solution:   Add a test.

patch 8.2.5110: icon filetype not recognized from the first line

Problem:    Icon filetype not recognized from the first line.
Solution:   Add a check for the first line. (Doug Kearns)

patch 8.2.5109: mode not updated after CTRL-O CTRL-C in Insert mode

Problem:    Mode not updated after CTRL-O CTRL-C in Insert mode.
Solution:   Set redraw_mode and use it. (closes #10581)

patch 8.2.5108: retab test disabled because it hangs on MS-Windows

Problem:    Retab test disabled because it hangs on MS-Windows.
Solution:   Also set got_int at the other place a overlong text is detected.

patch 8.2.5107: some callers of rettv_list_alloc() check for not OK

Problem:    Some callers of rettv_list_alloc() check for not OK. (Christ van
            Willegen)
Solution:   Use "==" instead of "!=" when checking the return value.

patch 8.2.5106: default cmdwin mappings are re-mappable

Problem:    Default cmdwin mappings are re-mappable.
Solution:   Make the default mappings not re-mappable. (closes #10580)  Use
            symbols for the first do_map() argument.

patch 8.2.5105: test still hangs on MS-Windows

Problem:    Test still hangs on MS-Windows.
Solution:   Skip "nocatch" test the right way.

patch 8.2.5104: test hangs on MS-Windows

Problem:    Test hangs on MS-Windows.
Solution:   Skip another test on MS-Windows.

patch 8.2.5103: build fails with small features

Problem:    Build fails with small features.
Solution:   Add #ifdef.  Skip test on MS-Windows.

patch 8.2.5102: interrupt not caught in test

Problem:    Interrupt not caught in test.
Solution:   Consider an exception thrown in the current try/catch when got_int
            is set.  Also catch early exit when not using try/catch.

patch 8.2.5101: MS-Windows with MinGW: $CC may be "cc" instead of "gcc"

Problem:    MS-Windows with MinGW: $CC may be "cc" instead of "gcc".
Solution:   Set $CC if it is not matching "clang". (Yegappan Lakshmanan,
            closes #10578)

patch 8.2.5100: memory usage tests are not retried

Problem:    Memory usage tests are not retried.
Solution:   Mark memory usage tests as flaky.

patch 8.2.5099: some terminal tests are not retried

Problem:    Some terminal tests are not retried.
Solution:   Mark terminal tests as flaky.

patch 8.2.5098: spelldump test sometimes hangs

Problem:    Spelldump test sometimes hangs.
Solution:   Catch the problem of the spell file not being found to avoid
            hanging in the download dialog.

patch 8.2.5097: using uninitialized memory when using 'listchars'

Problem:    Using uninitialized memory when using 'listchars'.
Solution:   Use the length returned by mb_char2bytes(). (closes #10576)

patch 8.2.5096: terminal test still fails with some shell commands

Problem:    Terminal test still fails with some shell commands.
Solution:   Add missing "call".  (closes #10530)

patch 8.2.5095: terminal test still fails with some shell commands

Problem:    Terminal test still fails with some shell commands.
Solution:   Disable setting the window title in the Vim instance running in a
            terminal window. (closes #10530)

Update runtime files

patch 8.2.5094: MS-Windows GUI: empty command may cause a dialog

Problem:    MS-Windows GUI: empty command may cause a dialog.
Solution:   Delete the dialog file.  Improve the message.

patch 8.2.5093: error message for unknown command may have the command twice

Problem:    Error message for unknown command may mention the command twice.
            (Malcolm Rowe)
Solution:   Add the did_append_cmd flag. (closes #10570)

patch 8.2.5092: using "'<,'>" in Ex mode may compare unrelated pointers

Problem:    Using "'<,'>" in Ex mode may compare unrelated pointers.
Solution:   Set eap->cmd to "+" only later.

patch 8.2.5091: terminal test fails with some shell commands

Problem:    Terminal test fails with some shell commands.
Solution:   Disable setting the window title. (closes #10530)

patch 8.2.5090: MS-Windows: vim.def is no longer used

Problem:    MS-Windows: vim.def is no longer used.
Solution:   Delete vim.def. (Ken Takata, closes #10569)

patch 8.2.5089: some functions return a different value on failure

Problem:    Some functions return a different value on failure.
Solution:   Initialize the return value earlier. (Yegappan Lakshmanan,
            closes #10568)

patch 8.2.5088: value of cmod_verbose is a bit complicated to use

Problem:    Value of cmod_verbose is a bit complicated to use.
Solution:   Use zero for not set, value + 1 when set. (closes #10564)

patch 8.2.5087: cannot build with clang on MS-Windows

Problem:    Cannot build with clang on MS-Windows.
Solution:   Add support for building with clang. (Yegappan Lakshmanan,
            closes #10557)

patch 8.2.5086: CI runs on Windows 2019

Problem:    CI runs on Windows 2019.
Solution:   Switch to Windows 2022.

patch 8.2.5085: gcc gives warning for signed/unsigned difference

Problem:    Gcc gives warning for signed/unsigned difference.
Solution:   Use a different pointer type. (John Marriott)

patch 8.2.5084: when the GUI shows a dialog tests get stuck

Problem:    When the GUI shows a dialog tests get stuck.
Solution:   Add the --gui-dialog-file argument.

patch 8.2.5083: autocmd test still fails on MS-Windows

Problem:    Autocmd test still fails on MS-Windows.
Solution:   Change backward to forward slashes.

patch 8.2.5082: retab test fails

Problem:    Retab test fails.
Solution:   Disable the test for now.

patch 8.2.5081: autocmd test fails on MS-Windows

Problem:    Autocmd test fails on MS-Windows.
Solution:   Set shellslash to get forward slashes.

Update runtime files.

patch 8.2.5080: when indenting gets out of hand it is hard to stop

Problem:    When indenting gets out of hand it is hard to stop.
Solution:   When line gets too long set got_int.

patch 8.2.5079: DirChanged autocommand may use freed memory

Problem:    DirChanged autocommand may use freed memory. (Shane-XB Qian)
Solution:   Free the memory later. (closes #10555)

patch 8.2.5078: substitute test has a one second delay

Problem:    Substitute test has a one second delay.
Solution:   Use ":silent!".  Add another test case. (closes #10558)

patch 8.2.5077: various warnings from clang on MS-Windows

Problem:    Various warnings from clang on MS-Windows.
Solution:   Avoid the warnings. (Yegappan Lakshmanan, closes #10553)

patch 8.2.5076: unnecessary code

Problem:    Unnecessary code.
Solution:   Remove code and replace with function call. (closes #10552)

patch 8.2.5075: clang gives an out of bounds warning

Problem:    Clang gives an out of bounds warning.
Solution:   adjust conditional expression (John Marriott)

patch 8.2.5074: spell test fails on MS-Windows

Problem:    Spell test fails on MS-Windows.
Solution:   Do not change 'encoding'

patch 8.2.5073: clang on MS-Windows produces warnings

Problem:    Clang on MS-Windows produces warnings.
Solution:   Avoid the warnings. (Yegappan Lakshmanan, closes #10546)

Update runtime files

patch 8.2.5072: using uninitialized value and freed memory in spell command

Problem:    Using uninitialized value and freed memory in spell command.
Solution:   Initialize "attr".  Check for empty line early.

patch 8.2.5071: with some Mac OS version clockid_t is redefined

Problem:    With some Mac OS version clockid_t is redefined.
Solution:   Adjust #ifdefs. (Ozaki Kiichi, closes #10549)

patch 8.2.5070: unnecessary code

Problem:    Unnecessary code.
Solution:   Remove code that isn't needed. (closes #10534)

patch 8.2.5069: various warnings from clang on MS-Windows

Problem:    Various warnings from clang on MS-Windows.
Solution:   Fix the code to avoid the warnings. (Yegappan Lakshmanan,
            closes #10538)

patch 8.2.5068: gcc 12.1 warning when building tee

Problem:    Gcc 12.1 warning when building tee.
Solution:   Change type to size_t. (John Marriott)

patch 8.2.5067: timer_create is not available on every Mac system

Problem:    Timer_create is not available on every Mac system. (Hisashi T
            Fujinaka)
Solution:   Adjust #ifdef.

patch 8.2.5066: timer_create is not available on every Mac system

Problem:    Timer_create is not available on every Mac system. (Hisashi T
            Fujinaka)
Solution:   Adjust #ifdef.

patch 8.2.5065: wrong return type for main() in tee.c

Problem:    Wrong return type for main() in tee.c.
Solution:   Use "int" instead of "void".  Remove unused variable.