patch 8.2.3997: Vim9: not enough testing for extend() and map()

Problem:    Vim9: not enough testing for extend() and map().
Solution:   Add more test cases.  Fix uncovered problems.  Remove unused type
            fields.

patch 8.2.3996: Vim9: type checking lacks information about declared type

Problem:    Vim9: type checking for list and dict lacks information about
            declared type.
Solution:   Add dv_decl_type and lv_decl_type.  Refactor the type stack to
            store two types in each entry.

patch 8.2.3995: not all sshconfig files are detected as such

Problem:    Not all sshconfig files are detected as such.
Solution:   Adjust the patterns used for sshconfig detection. (David Auer,
            closes #9322)

patch 8.2.3994: Vim9: extend() complains about type when it was not declared

Problem:    Vim9: extend() complains about the type even when it was not
            declared.
Solution:   Only check the list or dict type when it was declared.

patch 8.2.3993: when recording a change in Select mode char appears twice

Problem:    When recording a change in Select mode the first typed character
            appears twice.
Solution:   When putting the character back into typeahead remove it from
            recorded characters. (closes #9462)

patch 8.2.3992: wrong local-additions in the help with language mix

Problem:    Wrong local-additions in the help with language mix.
Solution:   Adjust how the local additions list is generated. (Hirohito
            Higashi, closes #9464)

patch 8.2.3991: Vim9: error when extending dict<any>

Problem:    Vim9: error when extending dict<any> with another type that it was
            initialized with.
Solution:   Also set the type for dict<any> if the initializer has a more
            specific type. (closes #9461)

patch 8.2.3990: testing wrong operator

Problem:    Testing wrong operator.
Solution:   Test "g@" instead of "r_". (Naohiro Ono, closes #9463)

patch 8.2.3989: some insert completion code is not tested

Problem:    Some insert completion code is not tested.
Solution:   Add a few tests.  Refactor thesaurus completion. (Yegappan
            Lakshmanan, closes #9460)

patch 8.2.3988: tiny build fails

Problem:    Tiny build fails.
Solution:   Fix misplaced #ifdef.

patch 8.2.3987: error messages are spread out

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

patch 8.2.3986: error messages are spread out

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

patch 8.2.3985: error messages are spread out

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

patch 8.2.3984: debugger test fails

Problem:    Debugger test fails.
Solution:   Adjust the test for modified debugging of a for loop.

patch 8.2.3983: error messages are spread out

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

patch 8.2.3982: some lines of code not covered by tests

Problem:    Some lines of code not covered by tests.
Solution:   Add a few more test cases. (Dominique Pellé, closes #9453)

patch 8.2.3981: Vim9: debugging a for loop doesn't stop before it starts

Problem:    Vim9: debugging a for loop doesn't stop before it starts.
Solution:   Keep the DEBUG instruction before the expression is evaluated.
            (closes #9456)

patch 8.2.3980: if 'operatorfunc' invokes an operator Visual mode is changed

Problem:    If 'operatorfunc' invokes an operator the remembered Visual mode
            may be changed. (Naohiro Ono)
Solution:   Save and restore the information for redoing the Visual area.
            (closes #9455)

patch 8.2.3979: Vim9: the feature is not mentioned in the right places

Problem:    Vim9: the feature is not mentioned in the right places.
Solution:   Add +vim9script to the help and :version output.

patch 8.2.3978: build error when using dynamycally loaded Python 3

Problem:    Build error when using dynamycally loaded Python 3.
Solution:   Adjust #ifdef.

patch 8.2.3977: error messages are spread out

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

patch 8.2.3976: FEARG_LAST is never used

Problem:    FEARG_LAST is never used. (Dominique Pellé)
Solution:   Remove FEARG_LAST and the related code.

patch 8.2.3975: error messages are spread out

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

patch 8.2.3974: Vim9: LISTAPPEND instruction does not check for a locked list

Problem:    Vim9: LISTAPPEND instruction does not check for a locked list.
Solution:   Check whether the list is locked. (closes #9452)

patch 8.2.3973: tiny build fails

Problem:    Tiny build fails.
Solution:   Adjust #ifdefs

patch 8.2.3972: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move the last errors from globals.h to errors.h.

patch 8.2.3971: build fails

Problem:    Build fails.
Solution:   Use the right error message name.

patch 8.2.3970: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3969: value of MAXCOL not available in Vim script

Problem:    Value of MAXCOL not available in Vim script.
Solution:   Add v:maxcol. (Naohiro Ono, closes #9451)

patch 8.2.3968: build failure

Problem:    Build failure.
Solution:   Add missing changes.

patch 8.2.3967: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3966: when using feedkeys() abbreviations may be blocked

Problem:    When using feedkeys() abbreviations may be blocked.
Solution:   Reset tb_no_abbr_cnt when running out of characters.
            (closes #9448)

patch 8.2.3965: Vim9: no easy way to check if Vim9 script is supported

Problem:    Vim9: no easy way to check if Vim9 script is supported.
Solution:   Add has('vim9script').

patch 8.2.3964: some common lisp and scheme files not recognized

Problem:    Some common lisp and scheme files not recognized.
Solution:   Recognize *.asd as lisp and *.sld as scheme. (Alex Vear,
            closes #9447)

patch 8.2.3963: build failure with tiny and small features

Problem:    Build failure with tiny and small features. (Tony Mechelynck)
Solution:   Adjust #ifdefs.

patch 8.2.3962: build fails for missing error message

Problem:    Build fails for missing error message.
Solution:   Add changes in missed file.

patch 8.2.3961: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3960: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3959: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3958: build failure compiling xxd with "-std=c2x"

Problem:    Build failure compiling xxd with "-std=c2x".
Solution:   define _XOPEN_SOURCE. (Yegappan Lakshmanan, closes #9444)

patch 8.2.3957: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3956: duplicate assignment

Problem:    Duplicate assignment.
Solution:   Remove the second assignment. (closes #9442)

patch 8.2.3955: error messages are spread out

Problem:    Error messages are spread out.
Solution:   Move more errors to errors.h.

patch 8.2.3954: Vim9: no error for shadowing if script var is declared later

Problem:    Vim9: no error for shadowing if script var is declared later.
Solution:   Check argument names when compiling a function.

patch 8.2.3953: insert completion code is too complicated

Problem:    Insert completion code is too complicated.
Solution:   More refactoring.  Move function arguments into a struct.
            (Yegappan Lakshmanan, closes #9437)

patch 8.2.3952: first line not redrawn when adding lines to an empty buffer

Problem:    First line not redrawn when adding lines to an empty buffer.
Solution:   Adjust the argument to appended_lines(). (closes #9439,
            closes #9438)

Update runtime files

patch 8.2.3951: Vim9: memory leak when text after a nested function

Problem:    Vim9: memory leak when text after a nested function.
Solution:   Free the function if text is found after "enddef".

patch 8.2.3950: going beyond the end of the line with /\%V

Problem:    Going beyond the end of the line with /\%V.
Solution:   Check for valid column in getvcol().

patch 8.2.3949: using freed memory with /\%V

Problem:    Using freed memory with /\%V.
Solution:   Get the line again after getvvcol().

patch 8.2.3948: Vim9: failure with partial with unknown argument count

Problem:    Vim9: failure with partial with unknown argument count.
Solution:   Do not copy argument types if there aren't any.

patch 8.2.3947: unnecessary check for NULL pointer

Problem:    Unnecessary check for NULL pointer.
Solution:   Remove the check. (closes #9434)

patch 8.2.3946: when an internal error makes Vim exit the error is not seen

Problem:    When an internal error makes Vim exit the error is not seen.
Solution:   Add the error to the test output.

patch 8.2.3945: Vim9: partial variable argument types are wrong

Problem:    Vim9: partial variable argument types are wrong, leading to a
            crash.
Solution:   When adjusting the argument count also adjust the argument types.
            (closes #9433)

patch 8.2.3944: insert mode completion functions are too long

Problem:    Insert mode completion functions are too long.
Solution:   Split up into multiple functions. (Yegappan Lakshmanan,
            closes #9431)

patch 8.2.3943: compiler warning from gcc for uninitialized variable

Problem:    Compiler warning from gcc for uninitialized variable.
Solution:   Initialize variable. (closes #9429)

patch 8.2.3942: Coverity reports a possible memory leak

Problem:    Coverity reports a possible memory leak.
Solution:   Free the array if allocation fails.

patch 8.2.3941: SIGTSTP is not handled

Problem:    SIGTSTP is not handled.
Solution:   Handle SIGTSTP like pressing CTRL-Z. (closes #9422)

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)