patch 8.1.2350: other text for CTRL-V in Insert mode with modifyOtherKeys

Problem:    Other text for CTRL-V in Insert mode with modifyOtherKeys.
Solution:   Convert the Escape sequence back to key as if modifyOtherKeys is
            not set, and use CTRL-SHIFT-V to get the Escape sequence itself.
            (closes #5254)

patch 8.1.2349: :lockvar and :unlockvar cannot be followed by "| endif"

Problem:    :lockvar and :unlockvar cannot be followed by "| endif".
Solution:   Check for following commands. (closes #5269)

patch 8.1.2348: :const cannot be followed by "| endif"

Problem:    :const cannot be followed by "| endif".
Solution:   Check following command for :const. (closes #5269)
            Also fix completion after :const.

patch 8.1.2347: MacOS: build fails

Problem:    MacOS: build fails.
Solution:   Don't define _XOPEN_SOURCE for Mac.

patch 8.1.2346: CTRL-R CTRL-R doesn't work with modifyOtherKeys

Problem:    CTRL-R CTRL-R doesn't work with modifyOtherKeys.
Solution:   Allow key codes when fetching argument for CTRL-R. (closes #5266)
            Also fix CTRL-G in Insert mode.

patch 8.1.2345: .cjs files are not recognized as Javascript

Problem:    .cjs files are not recognized as Javascript.
Solution:   Add the *.cjs pattern. (closes #5268)

patch 8.1.2344: Cygwin: warning for using strptime()

Problem:    Cygwin: warning for using strptime().
Solution:   Move defining _XOPEN_SOURCE and __USE_XOPEN to vim.h. (Ken Takata,
            closes #5265)  Use 700 for _XOPEN_SOURCE for mkdtemp().

patch 8.1.2343: using time() for srand() is not very random

Problem:    Using time() for srand() is not very random.
Solution:   use /dev/urandom if available

patch 8.1.2342: random number generator in Vim script is slow

Problem:    Random number generator in Vim script is slow.
Solution:   Add rand() and srand(). (Yasuhiro Matsumoto, closes #1277)

patch 8.1.2341: not so easy to interrupt a script programatically

Problem:    Not so easy to interrupt a script programatically.
Solution:   Add the interrupt() function. (Yasuhiro Matsumoto, closes #2834)

patch 8.1.2340: quickfix test fails under valgrind and asan

Problem:    Quickfix test fails under valgrind and asan.
Solution:   Make sure long line does not overflow IObuff. (Dominique Pelle,
            closes #5263)  Put back fix for large terminals. (Yegappan
            Lakshmanan, closes #5264)

patch 8.1.2339: insufficient testing for quickfix

Problem:    Insufficient testing for quickfix.
Solution:   Add a few more tests. (Yegappan Lakshmanan, closes #5261)

patch 8.1.2338: using Visual mark sith :s gives E20 if not set

Problem:    Using Visual mark sith :s gives E20 if not set.
Solution:   Ignore errors when handling 'incsearch'. (closes #3837)

patch 8.1.2337: double-click time sometimes miscomputed

Problem:    Double-click time sometimes miscomputed.
Solution:   Correct time computation. (Dominique Pelle, closes #5259)

patch 8.1.2336: when an expr mapping moves the cursor it is not restored

Problem:    When an expr mapping moves the cursor it is not restored.
Solution:   Position the cursor after an expr mapping. (closes #5256)

patch 8.1.2335: error message for function arguments may use NULL pointer

Problem:    Error message for function arguments may use NULL pointer.
            (Coverity)
Solution:   Use the original function name.

patch 8.1.2334: possible NULL pointer dereference in popup_locate()

Problem:    Possible NULL pointer dereference in popup_locate(). (Coverity)
Solution:   Check for NULL pointer.

patch 8.1.2333: with modifyOtherKeys CTRL-^ doesn't work

Problem:    With modifyOtherKeys CTRL-^ doesn't work.
Solution:   Handle the exception.

patch 8.1.2332: missing file in refactoring

Problem:    Missing file in refactoring.
Solution:   Update missing file.

patch 8.1.2331: the option.c file is still very big

Problem:    The option.c file is still very big.
Solution:   Move a few functions to where they fit better. (Yegappan
            Lakshmanan, closes #4895)

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.