patch 8.1.2383: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2382: MS-Windows: When using VTP bold+inverse doesn't work

Problem:    MS-Windows: When using VTP bold+inverse doesn't work.
Solution:   Compare with the default colors. (Nobuhiro Takasaki, closes #5303)

patch 8.1.2381: not all register related code is covered by tests

Problem:    Not all register related code is covered by tests.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #5301)

patch 8.1.2380: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2379: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2378: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2377: GUI: when losing focus a pending operator is executed

Problem:    GUI: when losing focus a pending operator is executed.
Solution:   Do not execute an operator when getting K_IGNORE. (closes #5300)

patch 8.1.2376: preprocessor indents are incorrect

Problem:    Preprocessor indents are incorrect.
Solution:   Fix the indents. (Ken Takata, closes #5298)

patch 8.1.2375: no suffucient testing for registers

Problem:    No suffucient testing for registers.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #5296)
            Fix that "p" on last virtual column of tab inserts spaces.

patch 8.1.2374: unused parts of libvterm are included

Problem:    Unused parts of libvterm are included.
Solution:   Delete the unused files.

patch 8.1.2373: cannot build with +popupwin but without +quickfix

Problem:    Cannot build with +popupwin but without +quickfix. (John Marriott)
Solution:   Adjust #ifdefs.

patch 8.1.2372: VMS: failing realloc leaks memory

Problem:    VMS: failing realloc leaks memory. (Chakshu Gupta)
Solution:   Free the memory. (partly fixes #5292)

patch 8.1.2371: FEAT_TEXT_PROP is a confusing name

Problem:    FEAT_TEXT_PROP is a confusing name.
Solution:   Use FEAT_PROP_POPUP. (Naruhiko Nishino, closes #5291)

patch 8.1.2370: build problems on VMS

Problem:    Build problems on VMS.
Solution:   Adjust the build file. (Zoltan Arpadffy)

patch 8.1.2369: cannot build with quickfix and without text properties

Problem:    Cannot build with quickfix and without text properties.
Solution:   Fix typo. (Naruhiko Nishino)

patch 8.1.2368: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2367: registers are not sufficiently tested

Problem:    Registers are not sufficiently tested.
Solution:   Add a few more test cases. (Yegappan Lakshmanan, closes #5288)

patch 8.1.2366: using old C style comments

Problem:    Using old C style comments.
Solution:   Use // comments where appropriate.

patch 8.1.2365: missing tests for recent popupwin changes

Problem:    Missing tests for recent popupwin changes.
Solution:   Add test cases.

Update runtime files.

patch 8.1.2364: termwinscroll test is flaky on FreeBSD

Problem:    Termwinscroll test is flaky on FreeBSD.
Solution:   Add to list of flaky tests.  Rename function.

patch 8.1.2363: ml_get error when accessing Visual area in 'statusline'

Problem:    ml_get error when accessing Visual area in 'statusline'.
Solution:   Disable Visual mode when using another window. (closes #5278)

patch 8.1.2362: cannot place signs in a popup window

Problem:    Cannot place signs in a popup window. (Maxim Kim)
Solution:   Use the group prefix "PopUp" to specify which signs should show up
            in a popup window. (closes #5277)

patch 8.1.2361: MS-Windows: test failures related to VIMDLL

Problem:    MS-Windows: test failures related to VIMDLL.
Solution:   Adjust code and tests. (Ken Takata, closes #5283)

patch 8.1.2360: quickfix test coverage can still be improved

Problem:    Quickfix test coverage can still be improved.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #5276)

patch 8.1.2359: cannot build without FEAT_FLOAT

Problem:    Cannot build without FEAT_FLOAT. (John Marriott)
Solution:   Fix #ifdefs around f_srand().

patch 8.1.2358: tests fail on Cirrus CI for FreeBSD

Problem:    Tests fail on Cirrus CI for FreeBSD.
Solution:   Fix a test and skip some. (Christian Brabandt, closes #5281)

patch 8.1.2357: no test with wrong argument for rand()

Problem:    No test with wrong argument for rand().
Solution:   Add a test case.

patch 8.1.2356: rand() does not use the best algorithm

Problem:    rand() does not use the best algorithm.
Solution:   use xoshiro128** instead of xorshift. (Kaito Udagawa,
            closes #5279)

patch 8.1.2355: test with "man" fails on FreeBSD

Problem:    Test with "man" fails on FreeBSD.
Solution:   Use "-P" instead of "--pager".

patch 8.1.2354: Cirrus CI runs on another repository

Problem:    Cirrus CI runs on another repository.
Solution:   Run Cirrus CI on vim/vim.

patch 8.1.2353: build failure on FreeBSD

Problem:    Build failure on FreeBSD.
Solution:   Change #ifdef to only check for Linux-like systems.

Merge branch 'master' of github.com:vim/vim

Merge pull request #5275 from vim/revert-5273-cirrus_ci

Revert "Cirrus CI: add a simple CI using BSD"

Revert "Cirrus CI: add a simple CI using BSD"

patch 8.1.2352: CI doesn't cover FreeBSD

Problem:    CI doesn't cover FreeBSD.
Solution:   Configure Cirrus-CI. (Christian Brabandt, closes #5273)

Merge pull request #5273 from chrisbra/cirrus_ci

Cirrus CI: add a simple CI using BSD

Cirrus CI: add a simple CI using BSD

Cirrus CI allows to run CI tests in a wide variety of systems, such as
Mac, Windows and FreeBSD. For a starter, add a Cirrus-CI test just for
FreeBSD using version 12, assuming Windows and Linux are already tested
using appveyor and Travis CI

patch 8.1.2351: 'wincolor' not used for > for not fitting double width char

Problem:    'wincolor' not used for > for not fitting double width char.
            Also: popup drawn on right half of double width character looks
            wrong.
Solution:   Adjust color for > character.  Clear left half of double width
            character if right half is being overwritten.

Update runtime files

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.