patch 8.2.4992: compiler warning for possibly uninitialized variable

Problem:    Compiler warning for possibly uninitialized variable. (Tony
            Mechelynck)
Solution:   Initialize variable in the caller instead of in the function.

patch 8.2.4991: no test for hwat patch 8.1.0535 fixes

Problem:    No test for hwat patch 8.1.0535 fixes.
Solution:   Add a test. (closes #10462)

patch 8.2.4990: memory leak when :defcompile fails

Problem:    Memory leak when :defcompile fails.
Solution:   Free fname when returning early.

patch 8.2.4989: cannot specify a function name for :defcompile

Problem:    Cannot specify a function name for :defcompile.
Solution:   Implement a function name argument for :defcompile.

Update runtime files.

patch 8.2.4988: textprop in wrong position when replacing multi-byte chars

Problem:    Textprop in wrong position when replacing multi-byte chars.
Solution:   Adjust textprop position. (closes #10461)

patch 8.2.4987: after deletion a small fold may be closable

Problem:    After deletion a small fold may be closable.
Solution:   Check for a reverse range. (Brandon Simmons, closes #10457)

patch 8.2.4986: some github actions are outdated

Problem:    Some github actions are outdated.
Solution:   Update CodeQl to v2, update checkout to v3. (closes #10450)

patch 8.2.4985: PVS warns for possible array underrun

Problem:    PVS warns for possible array underrun.
Solution:   Add a check for a positive value. (closes #10451)

patch 8.2.4984: dragging statusline fails for window with winbar

Problem:    Dragging statusline fails for window with winbar.
Solution:   Fix off-by-one error. (closes #10448)

patch 8.2.4983: colors test fails in the GUI

Problem:    Colors test fails in the GUI.
Solution:   Reset g:terminal_ansi_colors.

patch 8.2.4982: colors in terminal window are not 100% correct

Problem:    Colors in terminal window are not 100% correct.
Solution:   Use g:terminal_ansi_colors as documented. (closes #10429,
            closes #7227 closes #10347)

patch 8.2.4981: it is not possible to manipulate autocommands

Problem:    It is not possible to manipulate autocommands.
Solution:   Add functions to add, get and set autocommands. (Yegappan
            Lakshmanan, closes #10291)

patch 8.2.4980: when 'shortmess' contains 'A' loading session may still warn

Problem:    When 'shortmess' contains 'A' loading a session may still warn for
            an existing swap file. (Melker Österberg)
Solution:   Keep the 'A' flag to 'shortmess' in the session file.
            (closes #10443)

patch 8.2.4979: accessing freed memory when line is flushed

Problem:    Accessing freed memory when line is flushed.
Solution:   Make a copy of the pattern to search for.

patch 8.2.4978: no error if engine selection atom is not at the start

Problem:    No error if engine selection atom is not at the start.
Solution:   Give an error. (Christian Brabandt, closes #10439)

patch 8.2.4977: memory access error when substitute expression changes window

Problem:    Memory access error when substitute expression changes window.
Solution:   Disallow changing window in substitute expression.

patch 8.2.4976: Coverity complains about not restoring a saved value

Problem:    Coverity complains about not restoring a saved value.
Solution:   Restore value before handling error.

patch 8.2.4975: recursive command line loop may cause a crash

Problem:    Recursive command line loop may cause a crash.
Solution:   Limit recursion of getcmdline().

patch 8.2.4974: ":so" command may read after end of buffer

Problem:    ":so" command may read after end of buffer.
Solution:   Compute length of text properly.

patch 8.2.4973: Vim9: type error for list unpack mentions argument

Problem:    Vim9: type error for list unpack mentions argument.
Solution:   Mention variable. (close #10435)

patch 8.2.4972: Vim9: compilation fails when using dict member when skipping

Problem:    Vim9: compilation fails when using dict member when skipping.
Solution:   Do not generate ISN_USEDICT when skipping. (closes #10433)

patch 8.2.4971: Vim9: interpolated string seen as range

Problem:    Vim9: interpolated string seen as range.
Solution:   Recognize an interpolated string at the start of a command line.
            (closes #10434)

patch 8.2.4970: "eval 123" gives an error, "eval 'abc'" does not

Problem:    "eval 123" gives an error, "eval 'abc'" does not.
Solution:   Also give an error when evaluating only a string. (closes #10434)

patch 8.2.4969: changing text in Visual mode may cause invalid memory access

Problem:    Changing text in Visual mode may cause invalid memory access.
Solution:   Check the Visual position after making a change.

patch 8.2.4968: reading past end of the line when C-indenting

Problem:    Reading past end of the line when C-indenting.
Solution:   Check for NUL.

patch 8.2.4967: MS-Windows GUI: mouse event test sometimes fails

Problem:    MS-Windows GUI: mouse event test sometimes fails.
Solution:   Ignore one move event only if there is an extra event.

patch 8.2.4966: MS-Windows GUI: mouse event test gets extra event

Problem:    MS-Windows GUI: mouse event test gets extra event.
Solution:   Ignore one move event.

patch 8.2.4965: GUI: testing mouse move event depends on screen cell size

Problem:    GUI: testing mouse move event depends on screen cell size.
Solution:   Multiply the row and column with the screen cell size.

patch 8.2.4964: MS-Windows GUI: mouse event test is flaky

Problem:    MS-Windows GUI: mouse event test is flaky.
Solution:   Add a short delay after generating a mouse event.

patch 8.2.4963: expanding path with "/**" may overrun end of buffer

Problem:    Expanding path with "/**" may overrun end of buffer.
Solution:   Use vim_snprintf().

patch 8.2.4962: files show up in git status

Problem:    Files show up in git status.
Solution:   Adjust the list of ignored files.  Clean up more test files.
            (Shane xb Qian, closes #9929)

patch 8.2.4961: build error with a certain combination of features

Problem:    Build error with a certain combination of features.
Solution:   Adjust #if. (John Marriott)

patch 8.2.4960: text properties that cross lines not updated for deleted line

Problem:    Text properties that cross line boundary are not correctly updated
            for a deleted line.
Solution:   Correct computing location of text property entry. (Paul Ollis,
            closes #10431, closes #10430)

patch 8.2.4959: using NULL regexp program

Problem:    Using NULL regexp program.
Solution:   Check for regexp program becoming NULL in more places.

patch 8.2.4958: a couple conditions are always true

Problem:    A couple conditions are always true.
Solution:   Remove the conditions. (Goc Dundar, closes #10428)

patch 8.2.4957: text properties in a wrong position after a block change

Problem:    Text properties in a wrong position after a block change.
Solution:   Adjust the properties columns. (closes #10427)

patch 8.2.4956: reading past end of line with "gf" in Visual block mode

Problem:    Reading past end of line with "gf" in Visual block mode.
Solution:   Do not include the NUL in the length.

patch 8.2.4955: text property in wrong position after auto-indent

Problem:    Text property in wrong position after auto-indent.
Solution:   Adjust text property columns. (closes #10422, closes #7719)

patch 8.2.4954: inserting line breaks text property spanning two lines

Problem:    Inserting line breaks text property spanning more then one line.
Solution:   Check TP_FLAG_CONT_PREV and TP_FLAG_CONT_NEXT. (closes #10423)

Update runtime files, new color schemes

patch 8.2.4953: with 'si' inserting '}' after completion goes wrong

Problem:    With 'smartindent' inserting '}' after completion goes wrong.
Solution:   Check the cursor is in indent.  (closes #10420)

patch 8.2.4952: GUI test will fail if color scheme changes

Problem:    GUI test will fail if color scheme changes.
Solution:   Reduce the test for now.

patch 8.2.4951: smart indenting done when not enabled

Problem:    Smart indenting done when not enabled.
Solution:   Check option values before setting can_si. (closes #10420)

patch 8.2.4950: text properties position wrong after shifting text

Problem:    Text properties position wrong after shifting text.
Solution:   Adjust the text properties when shifting a block of text.
            (closes #10418)

patch 8.2.4949: Vim9: some code not covered by tests

Problem:    Vim9: some code not covered by tests.
Solution:   Add a few more test cases.  Fix double error message.

patch 8.2.4948: cannot use Perl heredoc in nested :def function

Problem:    Cannot use Perl heredoc in nested :def function. (Virginia
            Senioria)
Solution:   Only concatenate heredoc lines when not in a nested function.
            (closes #10415)

patch 8.2.4947: text properties not adjusted when accepting spell suggestion

Problem:    Text properties not adjusted when accepting spell suggestion.
Solution:   Adjust text properties when text changes. (closes #10414)

patch 8.2.4946: Vim9: some code not covered by tests

Problem:    Vim9: some code not covered by tests.
Solution:   Add a few more test cases.  Remove dead code.

patch 8.2.4945: inconsistent use of white space

Problem:    Inconsistent use of white space.
Solution:   Use Tabs and Spaces consistently.

Update runtime files and translations

patch 8.2.4944: text properties are wrong after "cc"

Problem:    Text properties are wrong after "cc". (Axel Forsman)
Solution:   Pass the deleted byte count to inserted_bytes(). (closes #10412,
            closes #7737, closes #5763)

patch 8.2.4943: changing 'switchbuf' may have no effect

Problem:    Changing 'switchbuf' may have no effect.
Solution:   Handle 'switchbuf' in didset_string_options(). (Sean Dewar,
            closes #10406)

patch 8.2.4942: error when setting 'filetype' in help file again

Problem:    Error when setting 'filetype' in help file again.
Solution:   Deal with text property type already existing. (closes #10409)

patch 8.2.4941: '[ and '] marks may be wrong after undo

Problem:    '[ and '] marks may be wrong after undo.
Solution:   Adjust the '[ and '] marks if needed. (closes #10407, closes #1281)

patch 8.2.4940: some code is never used

Problem:    Some code is never used.
Solution:   Remove dead code.  Add a few more test cases.

patch 8.2.4939: matchfuzzypos() with "matchseq" does not have all positions

Problem:    matchfuzzypos() with "matchseq" does not have all positions.
Solution:   Also add a position for white space. (closes #10404)

patch 8.2.4938: crash when matching buffer with invalid pattern

Problem:    Crash when matching buffer with invalid pattern.
Solution:   Check for NULL regprog.

patch 8.2.4937: no test for what 8.2.4931 fixes

Problem:    No test for what 8.2.4931 fixes.
Solution:   Add a test that triggers a valgrind error.

patch 8.2.4936: MS-Windows: mouse coordinates for scroll event are wrong

Problem:    MS-Windows: mouse coordinates for scroll event are wrong.
Solution:   Convert coordinates to the text area coordinates. (closes #10400)

patch 8.2.4935: with 'foldmethod' "indent" some lines not included in fold

Problem:    With 'foldmethod' "indent" some lines are not included in the
            fold. (Oleg Koshovetc)
Solution:   Fix it. (Brandon Simmons, closes #10399, closes #3214)

patch 8.2.4934: string interpolation fails when not evaluating

Problem:    String interpolation fails when not evaluating.
Solution:   Skip the expression when not evaluating. (closes #10398)

patch 8.2.4933: a few more capitalization mistakes in error messages

Problem:    A few more capitalization mistakes in error messages.
Solution:   Adjust capitalization. (Doug Kearns)

patch 8.2.4932: not easy to filter the output of maplist()

Problem:    Not easy to filter the output of maplist().
Solution:   Add mode_bits to the dictionary. (Ernie Rael, closes #10356)

patch 8.2.4931: Crash with sequence of Perl commands

Problem:    Crash with sequence of Perl commands.
Solution:   Move PUTBACK to another line. (closes #10386)

patch 8.2.4930: interpolated string expression requires escaping

Problem:    Interpolated string expression requires escaping.
Solution:   Do not require escaping in the expression.

patch 8.2.4929: off-by-one error in in statusline item

Problem:    Off-by-one error in in statusline item.
Solution:   Subtrace one less. (closes #10394, closes #5599)

patch 8.2.4928: various white space and cosmetic mistakes

Problem:    Various white space and cosmetic mistakes.
Solution:   Change spaces to tabs, improve comments.

Update runtime files, translations

patch 8.2.4927: return type of remove() incorrect when using three arguments

Problem:    Return type of remove() incorrect when using three arguments.
Solution:   Use first argument type when there are three arguments.
            (closes #10387)

patch 8.2.4926: #ifdef for crypt feature around too many lines

Problem:    #ifdef for crypt feature around too many lines.
Solution:   Move code outside of #ifdef. (closes #10388)

patch 8.2.4925: trailing backslash may cause reading past end of line

Problem:    Trailing backslash may cause reading past end of line.
Solution:   Check for NUL after backslash.

patch 8.2.4924: maparg() may return a string that cannot be reused

Problem:    maparg() may return a string that cannot be reused.
Solution:   use msg_outtrans_special() instead of str2special().
            (closes #10384)

patch 8.2.4923: test checks for terminal feature unnecessarily

Problem:    Test checks for terminal feature unnecessarily.
Solution:   Remove CheckRunVimInTerminal. (closes #10383)

patch 8.2.4922: mouse test fails on MS-Windows

Problem:    Mouse test fails on MS-Windows.
Solution:   Set 'mousemodel' to "extend".

patch 8.2.4921: spell test fails because of new illegal byte check

Problem:    Spell test fails because of new illegal byte check.
Solution:   Remove the test.

patch 8.2.4920: MS-Windows GUI: unused variables

Problem:    MS-Windows GUI: unused variables.
Solution:   Delete the variables. (John Marriott)

patch 8.2.4919: can add invalid bytes with :spellgood

Problem:    Can add invalid bytes with :spellgood.
Solution:   Check for a valid word string.

patch 8.2.4918: conceal character from matchadd() displayed too many times

Problem:    Conceal character from matchadd() displayed too many times.
Solution:   Check the syntax flag. (closes #10381, closes #7268)

patch 8.2.4917: fuzzy expansion of option names is not right

Problem:    Fuzzy expansion of option names is not right.
Solution:   Pass the fuzzy flag down the call chain. (Christian Brabandt,
            closes #10380, closes #10318)

patch 8.2.4916: mouse in Insert mode test fails

Problem:    Mouse in Insert mode test fails.
Solution:   Fix the text and check relevant positions.

patch 8.2.4915: sometimes the cursor is in the wrong position

Problem:    Sometimes the cursor is in the wrong position.
Solution:   When the cursor moved to another line, recompute w_botline.
            (closes #9736)

patch 8.2.4914: string interpolation in :def function may fail

Problem:    String interpolation in :def function may fail.
Solution:   Do not terminate the expression. (closes #10377)

patch 8.2.4913: popup_hide() does not always have effect

Problem:    Popup_hide() does not always have effect.
Solution:   Add the POPF_HIDDEN_FORCE flag. (closes #10376)

Update runtime files

patch 8.2.4912: using execute() to define a lambda doesn't work

Problem:    Using execute() to define a lambda doesn't work. (Ernie Rael)
Solution:   Put the getline function in evalarg. (closes #10375)

patch 8.2.4911: the mode #defines are not clearly named

Problem:    The mode #defines are not clearly named.
Solution:   Prepend MODE_.  Renumber them to put the mapped modes first.

patch 8.2.4910: imperfect coding

Problem:    Imperfect coding.
Solution:   Make code nicer.

patch 8.2.4909: MODE_ enum entries names are too generic

Problem:    MODE_ enum entries names are too generic.
Solution:   use CH_MODE_.

patch 8.2.4908: no text formatting for // comment after a statement

Problem:    No text formatting for // comment after a statement.
Solution:   format a comment when the 'c' flag is in 'formatoptions'.

patch 8.2.4907: some users do not want a line comment always inserted

Problem:    Some users do not want a line comment always inserted.
Solution:   Add the '/' flag to 'formatoptions' to not repeat the comment
            leader after a statement when using "o".

patch 8.2.4906: MS-Windows: cannot use transparent background

Problem:    MS-Windows: cannot use transparent background.
Solution:   Make transparent background work with 'termguicolors' and NONE
            background color. (Yasuhiro Matsumoto, closes #10310, closes #7162)

patch 8.2.4905: codecov includes MS-Windows install header file

Problem:    codecov includes MS-Windows install header file.
Solution:   Ignore dosinst.h.

patch 8.2.4904: codecov includes MS-Windows install files

Problem:    codecov includes MS-Windows install files.
Solution:   Ignore dosinst.c and uninstall.c.

patch 8.2.4903: cannot get the current cmdline completion type and position

Problem:    Cannot get the current cmdline completion type and position.
Solution:   Add getcmdcompltype() and getcmdscreenpos(). (Shougo Matsushita,
            closes #10344)

patch 8.2.4902: mouse wheel scrolling is inconsistent

Problem:    Mouse wheel scrolling is inconsistent.
Solution:   Use the MS-Winows system setting. (closes #10368)

patch 8.2.4901: NULL pointer access when using invalid pattern

Problem:    NULL pointer access when using invalid pattern.
Solution:   Check for failed regexp program.

patch 8.2.4900: Vim9 expression test fails without the job feature

Problem:    Vim9 expression test fails without the job feature.
Solution:   Add a check for the job feature. (Dominique Pellé, closes #10373)

patch 8.2.4899: with latin1 encoding CTRL-W might go before the cmdline

Problem:    With latin1 encoding CTRL-W might go before the start of the
            command line.
Solution:   Check already being at the start of the command line.

patch 8.2.4898: Coverity complains about pointer usage

Problem:    Coverity complains about pointer usage.
Solution:   Move code for increment/decerment.