patch 8.1.2170: cannot build without the +termresponse feature

Problem:    Cannot build without the +termresponse feature.
Solution:   Add #ifdef.

patch 8.1.2169: terminal flags are never reset

Problem:    Terminal flags are never reset.
Solution:   Reset the flags when setting 'term'.

patch 8.1.2168: heredoc assignment not skipped in if block

Problem:    Heredoc assignment not skipped in if block.
Solution:   Check if "skip" is set. (closes #5063)

patch 8.1.2167: mapping test fails on MS-Windows

Problem:    Mapping test fails on MS-Windows.
Solution:   Remove all the existing Insert-mode mappings.

patch 8.1.2166: rubyeval() not tested as a method

Problem:    Rubyeval() not tested as a method.
Solution:   Change a test case.

Update runtime files.

patch 8.1.2165: mapping test fails on Mac

Problem:    Mapping test fails on Mac.
Solution:   Remove the default Mac mapping.

patch 8.1.2164: stuck when using "j" in a popupwin with popup_filter_menu

Problem:    Stuck when using "j" in a popupwin with popup_filter_menu if a
            line wraps.
Solution:   Check the cursor line is visible. (closes #4577)

patch 8.1.2163: cannot build with +spell but without +syntax

Problem:    Cannot build with +spell but without +syntax.
Solution:   Add #ifdef. (John Marriott)

patch 8.1.2162: popup resize test is flaky

Problem:    Popup resize test is flaky. (Christian Brabandt)
Solution:   Add the function to the list of flaky tests.

patch 8.1.2161: mapping test fails

Problem:    Mapping test fails.
Solution:   Run the test separately.

patch 8.1.2160: cannot build with +syntax but without +terminal

Problem:    Cannot build with +syntax but without +terminal.
Solution:   Add #ifdef.

patch 8.1.2159: some mappings are listed twice

Problem:    Some mappings are listed twice.
Solution:   Skip mappings duplicated for modifyOtherKeys. (closes #5064)

patch 8.1.2158: terminal attributes missing in Terminal-normal mode

Problem:    Terminal attributes missing in Terminal-normal mode.
Solution:   Use "syntax_attr".

patch 8.1.2157: libvterm source files missing from distribution

Problem:    Libvterm source files missing from distribution.
Solution:   Rename source files. (closes #5065)

patch 8.1.2156: first character after Tab is not highlighted

Problem:    First character after Tab is not highlighted.
Solution:   Remember the syntax attribute for a column.

patch 8.1.2155: in a terminal window 'cursorlineopt' does not work properly

Problem:    In a terminal window 'cursorlineopt' does not work properly.
Solution:   Check the 'cursorlineopt' value. (closes #5055)

patch 8.1.2154: quickfix window height wrong when there is a tabline

Problem:    Quickfix window height wrong when there is a tabline. (Daniel
            Hahler)
Solution:   Take the tabline height into account. (closes #5058)

patch 8.1.2153: combining text property and syntax highlight is wrong

Problem:    Combining text property and syntax highlight is wrong. (Nick
            Jensen)
Solution:   Compute the syntax highlight attribute much earlier.
            (closes #5057)

patch 8.1.2152: problems navigating tags file on MacOS Catalina

Problem:    Problems navigating tags file on MacOS Catalina.
Solution:   Use fseek instead of lseek. (John Lamb, fixes #5061)

patch 8.1.2151: state test is a bit flaky

Problem:    State test is a bit flaky.
Solution:   Add to the list of flaky tests.

patch 8.1.2150: no test for 'ttymouse' set from xterm version response

Problem:    No test for 'ttymouse' set from xterm version response.
Solution:   Test the three possible values.

patch 8.1.2149: crash when running out of memory very early

Problem:    Crash when running out of memory very early.
Solution:   Do not use IObuff when it's NULL. (closes #5052)

patch 8.1.2148: no test for right click extending Visual area

Problem:    No test for right click extending Visual area.
Solution:   Add a test. (Dominique Pelle, closes #5018)

patch 8.1.2147: crash when allocating memory fails

Problem:    Crash when allocating memory fails. (Zu-Ming Jiang)
Solution:   Check that 'spellcapcheck' is not NULL. (closes #5048)

patch 8.1.2146: build failure

Problem:    Build failure.
Solution:   Include omitted changed file.

patch 8.1.2145: cannot map <C-H> when modifyOtherKeys is enabled

Problem:    Cannot map <C-H> when modifyOtherKeys is enabled.
Solution:   Add the <C-H> mapping twice, both with modifier and as 0x08.  Use
            only the first one when modifyOtherKeys has been detected.

patch 8.1.2144: side effects when using t_ti to enable modifyOtherKeys

Problem:    Side effects when using t_ti to enable modifyOtherKeys.
Solution:   Add t_TI and t_TE.

patch 8.1.2143: cannot see each command even when 'verbose' is set

Problem:    Cannot see each command even when 'verbose' is set.
Solution:   List each command when 'verbose' is at least 16.

patch 8.1.2142: some key mappings do not work with modifyOtherKeys

Problem:    Some key mappings do not work with modifyOtherKeys.
Solution:   Remove the Shift modifier if it is already included in the key.

patch 8.1.2141: :tselect has an extra hit-enter prompt

Problem:    :tselect has an extra hit-enter prompt.
Solution:   Do not set need_wait_return when only moving the cursor.
            (closes #5040)

patch 8.1.2140: "gk" and "gj" do not work correctly in number column

Problem:    "gk" and "gj" do not work correctly in number column.
Solution:   Allow for a negative "curswant". (Zach Wegner, closes #4969)

patch 8.1.2139: the modifyOtherKeys codes are not tested

Problem:    The modifyOtherKeys codes are not tested.
Solution:   Add a test case.

patch 8.1.2138: including the build number in the Win32 binary is confusing

Problem:    Including the build number in the Win32 binary is confusing.
Solution:   Only use the patchlevel.

patch 8.1.2137: parsing the termresponse is not tested

Problem:    Parsing the termresponse is not tested.
Solution:   Add a first test. (related to #5042)

patch 8.1.2136: using freed memory with autocmd from fuzzer

Problem:    using freed memory with autocmd from fuzzer. (Dhiraj Mishra,
            Dominique Pelle)
Solution:   Avoid using "wp" after autocommands. (closes #5041)

patch 8.1.2135: with modifyOtherKeys Alt-a does not work properly

Problem:    With modifyOtherKeys Alt-a does not work properly.
Solution:   Remove the ALT modifier.  Get multi-byte after applying ALT.

patch 8.1.2134: modifier keys are not always recognized

Problem:    Modifier keys are not always recognized.
Solution:   Handle key codes generated by xterm with modifyOtherKeys set.
            Add this to libvterm so we can debug it.

patch 8.1.2133: some tests fail when run as root

Problem:    Some tests fail when run as root.
Solution:   Add CheckNotRoot and use it. (James McCoy, closes #5020)

patch 8.1.2132: MS-Windows: screen mess when not recognizing insider build

Problem:    MS-Windows: screen mess when not recognizing insider build.
Solution:   Always move the cursor to the first column first. (Nobuhiro
            Takasaki, closes #5036)

patch 8.1.2131: MSVC tests fail

Problem:    MSVC tests fail.
Solution:   Replace backslashes with slashes.

patch 8.1.2130: MSVC build fails

Problem:    MSVC build fails.
Solution:   Add the source file name explicitly.

patch 8.1.2129: using hard coded executable path in test

Problem:    Using hard coded executable path in test.
Solution:   Use v:progpath. Use $VIMRUNTIME instead of "runtime". (James
            McCoy, closes #5025)

patch 8.1.2128: renamed libvterm sources makes merging difficult

Problem:    Renamed libvterm sources makes merging difficult.
Solution:   Rename back to the original name and only rename the .o files.
            Also clean the libvterm build artifacts. (James McCoy,
            closes #5027)

patch 8.1.2127: the indent.c file is a bit big

Problem:    The indent.c file is a bit big.
Solution:   Move C-indent code a a new cindent.c file.  Move other
            indent-related code to indent.c. (Yegappan Lakshmanan,
            closes #5031)

patch 8.1.2126: viminfo not sufficiently tested

Problem:    Viminfo not sufficiently tested.
Solution:   Add more test cases.  Clean up comments. (Yegappan Lakshmanan,
            closes #5032)

patch 8.1.2125: fnamemodify() fails when repeating :e

Problem:    Fnamemodify() fails when repeating :e.
Solution:   Do not go before the tail. (Rob Pilling, closes #5024)

patch 8.1.2124: ruler is not updated if win_execute() moves cursor

Problem:    Ruler is not updated if win_execute() moves cursor.
Solution:   Update the status line. (closes #5022)

patch 8.1.2123: parsing CSI sequence is messy

Problem:    Parsing CSI sequence is messy.
Solution:   Generalize parsing a CSI sequence.

patch 8.1.2122: cannot build without terminal feature

Problem:    Cannot build without terminal feature.
Solution:   Add #ifdef.

patch 8.1.2121: mode is not updated when switching to terminal

Problem:    Mode is not updated when switching to terminal in Insert mode.
Solution:   Redraw the mode when entering a terminal window. (Jason Franklin)

patch 8.1.2120: some MB_ macros are more complicated than necessary

Problem:    Some MB_ macros are more complicated than necessary. (Dominique
            Pelle)
Solution:   Simplify the macros.  Expand inline.

patch 8.1.2119: memory access error for empty string

Problem:    memory access error for empty string when 'encoding' is a single
            byte encoding.
Solution:   Check for empty string when getting the length. (Dominique Pelle,
            closes #5021, closes #5007)

patch 8.1.2118: termcodes test fails when $TERM is "dumb"

Problem:    Termcodes test fails when $TERM is "dumb".
Solution:   Skip the test. (James McCoy, closes #5019)

patch 8.1.2117: CursorLine highlight used while 'cursorline' is off

Problem:    CursorLine highlight used while 'cursorline' is off.
Solution:   Check 'cursorline' is set. (cloes #5017)

patch 8.1.2116: no check for out of memory

Problem:    No check for out of memory.
Solution:   Check for NULL pointer.

patch 8.1.2115: MS-Windows: shell commands fail if &shell contains a space

Problem:    MS-Windows: shell commands fail if &shell contains a space.
Solution:   Use quotes instead of escaping. (closes #4920)

patch 8.1.2114: when a popup is closed with CTRL-C the callback aborts

Problem:    When a popup is closed with CTRL-C the callback aborts.
Solution:   Reset got_int when invoking the callback. (closes #5008)

patch 8.1.2113: ":help expr-!~?" only works after searching

Problem:    ":help expr-!~?" only works after searching.
Solution:   Escape "~" after "expr-". (closes #5015)

patch 8.1.2112: build number for ConPTY is outdated

Problem:    Build number for ConPTY is outdated.
Solution:   Update to new build number. (Nobuhiro Takasaki, closes #5014)

patch 8.1.2111: viminfo file not sufficiently tested

Problem:    Viminfo file not sufficiently tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5009)

patch 8.1.2110: CTRL-C closes two popups instead of one

Problem:    CTRL-C closes two popups instead of one.
Solution:   Reset got_int when the filter consumed the key.

patch 8.1.2109: popup_getoptions() hangs with tab-local popup

Problem:    popup_getoptions() hangs with tab-local popup.
Solution:   Correct pointer name. (Marko Mahnič, closes #5006)

patch 8.1.2108: cannot close the cmdline window from CmdWinEnter

Problem:    Cannot close the cmdline window from CmdWinEnter. (George Brown)
Solution:   Reset cmdwin_result earlier. (Christian Brabandt, closes #4980)

patch 8.1.2107: various memory leaks reported by asan

Problem:    Various memory leaks reported by asan.
Solution:   Free the memory. (Ozaki Kiichi, closes #5003)

patch 8.1.2106: no tests for dragging the mouse beyond the window

Problem:    No tests for dragging the mouse beyond the window.
Solution:   Add a test. (Dominique Pelle, closes #5004)

patch 8.1.2105: MS-Windows: system() may crash

Problem:    MS-Windows: system() may crash.
Solution:   Do not use "itmp" when it is NULL. (Yasuhiro Matsumoto,
            closes #5005)

patch 8.1.2104: the normal.c file is too big

Problem:    The normal.c file is too big.
Solution:   Move do_pending_operator() to ops.c. (Yegappan Lakshmanan,
            closes #4999).

patch 8.1.2103: wrong error message if "termdebugger" is not executable

Problem:    wrong error message if "termdebugger" is not executable.
Solution:   Check if "termdebugger" is executable and give a clear error
            message. (Ozaki Kiichi, closes #5000)  Fix indents.

patch 8.1.2102: can't build with GTK and FEAT_GUI_GNOME

Problem:    Can't build with GTK and FEAT_GUI_GNOME. (Tony Mechelynck)
Solution:   Adjust the #ifdef. (Yegappan Lakshmanan)

patch 8.1.2101: write_session_file() often defined but not used

Problem:    write_session_file() often defined but not used.
Solution:   Adjust the #ifdef. (Yegappan Lakshmanan, closes #4998)

patch 8.1.2100: :mksession is not sufficiently tested

Problem:    :mksession is not sufficiently tested.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #4993)

patch 8.1.2099: state() test fails on some Mac systems

Problem:    state() test fails on some Mac systems.
Solution:   Increase the wait time. (closes #4983)

patch 8.1.2098: mksession test fails on MS-Windows

Problem:    mksession test fails on MS-Windows.
Solution:   Skip testing with backslashes on MS-Windows.

patch 8.1.2097: :mksession is not sufficiently tested

Problem:    :mksession is not sufficiently tested.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #4992)

patch 8.1.2096: too many #ifdefs

Problem:    Too many #ifdefs.
Solution:   Graduate FEAT_COMMENTS.

patch 8.1.2095: leaking memory when getting item from dict

Problem:    Leaking memory when getting item from dict.
Solution:   Also free the key when not evaluating.

patch 8.1.2094: the fileio.c file is too big

Problem:    The fileio.c file is too big.
Solution:   Move buf_write() to bufwrite.c. (Yegappan Lakshmanan,
            closes #4990)

patch 8.1.2093: MS-Windows: system() test fails

Problem:    MS-Windows: system() test fails.
Solution:   Expect CR when using systemlist().

patch 8.1.2092: MS-Windows: redirect in system() does not work

Problem:    MS-Windows: redirect in system() does not work.
Solution:   Handle 'shellxescape' and 'shellxquote' better. (Yasuhiro
            Matsumoto, closes #2054)

patch 8.1.2091: double free when memory allocation fails

Problem:    Double free when memory allocation fails. (Zu-Ming Jiang)
Solution:   Use VIM_CLEAR() instead of vim_free(). (closes #4991)

patch 8.1.2090: not clear why channel log file ends

Problem:    Not clear why channel log file ends.
Solution:   Add a "closing" line.

patch 8.1.2089: do not get a hint that $TEST_FILTER was active

Problem:    Do not get a hint that $TEST_FILTER was active.
Solution:   Mention $TEST_FILTER if no functions were executed.

patch 8.1.2088: renamed libvterm mouse.c file not in distributed file list

Problem:    Renamed libvterm mouse.c file not in distributed file list.
Solution:   Rename the file in the file list.

Update runtime files.

patch 8.1.2087: cannot easily select one test function to execute

Problem:    Cannot easily select one test function to execute.
Solution:   Support the $TEST_FILTER environment variable. (Ozaki Kiichi,
            closes #2695)

patch 8.1.2086: missing a few changes for the renamed files

Problem:    Missing a few changes for the renamed files.
Solution:   Rename in a few more places. (Ken Takata)

patch 8.1.2085: MS-Windows: draw error moving cursor over double-cell char

Problem:    MS-Windows: draw error moving cursor over double-cell character.
Solution:   Move the cursor to the left edge if needed. (Nobuhiro Takasaki,
            closes #4986)

patch 8.1.2084: Amiga: cannot get the user name

Problem:    Amiga: cannot get the user name.
Solution:   Use getpwuid() if available. (Ola Söder, closes #4985)

patch 8.1.2083: multi-byte chars do not work properly with "%.*S" in printf()

Problem:    Multi-byte chars do not work properly with "%.*S" in printf().
Solution:   Use mb_ptr2cells(). Daniel Hahler, closes #4989)

patch 8.1.2082: some files have a weird name to fit in 8.3 characters

Problem:    Some files have a weird name to fit in 8.3 characters.
Solution:   Use a nicer names.

patch 8.1.2081: the spell.c file is too big

Problem:    The spell.c file is too big.
Solution:   Move the code for spell suggestions to a separate file. (Yegappan
            Lakshmanan, closes #4988)

patch 8.1.2080: the terminal API is limited and can't be disabled

Problem:    The terminal API is limited and can't be disabled.
Solution:   Add term_setapi() to set the function prefix. (Ozaki Kiichi,
            closes #2907)

patch 8.1.2079: popup window test fails without +terminal

Problem:    Popup window test fails without +terminal.
Solution:   Check for the +terminal feature.

patch 8.1.2078: build error with +textprop but without +terminal

Problem:    Build error with +textprop but without +terminal. (Tony Mechelynck)
Solution:   Add #ifdef.

patch 8.1.2077: the ops.c file is too big

Problem:    The ops.c file is too big.
Solution:   Move code for dealing with registers to a new file. (Yegappan
            Lakshmanan, closes #4982)

patch 8.1.2076: crash when trying to put a terminal in a popup window

Problem:    Crash when trying to put a terminal buffer in a popup window.
Solution:   Check for NULL buffer.  Do not allow putting a terminal in a popup
            window.

patch 8.1.2075: get many log messages when waiting for a typed character

Problem:    Get many log messages when waiting for a typed character.
Solution:   Do not repeat the repeated messages when nothing happens.

patch 8.1.2074: test for SafeState autocommand is a bit flaky

Problem:    Test for SafeState autocommand is a bit flaky.
Solution:   Add to list of flaky tests.

patch 8.1.2073: when editing a buffer 'colorcolumn' may not work

Problem:    When editing a buffer 'colorcolumn' may not work.
Solution:   Set the buffer before copying option values. Call
            check_colorcolumn() after copying window options.