patch 8.2.4247: stack corruption when looking for spell suggestions

Problem:    Stack corruption when looking for spell suggestions.
Solution:   Prevent the depth increased too much.  Add a five second time
            limit to finding suggestions.

patch 8.2.4246: one error message not in errors.h

Problem:    One error message not in errors.h. (Antonio Colombo)
Solution:   Move the message and rename.

patch 8.2.4245: ":retab 0" may cause illegal memory access

Problem:    ":retab 0" may cause illegal memory access.
Solution:   Limit the value of 'tabstop' to 10000.

patch 8.2.4244: MS-Windows: warning from MSVC on debug build

Problem:    MS-Windows: warning from MSVC on debug build.
Solution:   Adjust "/opt"o options.  Remove unused variables.  Make variables
            uppercase for consistency. (Ken Takata, closes #9647)

patch 8.2.4243: Lua tests fail with Lua 5.4.4

Problem:    Lua tests fail with Lua 5.4.4.
Solution:   Check messages like before Lua 5.4.3. (Jakub Kulík, closes #9652)

patch 8.2.4242: put in Visual mode cannot be repeated

Problem:    Put in Visual mode cannot be repeated.
Solution:   Use "P" to put without yanking the deleted text into the unnamed
            register. (Shougo Matsushita, closes #9591)

patch 8.2.4241: some type casts are redundant

Problem:    Some type casts are redundant.
Solution:   Remove the type casts. (closes #9643)

patch 8.2.4240: error for using flatten() in Vim9 script is unclear

Problem:    Error for using flatten() in Vim9 script is unclear.
Solution:   Add a remark to use flattennew().

patch 8.2.4239: build fails with unusual configuration

Problem:    Build fails with unusual configuration.
Solution:   Adjust #ifdef. (closes #9651)

patch 8.2.4238: *.tf file could be fileytpe "tf" or "terraform"

Problem:    *.tf file could be fileytpe "tf" or "terraform".
Solution:   Detect the type from the file contents. (closes #9642)

patch 8.2.4237: record buffer wrong if character in Select mode was not typed

Problem:    Record buffer wrong if character in Select mode was not typed.
Solution:   Only delete the tail from the record buffer if the character was
            typed. (closes #9650)

patch 8.2.4236: accessing freed memory

Problem:    Accessing freed memory.
Solution:   Set the bh_curr pointer to NULL.

patch 8.2.4235: invalid check for NULL pointer

Problem:    Invalid check for NULL pointer.
Solution:   Remove the check.

patch 8.2.4234: test_garbagecollect_now() does not check v:testing

Problem:    test_garbagecollect_now() does not check v:testing as documented.
Solution:   Give an error if v:testing is not set.

patch 8.2.4233: crash when recording and using Select mode

Problem:    Crash when recording and using Select mode.
Solution:   When deleting the last recorded character check there is something
            to delete.

patch 8.2.4232: some compilers don't like a goto label without statement

Problem:    Some compilers don't like a goto label without statement.
Solution:   Return instead of using a goto.

patch 8.2.4231: Vim9: map() gives type error when type was not declared

Problem:    Vim9: map() gives type error when type was not declared.
Solution:   Only check the type when it was declared, like extend() does.
            (closes #9635)

patch 8.2.4230: MS-Windows: set_guifontwide() is included but won't work

Problem:    MS-Windows: set_guifontwide() is included but won't work.
Solution:   Include set_guifontwide() only for X11. (Ken Takata, closes #9640)

patch 8.2.4229: possible crash when invoking timer callback fails

Problem:    Possible crash when invoking timer callback fails.
Solution:   Initialize the typval.  Give an error for an empty callback.
            (closes #9636)

patch 8.2.4228: no tests for clicking in the GUI tabline

Problem:    No tests for clicking in the GUI tabline.
Solution:   Add test functions to generate the events.  Add tests using the
            functions. (Yegappan Lakshmanan, closes #9638)

patch 8.2.4227: Vim9: using "lockvar!" in :def function does not work

Problem:    Vim9: using "lockvar!" in :def function does not work.
Solution:   Add "!" instead of "-1". (closes #9634)

patch 8.2.4226: filter-map test fails

Problem:    Filter-map test fails.
Solution:   Only reject number argument in Vim9 script.

patch 8.2.4225: Vim9: depth argument of :lockvar not parsed in :def function

Problem:    Vim9: depth argument of :lockvar not parsed in :def function.
Solution:   Parse the optional depth argument. (closes #9629)
            Fix that locking doesn't work for a non-materialize list.

patch 8.2.4224: Vim9: no error when using a number for map() second argument

Problem:    Vim9: no error when using a number for map() second argument
Solution:   Disallow number to string conversion. (closes #9630)

patch 8.2.4223: long/int compiler warnings; function arguments swapped

Problem:    Long/int compiler warnings; function arguments swapped.
Solution:   Add type casts.  Swap arguments. (Ken Takata, closes #9632)

patch 8.2.4222: MS-Windows: clumsy way to suppress progress on CI

Problem:    MS-Windows: clumsy way to suppress progress on CI.
Solution:   Check for "$CI" in the Makefile itself. (Ken Takata, closes #9631)

patch 8.2.4221: some functions in normal.c are very long

Problem:    Some functions in normal.c are very long.
Solution:   Move code to separate functions. (Yegappan Lakshmanan,
            closes #9628)

patch 8.2.4220: MS-Windows: some old compiler support remains

Problem:    MS-Windows: some old compiler support remains.
Solution:   Remove obsolete compiler support. (Ken Takata, closes #9627)

patch 8.2.4219: reading before the start of the line

Problem:    Reading before the start of the line.
Solution:   Check boundary before trying to read the character.

patch 8.2.4218: illegal memory access with bracketed paste in Ex mode

Problem:    Illegal memory access with bracketed paste in Ex mode.
Solution:   Reserve space for the trailing NUL.

patch 8.2.4217: illegal memory access when undo makes Visual area invalid

Problem:    Illegal memory access when undo makes Visual area invalid.
Solution:   Correct the Visual area after undo.

patch 8.2.4216: Vim9: cannot use a function from an autoload import directly

Problem:    Vim9: cannot use a function from an autoload import directly.
Solution:   Add the AUTOLOAD instruction to figure out at runtime.
            (closes #9620)

patch 8.2.4215: illegal memory access when copying lines in Visual mode

Problem:    Illegal memory access when copying lines in Visual mode.
Solution:   Adjust the Visual position after copying lines.

patch 8.2.4214: illegal memory access with large 'tabstop' in Ex mode

Problem:    Illegal memory access with large 'tabstop' in Ex mode.
Solution:   Allocate enough memory.

patch 8.2.4213: too much code for supporting old MSVC versions

Problem:    Too much code for supporting old MSVC versions.
Solution:   Remove MSVC 2003 support. (Ken Takata, closes #9623)

patch 8.2.4212: window title test still fails in some configurations

Problem:    Window title test still fails in some configurations.
Solution:   Explicitly set the 'title' option.

patch 8.2.4211: window title test still fails in some configurations

Problem:    Window title test still fails in some configurations.
Solution:   Use WaitForAssert().

patch 8.2.4210: window title test fails in some configurations

Problem:    Window title test fails in some configurations.
Solution:   Only run the test if the title can be obtained.

patch 8.2.4209: partial in 'opfunc' cannot use an imported function

Problem:    partial in 'opfunc' cannot use an imported function.
Solution:   Also expand the function name in a partial. (closes #9614)

patch 8.2.4208: using setbufvar() may change the window title

Problem:    Using setbufvar() may change the window title.
Solution:   Do not redraw when creating the autocommand window. (closes #9613)

patch 8.2.4207: recursion test fails with MSVC

Problem:    Recursion test fails with MSVC.
Solution:   Use a smaller limit for MSVC.

patch 8.2.4206: condition with many "(" causes a crash

Problem:    Condition with many "(" causes a crash.
Solution:   Limit recursion to 1000.

patch 8.2.4205: the normal_cmd() function is too long

Problem:    The normal_cmd() function is too long.
Solution:   Move parts to separate functions. (Yegappan Lakshmanan,
            closes #9608)

patch 8.2.4204: screenpos() has non-zero row for invisible text

Problem:    screenpos() has non-zero row for invisible text.
Solution:   Only add the window row when the text is visible. (closes #9618)

patch 8.2.4203: entering a character with CTRL-V may include modifiers

Problem:    Entering a character with CTRL-V may include modifiers.
Solution:   Reset "mod_mask" when entering a character with digits after
            CTRL-V. (closes #9610)

patch 8.2.4202: Vim9: cannot export function that exists globally

Problem:    Vim9: cannot export function that exists globally.
Solution:   When checking if a function already exists only check for
            script-local functions. (closes #9615)

patch 8.2.4201: when using the GUI CTRL-Z does not stop gvim

Problem:    When using the GUI CTRL-Z does not stop gvim.
Solution:   When using the GUI set SIGTSTP to SIG_DFL. (Andrew Maltsev,
            closes #9570)

patch 8.2.4200: some tests do not clean up properly

Problem:    Some tests do not clean up properly.
Solution:   Delete created files. (Yegappan Lakshmanan, closes #9611)

patch 8.2.4199: MS-Windows: Support for MSVC 2003 is not useful

Problem:    MS-Windows: Support for MSVC 2003 is not useful.
Solution:   Remove the exceptions for MSVC 2003. (Ken Takata, closes #9616)

patch 8.2.4198: Vim9: the switch for executing instructions is too long

Problem:    Vim9: the switch for executing instructions is too long.
Solution:   Move some code to separate functions.

patch 8.2.4197: cannot use an import in the "expr" part of 'spellsuggest'

Problem:    Cannot use an import in the "expr" part of 'spellsuggest'.
Solution:   Set the script context when evaluating "expr" of 'spellsuggest'.

patch 8.2.4196: various file types not recognized

Problem:    Various file types not recognized.
Solution:   Add patterns to recognize more file types (closes #9607)

patch 8.2.4195: resizing terminal may cause to behave like CTRL-Z

Problem:    Resizing terminal may cause to behave like CTRL-Z.
Solution:   Set "got_tstp" only when in_mch_suspend is set. (Dorian Bivolaru,
            closes #9602, closes #9586)

patch 8.2.4194: MS-Windows: code for calculating font size is duplicated

Problem:    MS-Windows: code for calculating font size is duplicated.
Solution:   Move the code to a function. (Ken Takata, closes #9603)

patch 8.2.4193: cannot use an import in 'charconvert'

Problem:    Cannot use an import in 'charconvert'.
Solution:   Set the script context when evaluating 'charconvert'.  Also expand
            script-local functions in 'charconvert'.

patch 8.2.4192: cannot use an import in 'printexpr'

Problem:    Cannot use an import in 'printexpr'.
Solution:   Set the script context when evaluating 'printexpr'.

patch 8.2.4191: json5 files are not recognized

Problem:    json5 files are not recognized.
Solution:   Add a pattern for json5 files. (closes #9601)

patch 8.2.4190: all conceal tests are skipped without the screendumps feature

Problem:    All conceal tests are skipped without the screendumps feature.
Solution:   Only skip the tests that use screendumps. (closes #9599)

patch 8.2.4189: MS-Windows: code for "old look" is obsolete

Problem:    MS-Windows: code for "old look" is obsolete.
Solution:   Delete obsolete code.  Use "MS Shell Dlg" font. (Ken Takata,
            closes #9596)

Update runtime files

patch 8.2.4188: not all gitconfig files are recognized

Problem:    Not all gitconfig files are recognized.
Solution:   Add a few more patterns. (Tim Pope, closes #9597)

patch 8.2.4187: gnuplot file not recognized

Problem:    Gnuplot file not recognized.
Solution:   Recognize ".gnuplot". (closes #9588)

patch 8.2.4186: cannot use an import in 'patchexpr'

Problem:    Cannot use an import in 'patchexpr'.
Solution:   Set the script context when evaluating 'patchexpr'.  Do not
            require 'patchexpr' to return a bool, it was ignored anyway.

patch 8.2.4185: cannot use an import in 'indentexpr'

Problem:    Cannot use an import in 'indentexpr'.
Solution:   Set the script context when evaluating 'indentexpr'

patch 8.2.4184: cannot use an import in 'includeexpr'

Problem:    Cannot use an import in 'includeexpr'.
Solution:   Set the script context when evaluating 'includeexpr'

patch 8.2.4183: cannot use an import in 'formatexpr'

Problem:    Cannot use an import in 'formatexpr'.
Solution:   Set the script context when evaluating 'formatexpr'.

patch 8.2.4182: memory leak when evaluating 'diffexpr'

Problem:    Memory leak when evaluating 'diffexpr'.
Solution:   Use free_tv() instead of clear_tv().

patch 8.2.4181: Vim9: cannot use an import in 'diffexpr'

Problem:    Vim9: cannot use an import in 'diffexpr'.
Solution:   Set the script context when evaluating 'diffexpr'.  Do not require
            'diffexpr' to return a bool, it was ignored anyway.

patch 8.2.4180: 'balloonexpr' is evaluated in the current script context

Problem:    'balloonexpr' is evaluated in the current script context.
Solution:   Use the script context where the option was set.

patch 8.2.4179: 'foldtext' is evaluated in the current script context

Problem:    'foldtext' is evaluated in the current script context.
Solution:   Use the script context where the option was set.

patch 8.2.4178: Vim9: invalid error for return type of lambda when debugging

Problem:    Vim9: invalid error for return type of lambda when debugging.
Solution:   Do not check the return type of a lambda. (closes #9589)

patch 8.2.4177: Vim9: autoload script not loaded after "vim9script noclear"

Problem:    Vim9: autoload script not loaded after "vim9script noclear".
Solution:   Check IMP_FLAGS_AUTOLOAD properly.

patch 8.2.4176: Vim9: cannot use imported function with call()

Problem:    Vim9: cannot use imported function with call().
Solution:   Translate the function name. (closes #9590)

patch 8.2.4175: MS-Windows: runtime check for multi-line balloon is obsolete

Problem:    MS-Windows: runtime check for multi-line balloon is obsolete.
Solution:   Remove the obsolete code. (Ken Takata, closes #9592)

patch 8.2.4174: Vim9: can use an autoload name in normal script

Problem:    Vim9: can use an autoload name in normal script.
Solution:   Disallow using an autoload name.

patch 8.2.4173: cannot use an import in 'foldexpr'

Problem:    Cannot use an import in 'foldexpr'.
Solution:   Set the script context to where 'foldexpr' was set. (closes #9584)
            Fix that the script context was not set for all buffers.

patch 8.2.4172: filetype detection for BASIC is not optimal

Problem:    Filetype detection for BASIC is not optimal.
Solution:   Improve BASIC filetype detection. (Doug Kearns)

patch 8.2.4171: cannot invoke option function using autoload import

Problem:    Cannot invoke option function using autoload import.
Solution:   Expand the import to an autoload function name. (closes #9578)

patch 8.2.4170: MS-Windows: still using old message API calls

Problem:    MS-Windows: still using old message API calls.
Solution:   Call the "W" functions directly. (Ken Takata, closes #9582)

patch 8.2.4169: MS-Windows: unnessary casts and other minor things

Problem:    MS-Windows: unnessary casts and other minor things.
Solution:   Clean up the MS-Windows code. (Ken Takata, closes #9583)

patch 8.2.4168: disallowing empty function name breaks existing plugins

Problem:    Disallowing empty function name breaks existing plugins.
Solution:   Allow empty function name in legacy script.

patch 8.2.4167: Vim9: error message for old style import

Problem:    Vim9: error message for old style import.
Solution:   Use another error message.  Add a test.

patch 8.2.4166: undo synced when switching buffer in another window

Problem:    Undo synced when switching buffer in another window.
Solution:   Do not sync undo when not needed. (closes #9575)

patch 8.2.4165: the nv_g_cmd() function is too long

Problem:    The nv_g_cmd() function is too long.
Solution:   Move code to separate functions. (Yegappan Lakshmanan,
            closes #9576)

patch 8.2.4164: error in legacy code for function shadowing variable

Problem:    Error in legacy code for function shadowing variable.
Solution:   Only give the error in Vim9 script.

patch 8.2.4163: no error for omitting function name after autoload prefix

Problem:    No error for omitting function name after autoload prefix.
Solution:   Check for missing function name. (issue #9577)

patch 8.2.4162: Vim9: no error for redefining function with export

Problem:    Vim9: no error for redefining function with export.
Solution:   Check for existing function with/without prefix. (closes #9577)

patch 8.2.4161: Vim9: warning for missing white space after imported variable

Problem:    Vim9: warning for missing white space after imported variable.
Solution:   Do not skip white space. (closes #9567)

patch 8.2.4160: cannot change the register used for Select mode delete

Problem:    Cannot change the register used for Select mode delete.
Solution:   Make CTRL-R set the register to be used when deleting text for
            Select mode. (Shougo Matsushita, closes #9531)

patch 8.2.4159: MS-Windows: _WndProc() is very long

Problem:    MS-Windows: _WndProc() is very long.
Solution:   Move code to separate functions. (Ken Takata, closes #9573)

patch 8.2.4158: MS-Windows: memory leak in :browse

Problem:    MS-Windows: memory leak in :browse.
Solution:   Free stuff before returning. (Ken Takata, closes #9574)

patch 8.2.4157: terminal test fails because Windows sets the title

Problem:    Terminal test fails because Windows sets the title.
Solution:   Add the "vterm_title" testing override and use it in the test.
            (Ozaki Kiichi, closes #9556)

patch 8.2.4156: fileinfo message overwrites echo'ed message

Problem:    Fileinfo message overwrites echo'ed message.
Solution:   Reset need_fileinfo when displaying a message. (Rob Pilling,
            closes #9569)

patch 8.2.4155: translating strftime() argument results in check error

Problem:    Translating strftime() argument results in check error.
Solution:   Add gettext comment.

patch 8.2.4154: ml_get error when exchanging windows in Visual mode

Problem:    ml_get error when exchanging windows in Visual mode.
Solution:   Correct end of Visual area when entering another buffer.

patch 8.2.4153: MS-Windows: Global IME is no longer supported

Problem:    MS-Windows: Global IME is no longer supported.
Solution:   Remove the Global IME implementation. (Ken Takata, closes #9562)

patch 8.2.4152: block insert with double wide character fails

Problem:    Block insert with double wide character fails.
Solution:   Adjust the expected output.

patch 8.2.4151: reading beyond the end of a line

Problem:    Reading beyond the end of a line.
Solution:   For block insert only use the offset for correcting the length.

patch 8.2.4150: Coverity warns for using pointer after free

Problem:    Coverity warns for using pointer after free.
Solution:   Swap statements, even though using the pointer is no problem.

patch 8.2.4149: test override not restored, autocommand left behind

Problem:    Test override not restored, autocommand left behind.
Solution:   Correct restoring test override.  Delete autocommand afterwards.