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.

Update runtime files.

patch 8.2.5064: no test for what 8.1.0052 fixes

Problem:    No test for what 8.1.0052 fixes.
Solution:   Add a test. (closes #10531)

patch 8.2.5063: error for a command may go over the end of IObuff

Problem:    Error for a command may go over the end of IObuff.
Solution:   Truncate the message.

patch 8.2.5062: Coverity warns for dead code

Problem:    Coverity warns for dead code.
Solution:   Remove the dead code.

patch 8.2.5061: C89 requires signal handlers to return void

Problem:    C89 requires signal handlers to return void.
Solution:   Drop RETSIGTYPE and hard-code a void return value.

patch 8.2.5060: running configure fails

Problem:    Running configure fails.
Solution:   Remove line break.

patch 8.2.5059: autoconf 2.71 produces many obsolete warnings

Problem:    Autoconf 2.71 produces many obsolete warnings.
Solution:   Replace obsolete macros with non-obsolete ones, where the
            functionality does not change. (issue #10528)

patch 8.2.5058: input() does not handle composing characters properly

Problem:    input() does not handle composing characters properly.
Solution:   Use mb_cptr2char_adv() instead of mb_ptr2char_adv().
            (closes #10527)

patch 8.2.5057: using gettimeofday() for timeout is very inefficient

Problem:    Using gettimeofday() for timeout is very inefficient.
Solution:   Set a platform dependent timer. (Paul Ollis, closes #10505)

patch 8.2.5056: the channel log only contains some of the raw terminal output

Problem:    The channel log only contains some of the raw terminal output.
Solution:   Add the "o" flag to log all terminal output.  Use it for "--log".

patch 8.2.5055: statusline is not updated when terminal title changes

Problem:    Statusline is not updated when terminal title changes.
Solution:   Redraw the status line when the title changes. (issue #10425)

Update runtime files

patch 8.2.5054: no good filetype for conf files similar to dosini

Problem:    No good filetype for conf files similar to dosini.
Solution:   Add the confini filetype. (closes #10510)

patch 8.2.5053: cannot have a comment halfway an expression in a block

Problem:    Cannot have a comment halfway an expression in an autocmd command
            block.
Solution:   When skipping over the NL also skip over comments. (closes #10519)

patch 8.2.5052: CI checkout step title is a bit cryptic

Problem:    CI checkout step title is a bit cryptic.
Solution:   Add a better title. (closes #10509)

patch 8.2.5051: check for autocmd_add() event argument is confusing

Problem:    Check for autocmd_add() event argument is confusing.
Solution:   Make the code more straightforward.

patch 8.2.5050: using freed memory when searching for pattern in path

Problem:    Using freed memory when searching for pattern in path.
Solution:   Make a copy of the line.

patch 8.2.5049: insufficient tests for autocommands

Problem:    Insufficient tests for autocommands.
Solution:   Add a few more tests. (Yegappan Lakshmanan, closes #10507)

patch 8.2.5048: when using XIM the gui test may fail

Problem:    When using XIM the gui test may fail.
Solution:   Only use --not-a-term when not using XIM.

patch 8.2.5047: CurSearch highlight is often wrong

Problem:    CurSearch highlight is often wrong.
Solution:   Remember the last highlighted position and redraw when needed.

patch 8.2.5046: vim_regsub() can overwrite the destination

Problem:    vim_regsub() can overwrite the destination.
Solution:   Pass the destination length, give an error when it doesn't fit.

patch 8.2.5045: can escape a terminal popup window when the job is finished

Problem:    Can escape a terminal popup window when the job is finished.
Solution:   Only check for a finished job where it is relevant.
            (closes #10253)

patch 8.2.5044: command line test fails

Problem:    Command line test fails.
Solution:   Also beep when cmdline win can't be opened because of locks.
            Make the test not beep.  Make the test pass on MS-Windows.

patch 8.2.5043: can open a cmdline window from a substitute expression

Problem:    Can open a cmdline window from a substitute expression.
Solution:   Disallow opening a command line window when text or buffer is
            locked.

patch 8.2.5042: scrollbar thumb in tall scrolled popup not visible

Problem:    Scrollbar thumb in tall scrolled popup not visible.
Solution:   Show at least one thumb character. (fixes 10492)

patch 8.2.5041: cannot close a terminal popup with "NONE" job

Problem:    Cannot close a terminal popup with "NONE" job.
Solution:   Adjust the conditions for whether a job is running.
            (closes #10498)

patch 8.2.5040: scrollbar thumb in scrolled popup not visible

Problem:    Scrollbar thumb in scrolled popup not visible.
Solution:   Show at least one thumb character. (fixes 10492)

patch 8.2.5039: confusing error if first argument of popup_create() is wrong

Problem:    Confusing error if first argument of popup_create() is wrong.
Solution:   Give a more informative error.

patch 8.2.5038: a finished terminal in a popup window does not show scrollbar

Problem:    A finished terminal in a popup window does not show a scrollbar.
Solution:   Show the scrollbar if the terminal job is finished. (closes
            #10497)