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)

patch 8.2.3547: opening the quickfix window triggers BufWinEnter twice

Problem:    Opening the quickfix window triggers BufWinEnter twice. (Yorick
            Peterse)
Solution:   Only trigger BufWinEnter with "quickfix". (closes #9022)

patch 8.2.3546: build failure without the +eval feature

Problem:    Build failure without the +eval feature.
Solution:   Add #ifdef. (closes #9025)

patch 8.2.3545: setcellwidths() may make 'listchars' or 'fillchars' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes #9024)

patch 8.2.3544: Unix: may leak file descriptor using non-existing directory

Problem:    Unix: may leak file descriptor when using a non-existing
            directory.
Solution:   Always close the file. (closes #9023)

patch 8.2.3543: swapname has double slash when 'directory' ends in it

Problem:    Swapname has double slash when 'directory' ends in double slash.
            (Shane Smith)
Solution:   Remove the superfluous slash. (closes #8876)

patch 8.2.3542: too many comments are old style

Problem:    Too many comments are old style.
Solution:   Change comments to // style. (closes #9021)

patch 8.2.3541: compiler warning for unused variable in tiny version

Problem:    Compiler warning for unused variable in tiny version.
Solution:   Add #ifdef. (John Marriott)

patch 8.2.3540: the mark '] is wrong after put with a count

Problem:    The mark '] is wrong after put with a count. (Naohiro Ono)
Solution:   Use the right line number. (closes #8956)

patch 8.2.3539: GTK3: with 'rightleft' set scrollbar may move unintentionally

Problem:    GTK3: with 'rightleft' set scrollbar may move unintentionally.
Solution:   Ignore events while moving the scrollbar thumb. (closes #8958)

patch 8.2.3538: else-if indenting is confusing

Problem:    Else-if indenting is confusing.
Solution:   Add curly brackets. (Yegappan Lakshmanan, closes #9017)

patch 8.2.3537: mode() does not return the right value in 'operatorfunc'

Problem:    mode() does not return the right value in 'operatorfunc'.
Solution:   Reset finish_op while calling 'operatorfunc'.

patch 8.2.3536: the do_highlight() function is way too long

Problem:    The do_highlight() function is way too long.
Solution:   Split it into several functions. (Yegappan Lakshmanan,
            closes #9011)

patch 8.2.3535: if-else indenting is confusing

Problem:    If-else indenting is confusing.
Solution:   Add curly brackets and indent. (Dominique Pellé, closes #9010)

patch 8.2.3534: autoread test is a bit flaky

Problem:    Autoread test is a bit flaky.
Solution:   Wait a brief moment before overwriting the file.

patch 8.2.3533: inefficient code in xxd

Problem:    Inefficient code in xxd.
Solution:   Don't use "p" when "hextype" is non-zero. (closes #9013)

patch 8.2.3532: the previous '' mark is restored after moving the cursor

Problem:    The previous '' mark is restored after moving the cursor to the
            original jump position. (Tony Chen)
Solution:   Forget the previous position after checking. (closes #8985)

patch 8.2.3531: command line completion test fails on MS-Windows

Problem:    Command line completion test fails on MS-Windows.
Solution:   Do not test with "\{" on MS-Windows.

patch 8.2.3530: ":buf \{a}" fails while ":edit \{a}" works

Problem:    ":buf \{a}" fails while ":edit \{a}" works.
Solution:   Unescape "\{". (closes #8917)

patch 8.2.3529: xxd usage output is incomplete

Problem:    Xxd usage output is incomplete.
Solution:   Add "bytes" to "-g" flag. (Atsushi Sugawara, closes #8944)

patch 8.2.3528: 'thesaurus' and 'thesaurusfunc' do not have the same scope

Problem:    'thesaurus' and 'thesaurusfunc' do not have the same scope.
Solution:   Make 'thesaurusfunc' global-local.

patch 8.2.3527: gcc complains about uninitialized variable

Problem:    Gcc complains about uninitialized variable. (Tony Mechelynck)
Solution:   Initialize it.

patch 8.2.3526: tests have clumsy check for X11 based GUI

Problem:    Tests have clumsy check for X11 based GUI.
Solution:   Add CheckX11BasedGui.

patch 8.2.3525: option variable name does not match option name

Problem:    Option variable name does not match option name. (Christ van
            Willigen)
Solution:   Rename the variable.

patch 8.2.3524: GUI: ligatures are not used

Problem:    GUI: ligatures are not used.
Solution:   Add the 'guiligatures' option. (Dusan Popovic, closes #8933)

patch 8.2.3523: duplicated code in xxd

Problem:    Duplicated code in xxd.
Solution:   Remove duplicated lines. (closes #8972)

patch 8.2.3522: cannot use \x and \u when setting 'listchars'

Problem:    Cannot use \x and \u when setting 'listchars'.
Solution:   Support hex and unicode in hex form. (closes #9006)

patch 8.2.3521: options completion test fails

Problem:    Options completion test fails.
Solution:   Add 'thesaurusfunc' to the results.

patch 8.2.3520: cannot define a function for thesaurus completion

Problem:    Cannot define a function for thesaurus completion.
Solution:   Add 'thesaurusfunc'. (Yegappan Lakshmanan, closes #8987,
            closes 8950)

Update runtime files

patch 8.2.3519: TOML files are not recognized

Problem:    TOML files are not recognized.
Solution:   Add filetype patterns for TOML. (Aman Verma, closes #8984)

patch 8.2.3518: Test_xrestore sometimes fails

Problem:    Test_xrestore sometimes fails.
Solution:   Mark the test as flayky.  Move marking test as flaky to the test
            instead of listing them in runtest.

patch 8.2.3517: TextChanged does not trigger after TextChangedI

Problem:    TextChanged does not trigger after TextChangedI.
Solution:   Store the tick separately for TextChangedI. (Christian Brabandt,
            closes #8968, closes #8932)

patch 8.2.3516: terminal window does not have transparent background

Problem:    Terminal window does not have transparent background when
            'termguicolors' is used.
Solution:   Fix the background color. (closes #2361, closes #9002)

patch 8.2.3515: nano time test fails on Mac and FreeBSD

Problem:    Nano time test fails on Mac and FreeBSD.
Solution:   Also check nano time when not on Linux. (Ozaki Kiichi,
            closes #9000)

patch 8.2.3514: autoread test with nano second time sometimes fails

Problem:    Autoread test with nano second time sometimes fails.
Solution:   Mark the test as being flaky.

patch 8.2.3513: using freed memory when using a timer and searching

Problem:    Using freed memory when using a timer and searching. (Dominique
            Pellé)
Solution:   Allocated mr_pattern.

patch 8.2.3512: timestamp test fails on some systems

Problem:    Timestamp test fails on some systems.
Solution:   Sleep for a short while.

patch 8.2.3511: Vim9: entry for loop variable is created every round

Problem:    Vim9: entry for loop variable is created every round.
Solution:   Only create the entry once. (closes #8996)

patch 8.2.3510: changes are only detected with one second accuracy

Problem:    Changes are only detected with one second accuracy.
Solution:   Use the nanosecond time if possible.  (Leah Neukirchen,
            closes #8873, closes #8875)

patch 8.2.3509: undo file is not synced

Problem:    Undo file is not synced. (Sami Farin)
Solution:   Sync the undo file if 'fsync' is set. (Christian Brabandt,
            closes #8879, closes #8920)

patch 8.2.3508: Vim9: bad separators for "g" and "s" insufficiently tested

Problem:    Vim9: bad separators for "g" and "s" insufficiently tested.
Solution:   Add a few more test cases.

patch 8.2.3507: generating proto files may fail

Problem:    Generating proto files may fail.
Solution:   Define __attribute().

patch 8.2.3506: Vim9: special cases for "g" and "s" insufficiently tested

Problem:    Vim9: special cases for "g" and "s" insufficiently tested.
Solution:   Add a few more test cases.

patch 8.2.3505: Vim9: build failure without the +eval feature

Problem:    Vim9: build failure without the +eval feature.
Solution:   Add #ifdef.

patch 8.2.3504: Vim9: warning for signed vs unsigned

Problem:    Vim9: warning for signed vs unsigned.
Solution:   Add type cast.

patch 8.2.3503: Vim9: using g:pat:cmd is confusing

Problem:    Vim9: using g:pat:cmd is confusing.
Solution:   Do not recognize g: as the :global command.  Also for s:pat:repl.
            (closes #8982)