patch 8.2.3641: xxd code has duplicate expressions

Problem:    Xxd code has duplicate expressions.
Solution:   Refactor to avoid duplication. (closes #9185)

Update runtime files

patch 8.2.3640: freeze when calling term_wait() in a close callback

Problem:    Freeze when calling term_wait() in a close callback.
Solution:   Set a "closing" flag to tell term_wait() to return. (closes #9152)

patch 8.2.3639: line commented out accidentally

Problem:    Line commented out accidentally.
Solution:   Uncomment. (Volodymyr Kot, closes #9172)

patch 8.2.3638: getcompletion() always passes zero as position

Problem:    getcompletion() always passes zero as position to custom
            completion function.
Solution:   Pass the pattern length. (closes #9173)

patch 8.2.3637: typos in test files

Problem:    Typos in test files.
Solution:   Correct the typos. (Dominique Pellé, closes #9175)

patch 8.2.3636: Coverity warns for unreachable code

Problem:    Coverity warns for unreachable code.
Solution:   Remove unreachable else block.

patch 8.2.3635: GTK: composing underline does not show

Problem:    GTK: composing underline does not show.
Solution:   Include composing character in pango call. A few more
            optimizations for ligatures.  (Dusan Popovic, closes #9171,
            closes #9147)

patch 8.2.3634: error for already defined function uses wrong line number

Problem:    Error for already defined function uses wrong line number.
Solution:   Set SOURCING_LNUM before giving the error message. (closes #9085)

patch 8.2.3633: Vim9: line number of lambda is off by one

Problem:    Vim9: line number of lambda is off by one.
Solution:   Add one to the line number. (closes #9083)

patch 8.2.3632: GTK3: undercurl does not get removed properly

Problem:    GTK3: undercurl does not get removed properly.
Solution:   Set the cairo cursor first. (closes #9170)

patch 8.2.3631: "syntax enable" does not work properly in Vim9 context

Problem:    "syntax enable" does not work properly in Vim9 context.
Solution:   Also handle Vim9 context. (closes #9161)

patch 8.2.3630: printf() with %S does not handle multi-byte correctly

Problem:    Printf() with %S does not handle multi-byte correctly.
Solution:   Count cells instead of bytes. (closes #9169, closes #7486)

patch 8.2.3629: command completion in cmdline window uses global commands

Problem:    Command completion in cmdline window uses global user commands,
            not local commands for the window where it was opened from.
Solution:   Use local commands. (closes #9168)

patch 8.2.3628: looking terminal colors is a bit slow

Problem:    Looking terminal colors is a bit slow.
Solution:   Cache the terminal colors. (closes #9130, closes #9058)

patch 8.2.3627: difficult to know where the text starts in a window

Problem:    difficult to know where the text starts in a window. (Sergey
            Vlasov)
Solution:   Add the "textoff" entry in the result of getwininfo().
            (closes #9163)

patch 8.2.3626: "au! event" cannot be followed by another command

Problem:    "au!" and "au! event" cannot be followed by another command as
            documented.
Solution:   When a bar is found set nextcmd.

patch 8.2.3625: illegal memory access when C-indenting

Problem:    Illegal memory access when C-indenting.
Solution:   Also set the cursor column.

patch 8.2.3624: when renaming a terminal buffer status text is not updated

Problem:    When renaming a terminal buffer the status text is not updated.
Solution:   Clear the cached status text when renaming a terminal buffer.
            (closes #9162)

patch 8.2.3623: "$*" is expanded to "nonomatch"

Problem:    "$*" is expanded to "nonomatch".
Solution:   Only add "set nonomatch" when using a csh-like shell. (Christian
            Brabandt, closes #9159, closes #9153)

patch 8.2.3622: "verbose pwd" shows confusing info

Problem:    "verbose pwd" shows confusing info when :lcd does not change
            directory.
Solution:   Clear last_chdir_reason also when the directory does not change.
            (closes #9160)

patch 8.2.3621: build failure

Problem:    Build failure.
Solution:   Add missing change.

patch 8.2.3620: memory leak reported in libtlib

Problem:    Memory leak reported in libtlib.
Solution:   Call del_curterm() when cleaning up memory.  Rename term.h to
            termdefs.h to avoid a name clash.

patch 8.2.3619: cannot use a lambda for 'operatorfunc'

Problem:    Cannot use a lambda for 'operatorfunc'.
Solution:   Support using a lambda or partial. (Yegappan Lakshmanan,
            closes #8775)

patch 8.2.3618: getcwd() is unclear about how 'autochdir' is used

Problem:    getcwd() is unclear about how 'autochdir' is used.
Solution:   Update the help for getcwd().  Without any arguments always return
            the actual current directory. (closes #9142)

patch 8.2.3617: ":verbose pwd" does not mention 'autochdir' was applied

Problem:    ":verbose pwd" does not mention 'autochdir' was applied.
Solution:   Remember the last chdir was done by 'autochdir'.  (issue #9142)

patch 8.2.3616: arglist test does not clear the argument list consistently

Problem:    Arglist test does not clear the argument list consistently.
Solution:   Call Reset_arglist(). (Shougo Matsushita, closes #9154)

patch 8.2.3615: wrong indent in first line if re-formatting with indent expr

Problem:    When re-formatting with an indent expression the first line of a
            paragraph may get the wrong indent. (Martin F. Krafft)
Solution:   Apply the correct indenting function for the first line.
            (Christian Brabandt, closes #9150, closes #9056)

patch 8.2.3614: zindex of popup windows not used when redrawing popup menu

Problem:    zindex of popup windows not used when redrawing popup menu.
Solution:   Check the zindex when redrawing the popup menu.  (closes #9129,
            closes #9089)

patch 8.2.3613: :find test fails

Problem:    :find test fails.
Solution:   Put length check inside if block.

patch 8.2.3612: using freed memory with regexp using a mark

Problem:    Using freed memory with regexp using a mark.
Solution:   Get the line again after getting the mark position.

patch 8.2.3611: crash when using CTRL-W f without finding a file name

Problem:    Crash when using CTRL-W f without finding a file name.
Solution:   Bail out when the file name length is zero.

patch 8.2.3610: crash when ModeChanged triggered too early

Problem:    Crash when ModeChanged triggered too early.
Solution:   Trigger ModeChanged after setting VIsual.

patch 8.2.3609: internal error when ModeChanged is triggered recursively

Problem:    Internal error when ModeChanged is triggered when v:event is
            already in use.
Solution:   Save and restore v:event if needed.

patch 8.2.3608: users who type "q:" instead of ":q" are confused

Problem:    Users who type "q:" instead of ":q" are confused.
Solution:   Add an autocmd to give a message that explains this is the
            command-line window. (Egor Zvorykin, closes #9146)

patch 8.2.3607: GTK3 screen updating is slow

Problem:    GTK3 screen updating is slow.
Solution:   Remove some of the GTK3-specific code. (closes #9052)

patch 8.2.3606: file missing from list of distributed files

Problem:    File missing from list of distributed files.
Solution:   Add the file.

Update runtime files

Add missing change to version.c

patch 8.2.3605: cannot clear and unlinke a highlight group with hlset()

Problem:    Cannot clear and unlinke a highlight group with hlset() in a
            single call.
Solution:   Add the "force" option. (Yegappan Lakshmanan, closes #9117)

patch 8.2.3604: not all sudoers files are recognized

Problem:    Not all sudoers files are recognized.
Solution:   Add a file pattern. (Doug Kearns, closes #1192)

patch 8.2.3603: fish filetype not recognized

Problem:    Fish filetype not recognized.
Solution:   Add a file pattern and match script line. (Doug Kearns)

patch 8.2.3602: Python3 test fails with Python 3.10 on MS-Windows

Problem:    Python3 test fails with Python 3.10 on MS-Windows.
Solution:   Adjust the expected error. (Ken Takata, closes #9118)

patch 8.2.3601: check for overflow in put count does not work well

Problem:    Check for overflow in put count does not work well.
Solution:   Improve the overflow check. (Ozaki Kiichi, closes #9102)

patch 8.2.3600: filetype test fails

Problem:    Filetype test fails.
Solution:   Add missint change.

patch 8.2.3599: not all gdbinit files are recognized

Problem:    Not all gdbinit files are recognized.
Solution:   Add "gdbinit". (Doug Kearns)

patch 8.2.3598: RouterOS filetype is not recognized

Problem:    RouterOS filetype is not recognized.
Solution:   Add file and script patterns. (closes #9097)

patch 8.2.3597: Vim seems to hang when writing a long text to a terminal

Problem:    Vim seems to hang when writing a very long text to a terminal
            window.
Solution:   Limit the amount of text based on 'termwinscroll'. (issue #9080)

patch 8.2.3596: crash when using :pedit in Vim9 script

Problem:    Crash when using :pedit in Vim9 script.
Solution:   Move check for arguments to after checking there are arguments.
            (Yegappan Lakshmanan, closes #9134, closes #9135)

patch 8.2.3595: check for signed overflow might not work everywhere

Problem:    Check for signed overflow might not work everywhere.
Solution:   Limit to 32 bit int. (closes #9043, closes #9067)

patch 8.2.3594: xxd code is a bit difficult to understand

Problem:    Xxd code is a bit difficult to understand.
Solution:   Move some lines to a separate function. (closes #9037)

patch 8.2.3593: directory is wrong after executing "lcd" with win_execute()

Problem:    Directory is wrong after executing "lcd" with win_execute().
Solution:   Correct the directory when going back to the original window.
            (closes #9132)

patch 8.2.3592: Test_hlset fails when terminal has many columns

Problem:    Test_hlset fails when terminal has many columns.
Solution:   Set the number of columns to 80. (Dominique Pellé, closes #9101,
            closes #9100)

patch 8.2.3591: no event is triggered when closing a window

Problem:    No event is triggered when closing a window.
Solution:   Add the WinClosed event. (Naohiro Ono, closes #9110)

patch 8.2.3590: test for v:colornames sometimes fails

Problem:    Test for v:colornames sometimes fails. (Dominique Pellé)
Solution:   Check features.  Clear v:colornames between tests. (Drew Vogel,
            closes #9105, closes #9073)

patch 8.2.3589: failure when "term_rows" of term_start() is an unusual value

Problem:    Failure when the "term_rows" argument of term_start() is an
            unusual value.
Solution:   Limit to range of zero to 1000. (closes #9116)

patch 8.2.3588: break statement is never reached

Problem:    Break statement is never reached.
Solution:   Rely on return value of set_chars_option() not changing.
            (closes #9103)

patch 8.2.3587: compiler test fails with backslash file separator

Problem:    Compiler test fails with backslash file separator.
Solution:   Accept slash and backslash.

patch 8.2.3586: command completion test fails

Problem:    Command completion test fails.
Solution:   Add new argument to expected output

patch 8.2.3585: crash when passing float to "term_rows" of term_start()

Problem:    Crash when passing float to "term_rows" in the options argument of
            term_start(). (Virginia Senioria)
Solution:   Bail out if the argument is not a number. (closes #9116)

patch 8.2.3584: "verbose set efm" reports location of the :compiler command

Problem:    "verbose set efm" reports the location of the :compiler command.
            (Gary Johnson)
Solution:   Add the "-keepscript" argument to :command and use it when
            defining CompilerSet.

patch 8.2.3583: the "gd" and "gD" commands do not update search stats

Problem:    The "gd" and "gD" commands do not update search stats. (Gary
            Johnson)
Solution:   Clear search stats.

Update runtime files

patch 8.2.3582: reading uninitialized memory when giving spell suggestions

Problem:    Reading uninitialized memory when giving spell suggestions.
Solution:   Check that preword is not empty.

patch 8.2.3581: reading character past end of line

Problem:    Reading character past end of line.
Solution:   Correct the cursor column.

patch 8.2.3580: gj does not move properly with a wide character

Problem:    gj does not move properly with a wide character.
Solution:   Move one to the right. (Christian Brabandt, closes #8702)

patch 8.2.3579: CI sometimes fails for MinGW

Problem:    CI sometimes fails for MinGW.
Solution:   Use backslashes in HandleSwapExists(). (Christian Brabandt,
            closes #9078)

Add missing piece of patch 8.2.3578

patch 8.2.3578: manipulating highlighting is complicated

Problem:    Manipulating highlighting is complicated.
Solution:   Add the hlget() and hlset() functions. (Yegappan Lakshmanan,
            closes #9039)

patch 8.2.3577: overflow check fails with 32 ints

Problem:    Overflow check fails with 32 ints.
Solution:   Only test with 64 bit ints.

patch 8.2.3576: some functions are not documented for use with a method

Problem:    Some functions are not documented for use with a method.
Solution:   Add examples.  Fix that sign_unplacelist() only takes one
            argument. (Sean Dewar, closes #9081)

patch 8.2.3575: overflow check still fails when sizeof(int) == sizeof(long)

Problem:    Overflow check still fails when sizeof(int) == sizeof(long).
Solution:   Use a float to check the result.

patch 8.2.3574: divide by zero

Problem:    Divide by zero.
Solution:   Don't check for overflow if multiplicand is zero.

patch 8.2.3573: cannot decide whether to skip test that fails with 64 bit

Problem:    Cannot decide whether to skip test that fails with 64 bit ints.
            (closes #9072)
Solution:   Add v:sizeofint, v:sizeoflong and v:sizeofpointer.  Improve the
            check for multiply overflow.

patch 8.2.3572: memory leak when closing window and using "multispace"

Problem:    Memory leak when closing window and using "multispace" in
            'listchars'.
Solution:   Free the memory. (closes #9071)

patch 8.2.3571: some unicode control characters are considered printable

Problem:    Some unicode control characters are considered printable.
Solution:   Make 0x2060 - 0x2069 not printable.

patch 8.2.3570: Test_very_large_count fails on 32bit systems

Problem:    Test_very_large_count fails on 32bit systems.
Solution:   Bail out when using 32 bit numbers. (closes #9072)

patch 8.2.3569: error for :let when vimrc is Vim 9 script

Problem:    Error for :let when vimrc is Vim 9 script.
Solution:   Prepend :legacy in the code for converting arguments. (Christian
            Brabandt, closes #9068, closes #9077)

patch 8.2.3568: ctrl-hat test fails with Athena and Motif

Problem:    Ctrl-hat test fails with Athena and Motif. (Elimar Riesebieter)
Solution:   Run the test only with GTK. (Dominique Pellé, closes #9069)

patch 8.2.3567: CTRL-I in Insert mode is not tested

Problem:    CTRL-I in Insert mode is not tested
Solution:   Add a test case. (Dominique Pellé, closes #8866)

patch 8.2.3566: build failure on old systems when using nano timestamp

Problem:    Build failure on old systems when using nano timestamp.
Solution:   Define _BSD_SOURCE, _SVID_SOURCE and _DEFAULT_SOURCE. (Gary
            Johnson, closes #9054)

patch 8.2.3565: Makefile dependencies are outdated

Problem:    Makefile dependencies are outdated. (Gary Johnson)
Solution:   Run "make depend" and add missing dependencies.

patch 8.2.3564: invalid memory access when scrolling without valid screen

Problem:    Invalid memory access when scrolling without a valid screen.
Solution:   Do not set VALID_BOTLINE in w_valid.

patch 8.2.3563: build failure with +eval but without GUI or +termguicolors

Problem:    Build failure with +eval but without GUI or +termguicolors
Solution:   Adjust #ifdef. (John Marriott)

patch 8.2.3562: cannot add color names

Problem:    Cannot add color names.
Solution:   Add the v:colornames dictionary. (Drew Vogel, closes #8761)

patch 8.2.3561: cscope has a complicated way of giving an error message

Problem:    Cscope has a complicated way of giving an error message.
Solution:   Use semsg(). (James McCoy, closes #9038)

patch 8.2.3560: using freed memory with lambda

Problem:    Using freed memory with lambda.
Solution:   Do not free lines early, keep them until the expression is
            finished.

patch 8.2.3559: loop variable recreated every time

Problem:    Loop variable recreated every time.
Solution:   Keep the loop variable when looping.

Update runtime files

patch 8.2.3558: Vim9: asserting the wrong variable

Problem:    Vim9: asserting the wrong variable.
Solution:   Don't use Foo, use Goo.

patch 8.2.3557: Vim9: cannot call imported funcref at script level

Problem:    Vim9: cannot call imported funcref at script level.
Solution:   Check for an imported function. (closes #9007)

patch 8.2.3556: filler lines are incorrect for other window in diff mode

Problem:    Filler lines are incorrect for other window in diff mode after
            making a change.
Solution:   Copy filler lines from the current window. (closes #8809)

patch 8.2.3555: ModeChanged is not triggered on every mode change

Problem:    ModeChanged is not triggered on every mode change.
Solution:   Also trigger on minor mode changes. (Maguns Gross, closes #8999)

patch 8.2.3554: xxd has various way to exit

Problem:    Xxd has various way to exit.
Solution:   Add function to print error and exit. (closes #9035)

patch 8.2.3553: xxd test fails on MS-Windows

Problem:    Xxd test fails on MS-Windows.
Solution:   Split shell command in two.

patch 8.2.3552: xxd revert does not handle end of line correctly

Problem:    Xxd revert does not handle end of line correctly.
Solution:   Check for newline first. (closes #9034)

patch 8.2.3551: checking first character of url twice

Problem:    Checking first character of url twice.
Solution:   Only check once. (closes #9026)

patch 8.2.3550: completion() does not work properly

Problem:    completion() does not work properly.
Solution:   Set xp_line and add WILD_HOME_REPLACE. (Shougo Matsushita,
            closes #9016)

patch 8.2.3549: mistakes in test comments

Problem:    Mistakes in test comments.
Solution:   Fix the comments. (closes #9029)

patch 8.2.3548: GTK GUI crashen when reading from stdin

Problem:    GTK GUI crashen when reading from stdin.
Solution:   Do not overwrite the NUL after the string. (closes #9028)