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.

patch 8.2.4897: comment inside an expression in lambda ignores the rest

Problem:    Comment inside an expression in lambda ignores the rest of the
            expression.
Solution:   Truncate the line at the comment. (closes #10367)

patch 8.2.4896: expression in command block does not look after NL

Problem:    Expression in command block does not look after NL when command is
            typed.
Solution:   Skip over NL also when not in a script. (closes #10358)

patch 8.2.4895: buffer overflow with invalid command with composing chars

Problem:    Buffer overflow with invalid command with composing chars.
Solution:   Check that the whole character fits in the buffer.

patch 8.2.4894: MS-Windows: not using italics

Problem:    MS-Windows: not using italics.
Solution:   Use italics.  Simplify the code. (closes #10359)

patch 8.2.4893: distributed import files are not installed

Problem:    Distributed import files are not installed.
Solution:   Add rules to Makefile and NSIS.

patch 8.2.4892: test failures because of changed error messages

Problem:    Test failures because of changed error messages.
Solution:   Adjust the exptected error messages.

patch 8.2.4891: Vim help presentation could be better

Problem:    Vim help presentation could be better.
Solution:   Add an imported file for extra Vim help support.  Show highlight
            names in the color they have.

patch 8.2.4890: inconsistent capitalization in error messages

Problem:    Inconsistent capitalization in error messages.
Solution:   Make capitalization consistent. (Doug Kearns)

patch 8.2.4889: CI only tests with FreeBSD 12

Problem:    CI only tests with FreeBSD 12.
Solution:   Also test with FreeBSD 13. (closes #10366)

patch 8.2.4888: line number of lambda ignores line continuation

Problem:    Line number of lambda ignores line continuation.
Solution:   Use the line number of where the arguments are. Avoid outputting
            "..." twice. (closes #10364)

patch 8.2.4887: channel log does not show invoking a timer callback

Problem:    Channel log does not show invoking a timer callback.
Solution:   Add a ch_log() call.

patch 8.2.4886: Vim9: redir in skipped block seen as assignment

Problem:    Vim9: redir in skipped block seen as assignment.
Solution:   Check for valid assignment.

patch 8.2.4885: test fails with the job/channel feature

Problem:    Test fails with the job/channel feature.
Solution:   Move check for job/channel separately.

patch 8.2.4884: test fails without the job/channel feature

Problem:    Test fails without the job/channel feature. (Dominique Pellé)
Solution:   Add condition.

patch 8.2.4883: string interpolation only works in heredoc

Problem:    String interpolation only works in heredoc.
Solution:   Support interpolated strings.  Use syntax for heredoc consistent
            with strings, similar to C#. (closes #10327)

patch 8.2.4882: cannot make 'breakindent' use a specific column

Problem:    Cannot make 'breakindent' use a specific column.
Solution:   Add the "column" entry in 'breakindentopt'. (Christian Brabandt,
            closes #10362, closes #10325)

patch 8.2.4881: "P" in Visual mode still changes some registers

Problem:    "P" in Visual mode still changes some registers.
Solution:   Make "P" in Visual mode not change any register. (Shougo
            Matsushita, closes #10349)

patch 8.2.4880: Vim9: misplaced elseif causes invalid memory access

Problem:    Vim9: misplaced elseif causes invalid memory access.
Solution:   Check cs_idx not to be negative.

patch 8.2.4879: screendump test may fail when using valgrind

Problem:    Screendump test may fail when using valgrind.
Solution:   Wait longer for the first screendump.

patch 8.2.4878: valgrind warning for using uninitialized variable

Problem:    Valgrind warning for using uninitialized variable.
Solution:   Initialize the type of newtv.

patch 8.2.4877: MS-Windows: Wrongly using Normal colors for termguicolors

Problem:    MS-Windows: Using Normal colors for termguicolors causes problems.
Solution:   Do not use Normal colors to set sg_gui_fg and sg_gui_bg.
            (Christian Brabandt, closes #10317, closes #10241)

patch 8.2.4876: MS-Windows: Shift-BS results in strange char in powershell

Problem:    MS-Windows: Shift-BS results in strange character in powershell.
Solution:   Add K_S_BS. (Christian Brabandt, closes #10283, closes #10279)

patch 8.2.4875: MS-Windows: some .exe files are not recognized

Problem:    MS-Windows: some .exe files are not recognized.
Solution:   Parse APPEXECLINK junctions. (closes #10302)

patch 8.2.4874: Win32 GUI: horizontal scroll wheel not handled properly

Problem:    Win32 GUI: horizontal scroll wheel not handled properly.
Solution:   Also handle WM_MOUSEHWHEEL. (closes #10309)

patch 8.2.4873: Vim9: using "else" differs from using "endif/if !cond"

Problem:    Vim9: using "else" differs from using "endif/if !cond".
Solution:   Leave the block and enter another one. (closes #10320)

patch 8.2.4872: Vim9: no error for using an expression only

Problem:    Vim9: no error for using an expression only at the script level
            when followed by an empty line.
Solution:   Do not check the line number but whether something follows.
            (closes #10357)

patch 8.2.4871: Vim9: in :def function no error for misplaced range

Problem:    Vim9: in :def function no error for using a range with a command
            that does not accept one.
Solution:   Check for the command to accept a range. (closes #10330)

patch 8.2.4870: Vim9: expression in :substitute is not compiled

Problem:    Vim9: expression in :substitute is not compiled.
Solution:   Use an INSTR instruction if possible. (closes #10334)

patch 8.2.4869: expression in command block does not look after NL

Problem:    Expression in command block does not look after NL.
Solution:   Skip over NL to check what follows. (closes #10358)

patch 8.2.4868: when closing help window autocmds triggered for wrong window

Problem:    When closing help window autocmds triggered for the wrong window.
Solution:   Figure out the new current window earlier. (closes #10348)

patch 8.2.4867: listing of mapping with K_SPECIAL is wrong

Problem:    Listing of mapping with K_SPECIAL is wrong.
Solution:   Adjust escaping of special characters. (closes #10351)

patch 8.2.4866: duplicate code in "get" functions

Problem:    Duplicate code in "get" functions.
Solution:   Use get_var_from() for getwinvar(), gettabvar(), gettabwinvar()
            and getbufvar(). (closes #10335)

patch 8.2.4865: :startinsert right after :stopinsert may not work

Problem:    :startinsert right after :stopinsert does not work when popup menu
            is still visible.
Solution:   Use ins_compl_active() instead of pum_visible(). (closes #10352)

patch 8.2.4864: Vim9: script test fails

Problem:    Vim9: script test fails.
Solution:   Remove "if" around declaration.

patch 8.2.4863: accessing freed memory in test without the +channel feature

Problem:    Accessing freed memory in test without the +channel feature.
            (Dominique Pellé)
Solution:   Do not generted PUSHCHANNEL or PUSHJOB if they are not
            implemented. (closes #10350)

patch 8.2.4862: Vim9: test may fail when run with valgrind

Problem:    Vim9: test may fail when run with valgrind.
Solution:   Wait longer for callback if needed.

patch 8.2.4861: it is not easy to restore saved mappings

Problem:    It is not easy to restore saved mappings.
Solution:   Make mapset() accept a dict argument. (Ernie Rael, closes #10295)

patch 8.2.4860: MS-Windows: always uses current directory for executables

Problem:    MS-Windows: always uses current directory for executables.
Solution:   Check the NoDefaultCurrentDirectoryInExePath environment variable.
            (Yasuhiro Matsumoto, closes #10341)

patch 8.2.4859: wget2 files are not recognized

Problem:    wget2 files are not recognized.
Solution:   Add patterns to recognize wget2. (Doug Kearns)

patch 8.2.4858: K_SPECIAL may be escaped twice

Problem:    K_SPECIAL may be escaped twice.
Solution:   Avoid double escaping. (closes #10340)

patch 8.2.4857: Yaml indent for multiline is wrong

Problem:    Yaml indent for multiline is wrong.
Solution:   Adjust patterns. (closes #10328, closes #8740)

patch 8.2.4856: MinGW compiler complains about unknown escape sequence

Problem:    MinGW compiler complains about unknown escape sequence.
Solution:   Avoid using a backslash in path. (Christian Brabandt,
            closes #10337)

patch 8.2.4855: robot files are not recognized

Problem:    Robot files are not recognized.
Solution:   Add patterns for robot files. (Zoe Roux, closes #10339)

patch 8.2.4854: array size does not match usage

Problem:    Array size does not match usage.
Solution:   Make array size 3 instead of 4. (Christian Brabandt, closes #10336)

patch 8.2.4853: CI with FreeBSD is a bit outdated

Problem:    CI with FreeBSD is a bit outdated.
Solution:   Use 12.3 instead of 12.1. (closes #10333)

patch 8.2.4852: ANSI color index to RGB value not correct

Problem:    ANSI color index to RGB value not correct.
Solution:   Convert the cterm index to ANSI index. (closes #10321,
            closes #9836))

patch 8.2.4851: compiler warning for uninitialized variable

Problem:    Compiler warning for uninitialized variable.
Solution:   Use another variable to decide to restore option values.

patch 8.2.4850: mksession mixes up "tabpages" and "curdir" arguments

Problem:    Mksession mixes up "tabpages" and "curdir" arguments.
Solution:   Correct logic for storing tabpage in session. (closes #10312)

patch 8.2.4849: Gleam filetype not detected

Problem:    Gleam filetype not detected.
Solution:   Add a pattern for Gleam files. (Mathias Jean Johansen,
            closes #10326)