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)

patch 8.2.5037: cursor position may be invalid after "0;" range

Problem:    Cursor position may be invalid after "0;" range.
Solution:   Check the cursor position when it was set by ";" in the range.

patch 8.2.5036: using two counters for timeout check in NFA engine

Problem:    Using two counters for timeout check in NFA engine.
Solution:   Use only one counter.  Tune the counts based on guessing.

patch 8.2.5035: when splitting a window the changelist position moves

Problem:    When splitting a window the changelist position moves.
Solution:   Set the changelist index a bit later. (closes #10493)

patch 8.2.5034: there is no way to get the byte index from a virtual column

Problem:    There is no way to get the byte index from a virtual column.
Solution:   Add virtcol2col(). (Yegappan Lakshmanan, closes #10477,
            closes #10098)

patch 8.2.5033: build error with +eval but without +quickfix

Problem:    Build error with +eval but without +quickfix.  Warning for
            uninitialized variable.
Solution:   Adjust #ifdefs. (John Marriott)

patch 8.2.5032: Python 3 test fails without the GUI

Problem:    Python 3 test fails without the GUI.
Solution:   Check the balloon_eval feature is available.

patch 8.2.5031: cannot easily run the benchmarks

Problem:    Cannot easily run the benchmarks.
Solution:   Have "make benchmark" in the src directory work.

patch 8.2.5030: autocmd_add() can only handle one event and pattern

Problem:    autocmd_add() can only handle one event and pattern.
Solution:   Support a list of events and patterns. (Yegappan Lakshmanan,
            closes #10483)

patch 8.2.5029: "textlock" is always zero

Problem:    "textlock" is always zero.
Solution:   Remove "textlock" and rename "textwinlock" to "textlock".
            (closes #10489)

patch 8.2.5028: syntax regexp matching can be slow

Problem:    Syntax regexp matching can be slow.
Solution:   Adjust the counters for checking the timeout to check about once
            per msec. (closes #10487, closes #2712)

patch 8.2.5027: error for missing :endif when an exception was thrown

Problem:    Error for missing :endif when an exception was thrown. (Dani
            Dickstein)
Solution:   Do not give an error when aborting. (closes #10490)

patch 8.2.5026: Vim9: a few lines not covered by tests

Problem:    Vim9: a few lines not covered by tests.
Solution:   Delete dead code.  Add a few test cases. make "12->func()" work.

patch 8.2.5025: Vim9: a few lines not covered by tests

Problem:    Vim9: a few lines not covered by tests.
Solution:   Add a few tests.

patch 8.2.5024: using freed memory with "]d"

Problem:    Using freed memory with "]d".
Solution:   Copy the pattern before searching.

patch 8.2.5023: substitute overwrites allocated buffer

Problem:    Substitute overwrites allocated buffer.
Solution:   Disallow undo when in a substitute command.

patch 8.2.5022: 'completefunc'/'omnifunc' error does not end completion

Problem:    'completefunc'/'omnifunc' error does not end completion.
Solution:   Check if there was an error or exception. (closes #10486,
            closes #4218)

patch 8.2.5021: build fails with normal features and +terminal

Problem:    Build fails with normal features and +terminal. (Dominique Pellé)
Solution:   Add #ifdefs. (closes #10484)

patch 8.2.5020: using 'imstatusfunc' and 'imactivatefunc' breaks 'foldopen'

Problem:    Using 'imstatusfunc' and 'imactivatefunc' breaks 'foldopen'.
Solution:   Save and restore the KeyTyped flag. (closes #10479)

patch 8.2.5019: cannot get the first screen column of a character

Problem:    Cannot get the first screen column of a character.
Solution:   Let virtcol() optionally return a list. (closes #10482,
            closes #7964)

patch 8.2.5018: Vim9: some code is not covered by tests

Problem:    Vim9: some code is not covered by tests.
Solution:   Delete dead code.

patch 8.2.5017: gcc 12.1 warns for uninitialized variable

Problem:    Gcc 12.1 warns for uninitialized variable.
Solution:   Initialize the variable. (closes #10476)

patch 8.2.5016: access before start of text with a put command

Problem:    Access before start of text with a put command.
Solution:   Check the length is more than zero.

patch 8.2.5015: Hoon and Moonscript files are not recognized

Problem:    Hoon and Moonscript files are not recognized.
Solution:   Add filetype patterns. (Goc Dundar, closes #10478)

patch 8.2.5014: byte offsets are wrong when using text properties

Problem:    Byte offsets are wrong when using text properties.
Solution:   Make sure text properties do not affect the byte counts.
            (Paul Ollis, closes #10474)

patch 8.2.5013: after text formatting cursor may be in an invalid position

Problem:    After text formatting the cursor may be in an invalid position.
Solution:   Correct the cursor position after formatting.

patch 8.2.5012: cannot select one character inside ()

Problem:    Cannot select one character inside ().
Solution:   Do not try to extend the area if it is empty. (closes #10472,
            closes #6616)

patch 8.2.5011: Replacing an autocommand requires several lines

Problem:    Replacing an autocommand requires several lines.
Solution:   Add the "replace" flag to autocmd_add(). (Yegappan Lakshmanan,
            closes #10473)

patch 8.2.5010: the terminal debugger uses various global variables

Problem:    The terminal debugger uses various global variables.
Solution:   Add a dictionary to hold the terminal debugger preferences.

patch 8.2.5009: fold may not be closeable after appending

Problem:    Fold may not be closeable after appending.
Solution:   Set the fd_small flag to MAYBE. (Brandon Simmons, closes #10471)

patch 8.2.5008: when 'formatoptions' contains "/" wrongly wrapping comment

Problem:    When 'formatoptions' contains "/" wrongly wrapping a long trailing
            comment.
Solution:   Pass the OPENLINE_FORMAT flag.

patch 8.2.5007: spell suggestion may use uninitialized memory

Problem:    Spell suggestion may use uninitialized memory. (Zdenek Dohnal)
Solution:   Avoid going over the end of the word.

patch 8.2.5006: asan warns for undefined behavior

Problem:    Asan warns for undefined behavior.
Solution:   Cast the shifted value to unsigned.

patch 8.2.5005: compiler warning for uninitialized variable

Problem:    Compiler warning for uninitialized variable. (John Marriott)
Solution:   Initialize the pointer to NULL.

patch 8.2.5004: right shift on negative number does not work as documented

Problem:    Right shift on negative number does not work as documented.
Solution:   Use a uvarnumber_T type cast.

patch 8.2.5003: cannot do bitwise shifts

Problem:    Cannot do bitwise shifts.
Solution:   Add the >> and << operators. (Yegappan Lakshmanan, closes #8457)

patch 8.2.5002: deletebufline() may change Visual selection

Problem:    deletebufline() may change Visual selection.
Solution:   Disable Visual mode when using another buffer. (closes #10469)

patch 8.2.5001: checking translations affects the search pattern history

Problem:    Checking translations affects the search pattern history.
Solution:   Use "keeppatterns". (Doug Kearns)

patch 8.2.5000: no patch for documentation updates

Problem:    No patch for documentation updates.
Solution:   Update documentation files.

patch 8.2.4999: filetype test table is not properly sorted

Problem:    Filetype test table is not properly sorted.
Solution:   Sort by filetype. (Doug Kearns)

patch 8.2.4998: Vim9: crash when using multiple funcref()

Problem:    Vim9: crash when using multiple funcref().
Solution:   Check if varargs type is NULL. (closes #10467)

patch 8.2.4997: Python: changing hidden buffer can cause display mess up

Problem:    Python: changing hidden buffer can cause the display to be messed
            up.
Solution:   Do not mark changed lines when using another buffer. (Paul Ollis,
            closes #10437, closes #7972)

patch 8.2.4996: setbufline() may change Visual selection

Problem:    setbufline() may change Visual selection. (Qiming Zhao)
Solution:   Disable Visual mode when using another buffer. (closes #10466)

patch 8.2.4995: still a compiler warning for possibly uninitialized variable

Problem:    Still a compiler warning for possibly uninitialized variable.
            (Tony Mechelynck)
Solution:   Initialize variables.

patch 8.2.4994: tests are using legacy functions

Problem:    Tests are using legacy functions.
Solution:   Convert a few tests to use :def functions.

patch 8.2.4993: smart/C/lisp indenting is optional

Problem:    smart/C/lisp indenting is optional, which makes the code more
            complex, while it only reduces the executable size a bit.
Solution:   Graduate FEAT_CINDENT, FEAT_SMARTINDENT and FEAT_LISP.

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)