patch 8.2.3940: match highlight disappears when doing incsearch for ":s/pat"

Problem:    Match highlight disappears when doing incsearch for ":s/pat".
Solution:   Only use line limit for incsearch highlighting. (closes #9425)

patch 8.2.3939: MS-Windows: fnamemodify('', ':p') does not work

Problem:    MS-Windows: fnamemodify('', ':p') does not work.
Solution:   Do not consider an empty string a full path. (Yegappan Lakshmanan,
            closes #9428, closes #9427)

patch 8.2.3938: line comment start is also found in a string

Problem:    Line comment start is also found in a string.
Solution:   Skip line comments in a string.

patch 8.2.3937: Insert mode completion function is too long

Problem:    Insert mode completion function is too long.
Solution:   Refactor into multiple functions. (Yegappan Lakshmanan,
            closes #9423)

patch 8.2.3936: no proper test for maintaining change mark in diff mode

Problem:    No proper test for maintaining change mark in diff mode.
Solution:   Run the test with internal and external diff. (Sean Dewar,
            closes #9424)

patch 8.2.3935: CTRL-U in Insert mode does not fix the indent

Problem:    CTRL-U in Insert mode does not fix the indent.
Solution:   Fix the indent when 'cindent' is set.

patch 8.2.3934: repeating line comment is undesired for "O" command

Problem:    Repeating line comment is undesired for "O" command.
Solution:   Do not copy line comment leader for "O". (closes #9426)

patch 8.2.3933: after ":cd" fails ":cd -" is incorrect

Problem:    After ":cd" fails ":cd -" is incorrect.
Solution:   Set the previous directory only after successfully changing
            directory. (Richard Doty, closes #9419, closes #8983)

patch 8.2.3932: C line comment not formatted properly

Problem:    C line comment not formatted properly.
Solution:   If a line comment follows after "#if" the next line is not the end
            of a paragraph.

patch 8.2.3931: Coverity reports a memory leak

Problem:    Coverity reports a memory leak.
Solution:   Free memory in case of failure.

patch 8.2.3930: getcmdline() argument has a misleading type

Problem:    getcmdline() argument has a misleading type.
Solution:   Use the correct type, even though the value is not used.

patch 8.2.3929: using unititialized variable

Problem:    Using unititialized variable.
Solution:   Set the option flags to zero for a terminal option.

patch 8.2.3928: heredoc test fails

Problem:    Heredoc test fails.
Solution:   Correct order of function arguments.

patch 8.2.3927: Vim9: double free when using lambda

Problem:    Vim9: double free when using lambda.
Solution:   Don't free both cmdline and line_to_free.

patch 8.2.3926: build failure without the 'autochdir' option

Problem:    Build failure without the 'autochdir' option. (John Marriott)
Solution:   Add #ifdefs.

patch 8.2.3925: diff mode confused by NUL bytes

Problem:    Diff mode confused by NUL bytes.
Solution:   Handle NUL bytes differently. (Christian Brabandt, closes #9421,
            closes #9418)

patch 8.2.3924: Vim9: no error if something follows :enddef

Problem:    Vim9: no error if something follows :enddef in a nested function.
Solution:   Give an error.  Move common code to a function.

patch 8.2.3923: Vim9: double free with split argument list in nested function

Problem:    Vim9: double free if a nested function has a line break in the
            argument list.
Solution:   Set cmdlinep when freeing the previous line.

patch 8.2.3922: cannot build with dynamic Ruby 3.1

Problem:    Cannot build with dynamic Ruby 3.1.
Solution:   Add "_EXTRA" variables for CI.  Add missing functions. (Ozaki
            Kiichi, closes #9420)

patch 8.2.3921: the way xdiff is used is inefficient

Problem:    The way xdiff is used is inefficient.
Solution:   Use hunk_func instead of the out_line callback. (Lewis Russell,
            closes #9344)

patch 8.2.3920: restoring directory after using another window is inefficient

Problem:    Restoring directory after using another window is inefficient.
Solution:   Only restore the directory for win_execute().  Apply 'autochdir'
            only when needed.

patch 8.2.3919: Vim9: wrong argument for append() results in two errors

Problem:    Vim9: wrong argument for append() results in two errors.
Solution:   Check did_emsg.  Also for setline().  Adjust the help for
            appendbufline().

patch 8.2.3918: function list test fails

Problem:    Function list test fails.
Solution:   Adjust the test for the new location of the function list.

Update runtime files

patch 8.2.3917: the eval.txt help file is way too big

Problem:    The eval.txt help file is way too big.
Solution:   Move the builtin function details to a separate file.

patch 8.2.3916: no error for passing an invalid line number to append()

Problem:    No error for passing an invalid line number to append().
Solution:   In Vim9 script check for a non-negative number. (closes #9417)

patch 8.2.3915: illegal memory access when completing with invalid bytes

Problem:    illegal memory access when completing with invalid bytes.
Solution:   Avoid going over the end of the completion text.

patch 8.2.3914: various spelling mistakes in comments

Problem:    Various spelling mistakes in comments.
Solution:   Fix the mistakes. (Dominique Pellé, closes #9416)

patch 8.2.3913: help for expressions does not mention Vim9 syntax

Problem:    Help for expressions does not mention Vim9 syntax.
Solution:   Add the rules for Vim9 to the expression help.  Rename functions
            to match the help.

patch 8.2.3912: the ins_complete() function is much too long

Problem:    The ins_complete() function is much too long.
Solution:   Split it up into multiple functions. (Yegappan Lakshmanan,
            closes #9414)

patch 8.2.3911: Vim9: type check for filter() does not accept unknown

Problem:    Vim9: type check for filter() does not accept unknown.
Solution:   Also accept unknown for the return type. (closes #9413)

patch 8.2.3910: when compare function of sort() fails it does not abort

Problem:    When the compare function of sort() produces and error then sort()
            does not abort.
Solution:   Check if did_emsg was incremented.

patch 8.2.3909: Containerfile using prefix name not recognized

Problem:    Containerfile using prefix name not recognized.
Solution:   Recognize Containerfile.*.

patch 8.2.3908: cannot use a script-local function for 'foldtext'

Problem:    Cannot use a script-local function for 'foldtext'.
Solution:   Expand "s:" and "<SID>". (Yegappan Lakshmanan, closes #9411)

patch 8.2.3907: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move error messages to errors.h.  Avoid duplicates.

patch 8.2.3906: Vim9 help still contains "under development" warnings

Problem:    Vim9 help still contains "under development" warnings.
Solution:   Remove the explicit warning.

patch 8.2.3905: Dockerfile using prefix name not recognized

Problem:    Dockerfile using prefix name not recognized.
Solution:   Recognize Dockerfile.*. (closes #9410)

patch 8.2.3904: Vim9: skip expression type is not checked at compile time

Problem:    Vim9: skip expression type is not checked at compile time.
Solution:   Add argument type checks.

patch 8.2.3903: "gM" does not count tabs as expected

Problem:    "gM" does not count tabs as expected.
Solution:   Use linetabsize() instead of mb_string2cells(). (closes #9409)

patch 8.2.3902: Vim9: double free with nested :def function

Problem:    Vim9: double free with nested :def function.
Solution:   Pass "line_to_free" from compile_def_function() and make sure
            cmdlinep is valid.

patch 8.2.3901: Vim9: Cannot set 'cpo' in main .vimrc if using Vim9 script

Problem:    Vim9: Cannot set 'cpo' in main .vimrc if using Vim9 script.
Solution:   Do not restore 'cpo' at the end of the main .vimrc.

patch 8.2.3900: it is not easy to use a script-local function for an option

Problem:    It is not easy to use a script-local function for an option.
Solution:   recognize s: and <SID> at the start of the expression. (Yegappan
            Lakshmanan, closes #9401)

patch 8.2.3899: Vim9: test for map() on string fails

Problem:    Vim9: test for map() on string fails.
Solution:   Expect string return type.

patch 8.2.3898: Vim9: not sufficient testing for variable initialization

Problem:    Vim9: not sufficient testing for variable initialization.
Solution:   Add another test case.

patch 8.2.3897: Vim9: second argument of map() and filter() not checked

Problem:    Vim9: the second argument of map() and filter() is not checked at
            compile time.
Solution:   Add more specific type check for the second argument.

patch 8.2.3896: Vim9: no test for nested function not available later

Problem:    Vim9: no test for nested function not available later.
Solution:   Add a test.

patch 8.2.3895: Vim9: confusing error when using function() with a number

Problem:    Vim9: confusing error when using function() with a number.
Solution:   Check for a function or string argument.

patch 8.2.3894: Vim9: no proper type check for first argument of call()

Problem:    Vim9: no proper type check for first argument of call().
Solution:   Add specific type check.

patch 8.2.3893: Vim9: many local variables are initialized with an instruction

Problem:    Vim9: many local variables are initialized with an instruction.
Solution:   Initialize local variables to zero to avoid the instructions.

patch 8.2.3892: when modifyOtherKeys is used CTRL-C is not recognized

Problem:    When modifyOtherKeys is used CTRL-C is not recognized.
Solution:   Check for uppercase C as well, fix minimum length.

patch 8.2.3891: github CI: workflows may overlap

Problem:    Github CI: workflows may overlap.
Solution:   Cancel previous workflows when starting a new one. (Yegappan
            Lakshmanan, closes #9400)

patch 8.2.3890: Vim9: type check for using v: variables is basic

Problem:    Vim9: type check for using v: variables is basic.
Solution:   Specify a more precise type.

patch 8.2.3889: duplicate code for translating script-local function name

Problem:    Duplicate code for translating script-local function name.
Solution:   Move the code to get_scriptlocal_funcname(). (Yegappan Lakshmanan,
            closes #9393)

patch 8.2.3888: the argument list may contain duplicates

Problem:    The argument list may contain duplicates.
Solution:   Add the :argdedeupe command. (Nir Lichtman, closes #6235)

patch 8.2.3887: E1135 is used for two different errors

Problem:    E1135 is used for two different errors.
Solution:   Renumber one error.

patch 8.2.3886: can define autocmd for every event by using "au!"

Problem:    Can define autocmd for every event by using "au!".
Solution:   Check if a command is present also for "au!".

patch 8.2.3885: arglist test fails

Problem:    Arglist test fails.
Solution:   Adjust for locking the arglist for ":all".

patch 8.2.3884: crash when clearing the argument list while using it

Problem:    Crash when clearing the argument list while using it.
Solution:   Lock the argument list for ":all".

patch 8.2.3883: crash when switching to other regexp engine fails

Problem:    Crash when switching to other regexp engine fails.
Solution:   Check for regprog being NULL.

Update runtime files

patch 8.2.3882: more duplicated code in f_getreginfo()

Problem:    More duplicated code in f_getreginfo().
Solution:   Also use getreg_get_regname(). (closes #9398)

patch 8.2.3881: QNX: crash when compiled with GUI but using terminal

Problem:    QNX: crash when compiled with GUI but using terminal.
Solution:   Check the gui.in_use flag. (Hirohito Higashi, closes #9391)

patch 8.2.3880: Solution filter files are not recognized

Problem:    Solution filter files are not recognized.
Solution:   Add pattern *.slnf and use json. (Doug Kearns)

patch 8.2.3879: getreg() and getregtype() contain dead code

Problem:    getreg() and getregtype() contain dead code.
Solution:   Remove the needless check. (closes #9392)  Also refactor to put
            common code in a shared function.

patch 8.2.3878: Vim9: debugger tries to read more lines than there are

Problem:    Vim9: debugger tries to read more lines than there are.
Solution:   Check the number of lines. (closes #9394)

patch 8.2.3877: function does not abort after a type error in compare

Problem:    Function does not abort after a type error in compare
Solution:   Check getting number fails. (closes #9384)

patch 8.2.3876: 'cindent' does not recognize inline namespace

Problem:    'cindent' does not recognize inline namespace.
Solution:   Skip over "inline" to find "namespace". (closes #9383)

patch 8.2.3875: gcc complains about buffer overrun

Problem:    gcc complains about buffer overrun.
Solution:   Use mch_memmove() instead of STRCPY(). (John Marriott)

patch 8.2.3874: cannot highlight the number column for a sign

Problem:    Cannot highlight the number column for a sign.
Solution:   Add the "numhl" argument. (James McCoy, closes #9381)

patch 8.2.3873: go.mod files are not recognized

Problem:    go.mod files are not recognized.
Solution:   Check for the file name. (closes #9380)

patch 8.2.3872: Vim9: finddir() and uniq() return types can be more specific

Problem:    Vim9: finddir() and uniq() return types can be more specific.
Solution:   Adjust the return type.

patch 8.2.3871: list.c contains code for dict and blob

Problem:    List.c contains code for dict and blob.
Solution:   Refactor to put code where it belongs. (Yegappan Lakshmanan,
            closes #9386)

patch 8.2.3870: MS-Windows: wrong dir when using right-click context menu

Problem:    MS-Windows: wrong working directory when opening two files with
            right-click context menu. (Gabriel Dupras)
Solution:   Use the working directory and pass it on to the process creation.
            (Nir Lichtman, closes #9382, closes #8874)

patch 8.2.3869: Vim9: type checking for "any" is inconsistent

Problem:    Vim9: type checking for "any" is inconsistent.
Solution:   Always use a runtime type check for using "any" for a more
            specific type.

patch 8.2.3868: Vim9: function test fails

Problem:    Vim9: function test fails.
Solution:   Add missing changes.  Add test for earlier patch.

patch 8.2.3867: implementation of some list functions too complicated

Problem:    Implementation of some list functions too complicated.
Solution:   Refactor do_sort_uniq(), f_count() and extend() (Yegappan
            Lakshmanan, closes #9378)

patch 8.2.3866: Vim9: type checking global variables is inconsistent

Problem:    Vim9: type checking global variables is inconsistent.
Solution:   Use the "unknown" type in more places.

patch 8.2.3865: Vim9: compiler complains about using "try" as a struct member

Problem:    Vim9: compiler complains about using "try" as a struct member.
Solution:   Rename "try" to "tryref".

patch 8.2.3864: cannot disable requesting key codes from xterm

Problem:    Cannot disable requesting key codes from xterm.
Solution:   Add the 'xtermcodes' option, default on.

patch 8.2.3863: various build flags accidentally enabled

Problem:    Various build flags accidentally enabled.
Solution:   Revert several lines in Makefile.

patch 8.2.3862: crash on exit with EXITFREE and using win_execute()

Problem:    Crash on exit with EXITFREE and using win_execute().
Solution:   Also save and restore tp_topframe. (issue #9374)

patch 8.2.3861: list of distributed files is outdated

Problem:    List of distributed files is outdated.
Solution:   Add new files.

patch 8.2.3860: Vim9: codecov struggles with the file size

Problem:    Vim9: codecov struggles with the file size.
Solution:   Split vim9compile.c into four files.

patch 8.2.3859: Vim9: some code lines not tested

Problem:    Vim9: some code lines not tested.
Solution:   Add a few specific tests.

patch 8.2.3858: Vim9: not enough tests

Problem:    Vim9: not enough tests.
Solution:   Add tests for :try/:catch and :redir. Add missing type check.

patch 8.2.3857: Vim9: inconsistent error for using function()

Problem:    Vim9: inconsistent error for using function().
Solution:   Use a runtime type check for the result of function().
            (closes #8492)

patch 8.2.3856: Vim9: not enough tests

Problem:    Vim9: not enough tests.
Solution:   Run more expression tests also with Vim9. Fix an uncovered
            problem.

patch 8.2.3855: illegal memory access when displaying a blob

Problem:    Illegal memory access when displaying a blob.
Solution:   Append a NUL at the end. (Yegappan Lakshmanan, closes #9372)

patch 8.2.3854: Vim9: inconsistent arguments for test functions

Problem:    Vim9: inconsistent arguments for test functions.
Solution:   When :def function and script have different arguments use a list
            with two items instead of a separate function.

patch 8.2.3853: Vim9: not enough tests

Problem:    Vim9: not enough tests.
Solution:   Run more existing tests for Vim9 script.

patch 8.2.3852: Vim9: not enough tests

Problem:    Vim9: not enough tests.
Solution:   Also run existing tests for Vim9 script.  Make errors more
            consistent.

patch 8.2.3851: Vim9: overhead when comparing string, dict or function

Problem:    Vim9: overhead when comparing string, dict or function.
Solution:   Call the intented compare function directly.  Refactor to avoid
            duplicated code.

patch 8.2.3850: illegal memory access when displaying a partial

Problem:    Illegal memory access when displaying a partial.
Solution:   Terminate the string with a NUL. (closes #9371)

patch 8.2.3849: functions implementing reduce and map are too long

Problem:    Functions implementing reduce and map are too long.
Solution:   Use a function for each type of value.  Add a few more test cases
            and add to the help. (Yegappan Lakshmanan, closes #9370)

patch 8.2.3848: cannot use reduce() for a string

Problem:    Cannot use reduce() for a string.
Solution:   Make reduce() work with a string. (Naruhiko Nishino, closes #9366)

patch 8.2.3847: illegal memory access when using a lambda with an error

Problem:    Illegal memory access when using a lambda with an error.
Solution:   Avoid skipping over the NUL after a string.

patch 8.2.3846: no error when using control character for 'lcs' or 'fcs'

Problem:    No error when using control character for 'lcs' or 'fcs'.
Solution:   Use char2cells() to check the width. (closes #9369)

patch 8.2.3845: Vim9: test fails when the channel feature is missing

Problem:    Vim9: test fails when the channel feature is missing.
Solution:   Check for the channel feature. (Dominique Pellé, closes #9368)

patch 8.2.3844: Vim9: no type error if assigning func(number) to func(string)

Problem:    Vim9: no type error if assigning a value with type func(number) to
            a variable of type func(string).
Solution:   Use check_type_maybe(): return MAYBE if a runtime type check is
            useful.  (issue #8492)

patch 8.2.3843: dep3patch files are not recognized

Problem:    Dep3patch files are not recognized.
Solution:   Recognize dep3patch files by their location and content. (James
            McCoy, closes #9367)