patch 8.1.0724: build for MinGW fails

Problem:    Build for MinGW fails.
Solution:   Avoid specifying dependencies in included makefile.

patch 8.1.0723: cannot easily run specific test when in src/testdir

Problem:    Cannot run specific test when in src/testdir the same was as in
            the src directory.
Solution:   Move build rule to src/testdir/Makefile.

patch 8.1.0722: cannot build without the virtualedit feature

Problem:    Cannot build without the virtualedit feature.
Solution:   Make getviscol2() always available.

patch 8.1.0721: conceal mode is not sufficiently tested

Problem:    Conceal mode is not sufficiently tested.
Solution:   Add screendump tests.  Check all 'concealcursor' values.

patch 8.1.0720: cannot easily change the current quickfx list index

Problem:    Cannot easily change the current quickfx list index.
Solution:   Add the "idx" argument to setqflist(). (Yegappan Lakshmanan,
            closes #3701)

patch 8.1.0719: too many #ifdefs

Problem:    Too many #ifdefs.
Solution:   Always build with the +visualextra feature.

patch 8.1.0718: a couple compiler warnings

Problem:    A couple compiler warnings.
Solution:   Rename shadowed variables.  Add UNUSED.

patch 8.1.0717: there is no function for the ":sign jump" command

Problem:    There is no function for the ":sign jump" command.
Solution:   Add the sign_jump() function. (Yegappan Lakshmanan, closes #3780)

patch 8.1.0716: get warning message when 'completefunc' returns nothing

Problem:    Get warning message when 'completefunc' returns nothing.
Solution:   Allow for returning v:none to suppress the warning message.
            (Yasuhiro Matsumoto, closes #3789)

patch 8.1.0715: superfluous call to redraw_win_later()

Problem:    Superfluous call to redraw_win_later().
Solution:   Remove the call.

patch 8.1.0714: unessesary #if lines in GTK code

Problem:    Unessesary #if lines in GTK code.
Solution:   Remove the #if. (Ken Takata, closes #3785)

patch 8.1.0713: images for NSIS take up too much space

Problem:    Images for NSIS take up too much space.
Solution:   Put the images in a zip file.

patch 8.1.0712: MS-Windows build instructions are a bit outdated

Problem:    MS-Windows build instructions are a bit outdated.
Solution:   Update the instructions. (Ken Takata)

patch 8.1.0711: test files still use function!

Problem:    Test files still use function!.
Solution:   Remove the exclamation mark.  Fix overwriting a function.

patch 8.1.0710: when using timers may wait for job exit quite long

Problem:    When using timers may wait for job exit quite long.
Solution:   Return from ui_wait_for_chars_or_timer() when a job or channel
            needs to be handled. (Ozaki Kiichi, closes #3783)

patch 8.1.0709: windows are updated for every added/deleted sign

Problem:    Windows are updated for every added/deleted sign.
Solution:   Do not call update_debug_sign().  Only redraw when the line with
            the sign is visible.  (idea from neovim #9479)

patch 8.1.0708: third argument for redrawWinline() is always FALSE

Problem:    Third argument for redrawWinline() is always FALSE.
Solution:   Drop the argument. (neovim #9479)

patch 8.1.0707: text property columns are not adjusted for changed indent

Problem:    Text property columns are not adjusted for changed indent.
Solution:   Adjust text properties.

patch 8.1.0706: tabline is not always redrawn

Problem:    Tabline is not always redrawn when something that is used in
            'tabline' changes.
Solution:   Add ":redrawtabline" so that a plugin can at least cause the
            redraw when needed.

patch 8.1.0705: :colorscheme isn't tested enough

Problem:    :colorscheme isn't tested enough
Solution:   Improve test coverage of :colorscheme. (Dominique Pelle, closes
            #3777)  Remove unnecessary sleep.

patch 8.1.0704: building with Ruby 2.6 gives compiler warnings

Problem:    Building with Ruby 2.6 gives compiler warnings.
Solution:   Define a stub for rb_ary_detransient. (Ozaki Kiichi, closes #3779)

patch 8.1.0703: compiler warnings with 64-bit compiler

Problem:    Compiler warnings with 64-bit compiler.
Solution:   Change types, add type casts. (Mike Williams)

patch 8.1.0702: ":sign place" only uses the current buffer

Problem:    ":sign place" only uses the current buffer.
Solution:   List signs for all buffers when there is no buffer argument.
            Fix error message for invalid buffer name in sign_place().
            (Yegappan Lakshmanan, closes #3774)

patch 8.1.0701: sign message not translated and inconsistent spacing

Problem:    Sign message not translated and inconsistent spacing.
Solution:   Add _() for translation.  Add a space. (Ken Takata)  Also use
            MSG_BUF_LEN instead of BUFSIZ.

patch 8.1.0700: using "gt" sometimes does not redraw a tab

Problem:    Using "gt" sometimes does not redraw a tab. (Jason Franklin)
Solution:   Always set must_redraw in redraw_all_later().

patch 8.1.0699: compiler warning for uninitialized variable

Problem:    Compiler warning for uninitialized variable. (Tony Mechelynck)
Solution:   Add a dummy init.

patch 8.1.0698: clearing the window is used too often

Problem:    Clearing the window is used too often, causing the command line
            to be cleared when opening a tab. (Miroslav Koškár)
Solution:   Use NOT_VALID instead of CLEAR.  (suggested by Jason Franklin,
            closes #630)  Also do this for a few other places where clearing
            the screen isn't really needed.

patch 8.1.0697: ":sign place" requires the buffer argument

Problem:    ":sign place" requires the buffer argument.
Solution:   Make the argument optional.  Also update the help and clean up the
            sign test. (Yegappan Lakshmanan, closes #3767)

patch 8.1.0696: when test_edit fails 'insertmode' may not be reset

Problem:    When test_edit fails 'insertmode' may not be reset and the next
            test may get stuck. (James McCoy)
Solution:   Always reset 'insertmode' after executing a test.  Avoid that an
            InsertCharPre autocommand or a 'complete' function can change the
            state. (closes #3768)

patch 8.1.0695: internal error when using :popup

Problem:    Internal error when using :popup.
Solution:   When a menu only exists in Terminal mode give an error. (Naruhiko
            Nishino, closes #3765)

patch 8.1.0694: when using text props may free memory that is not allocated

Problem:    When using text props may free memory that is not allocated.
            (Andy Massimino)
Solution:   Allocate the line when adjusting text props. (closes #3766)

patch 8.1.0693: channel test fails sometimes

Problem:    Channel test fails sometimes.
Solution:   Avoid race condition.

patch 8.1.0692: if a buffer was deleted a channel can't write to it

Problem:    If a buffer was deleted a channel can't write to it.
Solution:   When the buffer exists but was unloaded, prepare it for writing.
            (closes #3764)

patch 8.1.0691: text properties are not adjusted for :substitute

Problem:    Text properties are not adjusted for :substitute.
Solution:   Adjust text properties as well as possible.

patch 8.1.0690: setline() and setbufline() do not clear text properties

Problem:    setline() and setbufline() do not clear text properties.
Solution:   Clear text properties when setting the text.

patch 8.1.0689: undo with text properties not tested

Problem:    Undo with text properties not tested.
Solution:   Add a test function.

patch 8.1.0688: text properties are not restored by undo

Problem:    Text properties are not restored by undo.
Solution:   Also save text properties for undo.

patch 8.1.0687: sentence text object in Visual mode is not tested

Problem:    Sentence text object in Visual mode is not tested.
Solution:   Add a test. (Dominique Pelle, closes #3758)

patch 8.1.0686: when 'y' is in 'cpoptions' yanking for the clipboard changes redo

Problem:    When 'y' is in 'cpoptions' yanking for the clipboard changes redo.
Solution:   Do not use the 'y' flag when "gui_yank" is TRUE. (Andy Massimino,
            closes #3760)

patch 8.1.0685: get_buf_tv() is named inconsistently

Problem:    get_buf_tv() is named inconsistently.
Solution:   Rename it to tv_get_buf(). (Yegappan Lakshmanan, closes #3759)

patch 8.1.0684: warnings from 64-bit compiler

Problem:    Warnings from 64-bit compiler.
Solution:   Add type casts. (Mike Williams)

patch 8.1.0683: spell highlighting does not always end

Problem:    Spell highlighting does not always end. (Gary Johnson)
Solution:   Also reset char_attr when spell errors are highlighted.

patch 8.1.0682: text properties not adjusted when backspacing replaced text

Problem:    Text properties are not adjusted when backspacing replaced text.
Solution:   Keep text properties on text restored in replace mode.

patch 8.1.0681: text properties as not adjusted for deleted text

Problem:    Text properties as not adjusted for deleted text.
Solution:   Adjust text properties when backspacing to delete text.

patch 8.1.0680: not easy to see what features are unavailable

Problem:    Not easy to see what features are unavailable.
Solution:   Highlight disabled features in the :version output. (Nazri Ramliy,
            closes #3756)

patch 8.1.0679: sign functions do not take buffer argument as documented

Problem:    Sign functions do not take buffer argument as documented.
Solution:   Use get_buf_tv(). (Yegappan Lakshmanan, closes #3755)

patch 8.1.0678: text properties as not adjusted for inserted text

Problem:    Text properties as not adjusted for inserted text.
Solution:   Adjust text properties when inserting text.

patch 8.1.0677: look-behind match may use the wrong line number

Problem:    Look-behind match may use the wrong line number. (Dominique Pelle)
Solution:   Use the line number in regsave instead of the one in behind_pos,
            we may be looking at the previous line. (closes #3749)

patch 8.1.0676: textprop screendump test fails

Problem:    Textprop screendump test fails.
Solution:   Add missing changes.

patch 8.1.0675: text property column in screen columns is not practical

Problem:    Text property column is screen columns is not practical.
Solution:   Use byte values for the column.

Update runtime files

patch 8.1.0674: leaking memory when updating a single line

Problem:    Leaking memory when updating a single line.
Solution:   Do not call start_search_hl() twice.

patch 8.1.0673: functionality for signs is spread out over several files

Problem:    Functionality for signs is spread out over several files.
Solution:   Move most of the sign functionality into sign.c. (Yegappan
            Lakshmanan, closes #3751)

patch 8.1.0672: the Lua interface doesn't know about v:null

Problem:    The Lua interface doesn't know about v:null.
Solution:   Add Lua support for v:null. (Uji, closes #3744)

patch 8.1.0671: cursor in the wrong column after auto-formatting

Problem:    Cursor in the wrong column after auto-formatting.
Solution:   Check for deleting more spaces than adding. (closes #3748)

patch 8.1.0670: macro for popup menu width is unused

Problem:    Macro for popup menu width is unused.
Solution:   Remove it. (Hirohito Higashi)

patch 8.1.0669: the ex_sign() function is too long

Problem:    The ex_sign() function is too long.
Solution:   Refactor the function.  Add a bit more testing. (Yegappan
            Lakshmanan, closes #3745)

patch 8.1.0668: no test for overstrike mode in the command line

Problem:    No test for overstrike mode in the command line.
Solution:   Add a test. (Dominique Pelle, closes #3742)

patch 8.1.0667: textprop test leaves file behind

Problem:    Textprop test leaves file behind.
Solution:   Delete the file. (Dominique Pelle, closes #3743)

patch 8.1.0666: text property test fails

Problem:    Text property test fails.
Solution:   Update screenshot.

patch 8.1.0665: text property display wrong when 'spell' is set

Problem:    Text property display wrong when 'spell' is set. (Dominique Pelle)
Solution:   Remove unnecessary assignment to char_attr.  Combine attributes if
            needed.  Add a screenshot test.

patch 8.1.0664: configure "fail-if-missing" does not apply to enable-gui

Problem:    Configure "fail-if-missing" does not apply to the enable-gui
            argument. (Rhialto)
Solution:   Make configure fail if a GUI was specifified and "fail-if-missing"
            is enabled and the GUI test fails.

patch 8.1.0663: text property display wrong when 'number' is set

Problem:    Text property display wrong when 'number' is set. (Dominique
            Pelle)
Solution:   Compare with "vcol"  instead of "col".

patch 8.1.0662: needlessly searching for tilde in string

Problem:    Needlessly searching for tilde in string.
Solution:   Only check the first character. (James McCoy, closes #3734)

patch 8.1.0661: clipboard regexp might be used recursively

Problem:    Clipboard regexp might be used recursively.
Solution:   Check for recursive use and bail out.

patch 8.1.0660: sign_cleanup() may leak memory

Problem:    sign_cleanup() may leak memory.
Solution:   Free the group name before returning.  Add a few more tests.
            (Yegappan Lakshmanan)

patch 8.1.0659: build failure without the sign feature

Problem:    Build failure without the sign feature.
Solution:   Put the sign struct declarations outside of the #ifdef.

patch 8.1.0658: deleting signs and completion for :sign is insufficient

Problem:    Deleting signs and completion for :sign is insufficient.
Solution:   Add deleting signs in a specified or any group from the current
            cursor location.  Add group and priority to sign command
            completion. Add tests for different sign unplace commands. Update
            help text.  Add tests for sign jump with group. Update help for
            sign jump. (Yegappan Lakshmanan, closes #3731)

patch 8.1.0657: get error for using regexp recursively

Problem:    Get error for using regexp recursively. (Dominique Pelle)
Solution:   Do no check if connection is desired.

patch 8.1.0656: trying to reconnect to X server may cause problems

Problem:    Trying to reconnect to X server may cause problems.
Solution:   Do no try reconnecting when exiting. (James McCoy)

patch 8.1.0655: when appending a line text property flags are not added

Problem:    When appending a line text property flags are not added.
Solution:   Add text properties to a newly added line.

patch 8.1.0654: when deleting a line text property flags are not adjusted

Problem:    When deleting a line text property flags are not adjusted.
Solution:   Adjust text property flags in preceding and following lines.

patch 8.1.0653: arglist test fails on MS-windows

Problem:    Arglist test fails on MS-windows.
Solution:   Only use a file name with a double quote on Unix.

patch 8.1.0652: freeing memory for balloon eval too early

Problem:    Freeing memory for balloon eval too early.
Solution:   Store the pointer in BalloonEval and free it later. (Yasuhiro
            Matsumoto, closes #3725)

patch 8.1.0651: :args \"foo works like :args without argument

Problem:    :args \"foo works like :args without argument.
Solution:   Fix check for empty argument. (closes #3728)

patch 8.1.0650: command line argument -q [errorfile] is not tested

Problem:    Command line argument -q [errorfile] is not tested.
Solution:   Add a test. (Dominique Pelle, closes #3730)

patch 8.1.0649: setjmp() variables defined globally are used in one file

Problem:    setjmp() variables defined globally are used in one file.
Solution:   Move the declarations to that file.

Update runtime files.

patch 8.1.0648: custom operators can't act upon a forced motion

Problem:    Custom operators can't act upon a forced motion. (Christian
            Wellenbrock)
Solution:   Add the forced motion to the mode() result. (Christian Brabandt,
            closes #3490)

patch 8.1.0647: MS-Windows: balloon_show() does not handle wide characters

Problem:    MS-Windows: balloon_show() does not handle wide characters.
Solution:   Use CreateWindowExW(). (Yasuhiro Matsumoto, closes #3708)

patch 8.1.0646: cannot build with Ruby 2.6.0

Problem:    Cannot build with Ruby 2.6.0.
Solution:   Add rb_ary_detransient(). (Ozaki Kiichi, closes #3724)

patch 8.1.0645: Coverity warns for possible use of NULL pointer

Problem:    Coverity warns for possible use of NULL pointer.
Solution:   Check return value of vterm_obtain_screen().

patch 8.1.0644: finding next sign ID is inefficient

Problem:    Finding next sign ID is inefficient.
Solution:   Add next_sign_id. (Yegappan Lakshmanan, closes #3717)

patch 8.1.0643: computing byte offset wrong

Problem:    Computing byte offset wrong. (Bjorn Linse)
Solution:   Use the right variable for array index.

patch 8.1.0642: swapinfo() leaks memory

Problem:    swapinfo() leaks memory.
Solution:   Avoid allocating the strings twice.

patch 8.1.0641: no check for out-of-memory when converting regexp

Problem:    No check for out-of-memory when converting regexp.
Solution:   Bail out when lalloc() returns NULL. (John Marriott)

patch 8.1.0640: get E14 while typing command :tab with 'incsearch' set

Problem:    Get E14 while typing command :tab with 'incsearch' set.
Solution:   Do not give an error when looking for the command. (Yasuhiro
            Higashi)

patch 8.1.0639: text properties test fails on MS-Windows

Problem:    text properties test fails on MS-Windows
Solution:   Set fileformat to "unix".

patch 8.1.0638: text property highlighting is off by one column

Problem:    Text property highlighting is off by one column. (Bjorn Linse)
Solution:   Update text property highlighting earlier.  Let it overrule syntax
            highlighting.

patch 8.1.0637: nsis file no longer used

Problem:    Nsis file no longer used.
Solution:   Remove the file. (Ken Takata)

patch 8.1.0636: line2byte() gives wrong values with text properties

Problem:    line2byte() gives wrong values with text properties. (Bjorn Linse)
Solution:   Compute byte offsets differently when text properties were added.
            (closes #3718)

patch 8.1.0635: Coverity complains about null pointer use

Problem:    Coverity complains about null pointer use.
Solution:   Avoid using a null pointer.

patch 8.1.0634: text properties cannot cross line boundaries

Problem:    Text properties cannot cross line boundaries.
Solution:   Support multi-line text properties.

patch 8.1.0633: crash when out of memory while opening a terminal window

Problem:    Crash when out of memory while opening a terminal window.
Solution:   Handle out-of-memory more gracefully.

patch 8.1.0632: using sign group names is inefficient

Problem:    Using sign group names is inefficient.
Solution:   Store group names in a hash table and use a reference to them.
            Also remove unnecessary use of ":exe" from the tests.  (Yegappan
            Lakshmanan, closes #3715)

patch 8.1.0631: test for :stop fails on Arch

Problem:    Test for :stop fails on Arch.
Solution:   Check five lines for the expected output. (closes #3714)

patch 8.1.0630: "wincmd p" does not work after using an autocmd window

Problem:   "wincmd p" does not work after using an autocmd window.
Solution:   Store "prevwin" in aco_save_T. (Christian Brabandt, closes #3690)

patch 8.1.0629: "gn" selects the wrong text with a multi-line match

Problem:    "gn" selects the wrong text with a multi-line match.
Solution:   Get the end position from searchit() directly. (closes #3695)

Ignore output files from indent tests.

patch 8.1.0628: Compiler warning on MS-Windows.

Problem:    Compiler warning on MS-Windows.
Solution:   Add type cast. (Mike Williams)