patch 8.1.2330: vi' does not always work when 'selection' is exclusive

Problem:    vi' does not always work when 'selection' is exclusive.
Solution:   Adjust start position.

patch 8.1.2329: mouse multiple click test is a bit flaky

Problem:    Mouse multiple click test is a bit flaky.
Solution:   Add it to the list of flaky tests.

patch 8.1.2328: a few hangul input pieces remain

Problem:    A few hangul input pieces remain.
Solution:   Update VMS makefile.

patch 8.1.2327: cannot build with Hangul input

Problem:    Cannot build with Hangul input.
Solution:   Remove Hangul input support.

patch 8.1.2326: cannot parse a date/time string

Problem:    Cannot parse a date/time string.
Solution:   Add strptime(). (Stephen Wall, closes #)

patch 8.1.2325: crash when using balloon with empty line

Problem:    Crash when using balloon with empty line.
Solution:   Handle empty lines. (Markus Braun)

patch 8.1.2324: with of scrollbar in popup menu not taken into account

Problem:    With of scrollbar in popup menu not taken into account.
Solution:   Add the width of the scrollbar.

Update runtime files.

patch 8.1.2323: Old MSVC version no longer tested.

Problem:    Old MSVC version no longer tested.
Solution:   Drop support for MSCV 2008 and older. (Ken Takata, closes #5248)

patch 8.1.2322: quickfix test fails in very big terminal

Problem:    Quickfix test fails in very big terminal.
Solution:   Adjust the expected result for the width. (Masato Nishihata,
            closes #5244)

patch 8.1.2321: cannot select all text with the mouse

Problem:    Cannot select all text with the mouse. (John Marriott)
Solution:   Move limiting the mouse column to f_getmousepos(). (closes #5242)

patch 8.1.2320: insufficient test coverage for quickfix

Problem:    Insufficient test coverage for quickfix.
Solution:   Add more tests.  Fix uncovered problem. (Yegappan Lakshmanan,
            closes #5238)

patch 8.1.2319: compiler warning for int size

Problem:    Compiler warning for int size.
Solution:   Add typecast. (Mike Williams)

patch 8.1.2318: MS-Windows GUI: main background shows in toolbar

Problem:    MS-Windows GUI: main background shows in toolbar.
Solution:   Remove transparency from the toolbar. (Simon Sadler)

patch 8.1.2317: no test for spell affix file with flag on suffix

Problem:    No test for spell affix file with flag on suffix.
Solution:   Add a test case.

patch 8.1.2316: FORTIFY_SOURCE can also be present in CPPFLAGS

Problem:    FORTIFY_SOURCE can also be present in CPPFLAGS.
Solution:   Remove it in configure. (Benedikt Morbach, closes #2786)

patch 8.1.2315: not always using the right window when jumping to an error

Problem:    Not always using the right window when jumping to an error.
Solution:   Add the "uselast" flag in 'switchbuf'. (closes #1652)

patch 8.1.2314: vi' sometimes does not select anything

Problem:    vi' sometimes does not select anything.
Solution:   Recognize an empty selection. (Christian Brabandt, closes #5183)

patch 8.1.2313: debugging where a delay comes from is not easy

Problem:    Debugging where a delay comes from is not easy.
Solution:   Use different values when calling ui_delay().

patch 8.1.2312: "line:" field in tags file not used

Problem:    "line:" field in tags file not used.
Solution:   Recognize the field and use the value. (Andy Massimino, Daniel
            Hahler, closes #5232, closes #2546, closes #1057)

Update runtime files.

patch 8.1.2311: warning for missing function prototype

Problem:    Warning for missing function prototype.
Solution:   Add the proto. (Dominique Pelle, closes #5233)

patch 8.1.2310: no proper test for directory changes in quickfix

Problem:    No proper test for directory changes in quickfix.
Solution:   Add a test that uses multiple directories. (Yegappan Lakshmanan,
            closes #5230)

patch 8.1.2309: compiler warning for argument type

Problem:    Compiler warning for argument type.
Solution:   Use linenr_T and cast to varnumber_T. (John Marriott)

patch 8.1.2308: deleting text before zero-width textprop removes it

Problem:    Deleting text before zero-width textprop removes it.
Solution:   Keep zero-width textprop when deleting text.

patch 8.1.2307: positioning popup doesn't work for buffer-local textprop

Problem:    Positioning popup doesn't work for buffer-local textprop.
Solution:   Make it work. (closes #5225)

patch 8.1.2306: double and triple clicks are not tested

Problem:    Double and triple clicks are not tested.
Solution:   Test mouse clicks to select text. (closes #5226)

patch 8.1.2305: no warning for wrong entry in translations

Problem:    No warning for wrong entry in translations.
Solution:   Check semicolons in keywords entry of desktop file.

patch 8.1.2304: cannot get the mouse position when getting a mouse click

Problem:    Cannot get the mouse position when getting a mouse click.
Solution:   Add getmousepos().

patch 8.1.2303: cursor in wrong position after horizontal scroll

Problem:    Cursor in wrong position after horizontal scroll.
Solution:   Set w_valid_leftcol.  (closes #5214, closes #5224)

patch 8.1.2302: :lockmarks does not work for '[ and ']

Problem:    :lockmarks does not work for '[ and '].
Solution:   save and restore '[ and '] marks. (James McCoy, closes #5222)

patch 8.1.2301: MS-Windows GUI: drawing error when background color changes

Problem:    MS-Windows GUI: drawing error when background color changes.
Solution:   Implement gui_mch_new_colors(). (Simon Sadler)

Fix vim.desktop generation.

Update runtime files

patch 8.1.2300: redraw breaks going through list of popup windows

Problem:    Redraw breaks going through list of popup windows.
Solution:   Use different flags for popup_reset_handled(). (closes #5216)

patch 8.1.2299: ConPTY in MS-Windows 1909 is still wrong

Problem:    ConPTY in MS-Windows 1909 is still wrong.
Solution:   Use same solution as for 1903. (Nobuhiro Takasaki, closes #5217)

patch 8.1.2298: missing part of 8.1.2296

Problem:    Missing part of 8.1.2296.
Solution:   s/test/text/

patch 8.1.2297: the ex_vimgrep() function is too long

Problem:    The ex_vimgrep() function is too long.
Solution:   Split it in three parts. (Yegappan Lakshmanan, closes #5211)

patch 8.1.2296: text properties are not combined with syntax by default

Problem:    Text properties are not combined with syntax by default.
Solution:   Make it work as documented. (closes #5190)

patch 8.1.2295: if buffer of popup is in another window cursorline sign shows

Problem:    If buffer of popup is in another window cursorline sign shows.
Solution:   Check the group of the sign.

patch 8.1.2294: cursor pos wrong with concealing and search causes a scroll

Problem:    Cursor position wrong when characters are concealed and asearch
            causes a scroll.
Solution:   Fix the cursor column in a concealed line after window scroll.
            (closes #5215, closes #5012)

patch 8.1.2293: join adds trailing space when second line is empty

Problem:    Join adds trailing space when second line is empty. (Brennan
            Vincent)
Solution:   Do not add a trailing space.

patch 8.1.2292: v:mouse_winid not set on click in popup window

Problem:    v:mouse_winid not set on click in popup window.
Solution:   Set v:mouse_winid. (closes #5171)

patch 8.1.2291: memory leak when executing command in a terminal

Problem:    Memory leak when executing command in a terminal.
Solution:   Free "argv". (Dominique Pelle, closes #5208)

Update runtime files

patch 8.1.2290: autocommand test fails

Problem:    Autocommand test fails.
Solution:   Remove 'closeoff' from 'diffopt'.

patch 8.1.2289: after :diffsplit closing the window does not disable diff

Problem:    After :diffsplit closing the window does not disable diff.
Solution:   Add "closeoff" to 'diffopt' and add it to the default.

patch 8.1.2288: not using all space when popup with "topleft" flips to above

Problem:    Not using all space when popup with "topleft" flips to above.
Solution:   Recompute the height when a popup flips from below to above.
            (closes #5151)

patch 8.1.2287: using EndOfBuffer highlight in popup does not look good

Problem:    Using EndOfBuffer highlight in popup does not look good.
Solution:   Do not EndOfBuffer highlight. (closes #5204)

patch 8.1.2286: using border highlight in popup window leaks memory

Problem:    Using border highlight in popup window leaks memory.
Solution:   Free memory before overwriting. (Dominique Pelle, closes #5203)

patch 8.1.2285: padding in structures wastes memory

Problem:    Padding in structures wastes memory.
Solution:   Move fields to avoid padding. (Dominique Pelle, closes #5202)

patch 8.1.2284: compiler warning for unused variable

Problem:    Compiler warning for unused variable. (Tony Mechelynck)
Solution:   Add #ifdef.

patch 8.1.2283: missed on use of p_sbr

Problem:    Missed on use of p_sbr.
Solution:   Add missing p_sbr change.

patch 8.1.2282: crash when passing many arguments through a partial

Problem:    Crash when passing many arguments through a partial. (Andy
            Massimino)
Solution:   Check the number of arguments. (closes #5186)

patch 8.1.2281: 'showbreak' cannot be set for one window

Problem:    'showbreak' cannot be set for one window.
Solution:   Make 'showbreak' global-local.

patch 8.1.2280: crash when passing partial to substitute()

Problem:    Crash when passing partial to substitute().
Solution:   Take extra arguments into account. (closes #5186)

patch 8.1.2279: computation of highlight attributes is too complicated

Problem:    Computation of highlight attributes is too complicated.
Solution:   Simplify the attribute computation and make it more consistent.
            (closes #5190)  Fix that 'combine' set to zero doesn't work.

patch 8.1.2278: using "cd" with "exe" may fail

Problem:    Using "cd" with "exe" may fail.
Solution:   Use chdir() instead.

patch 8.1.2277: terminal window is not updated when info popup changes

Problem:    Terminal window is not updated when info popup changes.
Solution:   Redraw windows when re-using an info popup. (closes #5192)

patch 8.1.2276: MS-Windows: session test leaves files behind

Problem:    MS-Windows: session test leaves files behind.
Solution:   Wipe out buffers before deleting the directory. (closes #5187)

patch 8.1.2275: using "seesion" looks like a mistake

Problem:    Using "seesion" looks like a mistake.
Solution:   Use an underscore to make the function sort first.

patch 8.1.2274: newlines in 'balloonexpr' result only work in the GUI

Problem:    Newlines in 'balloonexpr' result only work in the GUI.
Solution:   Also recognize newlines in the terminal. (closes #5193)

patch 8.1.2273: wrong default when "pos" is changed with popup_atcursor()

Problem:    Wrong default when "pos" is changed with popup_atcursor().
Solution:   Adjust the default line and col when "pos" is not the default
            value. (#5151)

patch 8.1.2272: test may hang at more prompt

Problem:    Test may hang at more prompt.
Solution:   Reset 'more' after resetting 'compatible'. (Michael Soyka)

patch 8.1.2271: build error if FEAT_TAG_BINS is not defined

Problem:    Build error if FEAT_TAG_BINS is not defined. (John Marriott)
Solution:   Add #ifdef.

patch 8.1.2270: "gf" is not tested in Visual mode

Problem:    "gf" is not tested in Visual mode.
Solution:   Add Visual mode test and test errors. (Dominique Pelle,
            closes #5197)

patch 8.1.2269: tags file with very long line stops using binary search

Problem:    Tags file with very long line stops using binary search.
Solution:   Reallocate the buffer if needed.

patch 8.1.2268: spell file flag zero is not recognized

Problem:    Spell file flag zero is not recognized.
Solution:   Use -1 as an error value, so that zero can be used as a valid flag
            number.

patch 8.1.2267: compiler warning for uninitialized variable

Problem:    Compiler warning for uninitialized variable. (Tony Mechelynck)
Solution:   Rearrange the code.

patch 8.1.2266: position unknown for a mouse click in a popup window

Problem:    Position unknown for a mouse click in a popup window.
Solution:   Set v:mouse_col and v:mouse_lnum. (closes #5171)

patch 8.1.2265: when popup with "botleft" does not fit it flips incorrectly

Problem:    When popup with "botleft" does not fit it flips incorrectly.
Solution:   Only flip when there is more space on the other side.  Add the
            "posinvert" option to disable flipping and do it in both
            directions if enabled.  (closes #5151)

patch 8.1.2264: there are two test files for :let

Problem:    There are two test files for :let.
Solution:   Merge the two files.

patch 8.1.2263: 'noesckeys' test fails in GUI

Problem:    'noesckeys' test fails in GUI.
Solution:   Skip the test in the GUI.

patch 8.1.2262: unpack assignment in function not recognized

Problem:    Unpack assignment in function not recognized.
Solution:   Skip over "[a, b]". (closes #5051)

patch 8.1.2261: with modifyOtherKeys set 'noesckeys' doesn't work

Problem:    With modifyOtherKeys set 'noesckeys' doesn't work. (James McCoy)
Solution:   Disable modifyOtherKeys while in Insert mode when 'noesckeys' is
            set. (closes #5180)

patch 8.1.2260: terminal test may fail on MS-Windows

Problem:    Terminal test may fail on MS-Windows.
Solution:   Catch the situation that "term dir" fails with a CreateProcess
            error.

patch 8.1.2259: running tests may leave XfakeHOME behind

Problem:    Running tests may leave XfakeHOME behind.
Solution:   Source summarize.vim without using setup.vim. (closes #5177)
            Also fix that on MS-Windows the test log isn't echoed.

patch 8.1.2258: may get hit-enter prompt after entering a number

Problem:    May get hit-enter prompt after entering a number. (Malcolm Rowe)
Solution:   Put back accidentally deleted lines. (closes #5176)

patch 8.1.2257: MS-Windows GUI: scroll wheel always uses current window

Problem:    MS-Windows GUI: scroll wheel always uses current window.
Solution:   Add the 'scrollfocus' option for MS-Windows.

patch 8.1.2256: test for ":term ++shell" fails on MS-Windows

Problem:    Test for ":term ++shell" fails on MS-Windows.
Solution:   Accept failure of "dir" executable.

patch 8.1.2255: ":term ++shell" does not work on MS-Windows

Problem:    ":term ++shell" does not work on MS-Windows.
Solution:   Add MS-Windows support.

patch 8.1.2254: MS-Windows: mouse scroll wheel doesn't work in popup

Problem:    MS-Windows: mouse scroll wheel doesn't work in popup.
Solution:   Handle mouse wheel events separately. (closes #5138)

patch 8.1.2253: using "which" to check for an executable is not reliable

Problem:    Using "which" to check for an executable is not reliable.
Solution:   Use "command -v" instead.  Also exit with error code when
            generating tags has an error. (closes #5174)

patch 8.1.2252: compiler warning for int size

Problem:    Compiler warning for int size.
Solution:   Add type cast. (Mike Williams)

patch 8.1.2251: ":term command" may not work without a shell

Problem:    ":term command" may not work without a shell.
Solution:   Add the ++shell option to :term. (closes #3340)

patch 8.1.2250: CTRL-U and CTRL-D don't work in popup window

Problem:    CTRL-U and CTRL-D don't work in popup window.
Solution:   Initialize 'scroll'.  Add "lastline" in popup_getpos().
            (closes #5170)

patch 8.1.2249: "make vimtags" does not print any message

Problem:    "make vimtags" does not print any message.
Solution:   Add a message that the tags have been updated.

patch 8.1.2248: CTRL-W dot does not work when modifyOtherKeys is enabled

Problem:    CTRL-W dot does not work in a terminal when modifyOtherKeys is
            enabled.
Solution:   Use the modifier when needed.  Pass the modifier along with the
            key to avoid mistakes.

patch 8.1.2247: "make vimtags" does not work in runtime/doc

Problem:    "make vimtags" does not work in runtime/doc.
Solution:   Test existence with "which" instead of "test -x". (Ken Takata)

patch 8.1.2246: some tests are still in old style

Problem:    Some tests are still in old style.
Solution:   Change a few tests to new style. (Yegappan Lakshmanan)

patch 8.1.2245: third character of 'listchars' tab shows in wrong place

Problem:    Third character of 'listchars' tab shows in wrong place when
            'breakindent' is set.
Solution:   Set c_final to NUL. (Naruhiko Nishino, closes #5165)

patch 8.1.2244: 'wrapscan' is not used for "gn"

Problem:    'wrapscan' is not used for "gn".
Solution:   Only reset 'wrapscan' for the first search round. (closes #5164)

patch 8.1.2243: typos in comments

Problem:    Typos in comments.
Solution:   Fix the typos. (Dominique Pelle, closes #5160)  Also adjust
            formatting a bit.

patch 8.1.2242: creating docs tags uses user preferences

Problem:    Creating docs tags uses user preferences. (Tony Mechelynck)
Solution:   Add "--clean".

patch 8.1.2241: match highlight does not combine with 'wincolor'

Problem:    Match highlight does not combine with 'wincolor'.
Solution:   Apply 'wincolor' last on top of any other attribute. (closes #5159)

patch 8.1.2240: popup window width changes when scrolling

Problem:    Popup window width changes when scrolling.
Solution:   Also adjust maxwidth when applying minwidth and there is a
            scrollbar.  Fix off-by-one error. (closes #5162)

patch 8.1.2239: CI fails when running tests without building Vim

Problem:    CI fails when running tests without building Vim.
Solution:   Skip creating doc tags if the execute does not exist.

Update runtime files.

patch 8.1.2238: error in docs tags goes unnoticed

Problem:    Error in docs tags goes unnoticed.
Solution:   Adjust tags build command. (Ken Takata, closes #5158)

patch 8.1.2237: mode() result depends on whether CURSOR_SHAPE is defined

Problem:    Mode() result after usign "r" depends on whether CURSOR_SHAPE is
            defined. (Christian Brabandt)
Solution:   Move the #ifdef to only skip ui_cursor_shape().