patch 8.1.1718: popup menu highlighting does not look good

Problem:    Popup menu highlighting does not look good.
Solution:   Highlight the whole window line.  Fix that sign line HL is not
            displayed in a window with a background color.

patch 8.1.1717: last char in menu popup window highlighted

Problem:    Last char in menu popup window highlighted.
Solution:   Do not highlight an extra character twice.

patch 8.1.1716: old style comments are wasting space

Problem:    Old style comments are wasting space
Solution:   Use new style comments in option header file.

patch 8.1.1715: emoji characters are seen as word characters for spelling

Problem:    Emoji characters are seen as word characters for spelling. (Gautam
            Iyer)
Solution:   Exclude class 3 from word characters.

Update runtime files

patch 8.1.1714: cannot preview a file in a popup window

Problem:    Cannot preview a file in a popup window.
Solution:   Add the 'previewpopup' option.

patch 8.1.1713: highlighting cursor line only works with popup_menu()

Problem:    Highlighting cursor line only works with popup_menu().
Solution:   Add the "cursorline" property. (Naruhiko Nishino, closes #4671)

patch 8.1.1712: signs in number column cause text to be misaligned

Problem:    Signs in number column cause text to be misaligned.
Solution:   Improve alignment. (Yasuhiro Matsumoto, closes #4694)

patch 8.1.1711: listener callback called at the wrong moment

Problem:    Listener callback called at the wrong moment
Solution:   Invoke listeners before calling ml_delete_int(). (closes #4657)

patch 8.1.1710: Coverity found dead code

Problem:    Coverity found dead code.
Solution:   Remove merging of listener changes.

patch 8.1.1709: Coverity warns for possibly using a NULL pointer

Problem:    Coverity warns for possibly using a NULL pointer.
Solution:   Make sure no NULL pointer is used.

patch 8.1.1708: Coverity warns for using uninitialized variable

Problem:    Coverity warns for using uninitialized variable.
Solution:   Set the start col when col is set.

patch 8.1.1707: Coverity warns for possibly using a NULL pointer

Problem:    Coverity warns for possibly using a NULL pointer.
Solution:   Change the logic to make sure no NULL pointer is used.

patch 8.1.1706: typo in #ifdef

Problem:    Typo in #ifdef.
Solution:   Change PROT to PROTO.

patch 8.1.1705: using ~{} for a literal dict is not nice

Problem:    Using ~{} for a literal dict is not nice.
Solution:   Use #{} instead.

patch 8.1.1704: C-R C-W does not work after C-G when using 'incsearch'

Problem:    C-R C-W does not work after C-G when using 'incsearch'.
Solution:   Put cursor at end of the match. (Yasuhiro Matsumoto, closes #4664)

patch 8.1.1703: breaking out of loop by checking window pointer insufficient

Problem:    Breaking out of loop by checking window pointer is insufficient.
Solution:   Check the window ID and the buffer number. (closes #4683)

patch 8.1.1702: compiler warning for uninitialized variable

Problem:    Compiler warning for uninitialized variable.
Solution:   Initialize it. (Christian Brabandt)

patch 8.1.1701: Appveyor build with MSVC fails puts progress bar in log

Problem:    Appveyor build with MSVC fails puts progress bar in log.
Solution:   Adjust the sed command. (Ken Takata)

patch 8.1.1700: listener callback called for the wrong buffer

Problem:    Listener callback called for the wrong buffer.
Solution:   Invoke listeners before calling ml_append_int().

patch 8.1.1699: highlight_ga can be local instead of global

Problem:    Highlight_ga can be local instead of global.
Solution:   Move highlight_ga into highlight.c. (Yegappan Lakshmanan,
            closes #4675)

patch 8.1.1698: Appveyor build with MSVC fails

Problem:    Appveyor build with MSVC fails.
Solution:   Remove the sed command

patch 8.1.1697: cannot build with MSVC

Problem:    Cannto build with MSVC.
Solution:   Remove the backslashes after the @<< mechanism.

patch 8.1.1696: MSVC: link command line is too long

Problem:    MSVC: link command line is too long.
Solution:   Use the @<< mechanism to pass the arguments via a file. (Christian
            Brabandt)

patch 8.1.1695: Windows 10: crash when cursor is at bottom of terminal

Problem:    Windows 10: crash when cursor is at bottom of terminal.
Solution:   Position the cursor before resizing. (Yasuhiro Matsumoto,
            closes #4679)

patch 8.1.1694: the RUN_VIM variable is longer than needed

Problem:    The RUN_VIM variable is longer than needed.
Solution:   Shorten RUN_VIM. (Daniel Hahler, closes #4643)

patch 8.1.1693: syntax coloring and highlighting is in one big file

Problem:    Syntax coloring and highlighting is in one big file.
Solution:   Move the highlighting to a separate file. (Yegappan Lakshmanan,
            closes #4674)

patch 8.1.1692: using *{} for literal dict is not backwards compatible

Problem:    Using *{} for literal dict is not backwards compatible. (Yasuhiro
            Matsumoto)
Solution:   Use ~{} instead.

patch 8.1.1691: diff test fails on some systems

Problem:    Diff test fails on some systems. (Elimar Riesebieter)
Solution:   Add a term_wait() call.

patch 8.1.1690: default padding for popup window menu is too much

Problem:    Default padding for popup window menu is too much.
Solution:   Only add padding left and right.

patch 8.1.1689: profiling code is spread out

Problem:    Profiling code is spread out.
Solution:   Move more profiling code to profiler.c. (Yegappan Lakshmanan,
            closes #4668)

patch 8.1.1688: old makefiles are no longer useful

Problem:    Old makefiles are no longer useful.
Solution:   Delete the makefiles, they most likely don't work anyway.

patch 8.1.1687: the evalfunc.c file is too big

Problem:    The evalfunc.c file is too big.
Solution:   Move testing support to a separate file.

patch 8.1.1686: "*" of "*{" is recognized as multipy operator

Problem:    "*" of "*{" is recognized as multipy operator. (Yasuhiro Matsumoto)
Solution:   Check for the "{".

patch 8.1.1685: missing file in distributed file list

Problem:    Missing file in distributed file list.
Solution:   Add profiling.pro

Update runtime files

patch 8.1.1684: profiling functionality is spread out

Problem:    Profiling functionality is spread out.
Solution:   Put profiling functionality in profiling.c. (Yegappan Lakshmanan,
            closes #4666)

patch 8.1.1683: dictionary with string keys is longer than needed

Problem:    Dictionary with string keys is longer than needed.
Solution:   Use *{key: val} for literaly keys.

patch 8.1.1682: placing a larger number of signs is slow

Problem:    Placing a larger number of signs is slow.
Solution:   Add functions for dealing with a list of signs. (Yegappan
            Lakshmanan, closes #4636)

patch 8.1.1681: insert stray "{" when listener gets buffer line

Problem:    Insert stray "{" when listener gets buffer line. (Paul Jolly)
Solution:   Flush the cached line after invoking listeners. (closes #4455)

patch 8.1.1680: the command table is not well aligned

Problem:    The command table is not well aligned.
Solution:   Adjust indent.

patch 8.1.1679: test using SwapExists autocommand file may fail

Problem:    Test using SwapExists autocommand file may fail.
Solution:   Remove the SwapExists autocommand.

patch 8.1.1678: using popup_menu() does not scroll to show the selected line

Problem:    When using popup_menu() does not scroll to show the selected line.
Solution:   Scroll the text. (Naruhiko Nishino, closes #4651)

patch 8.1.1677: tests get stuck when running into an existing swapfile

Problem:    Tests get stuck when running into an existing swapfile.
Solution:   Set v:swapchoice to "q" and report an error. (Daniel Hahler,
            closes #4644)

patch 8.1.1676: "maxwidth" of popup window does not always work properly

Problem:    "maxwidth" of popup window does not always work properly.
Solution:   Adjust the computation. (Naruhiko Nishino, closes #4653)

patch 8.1.1675: listener list not correctly updated on listener_remove()

Problem:    Listener list not correctly updated on listener_remove().
Solution:   Only set "prev" when not removing a listener.  Return one if the
            listener was found and removed.

patch 8.1.1674: script to check a colorscheme can be improved

Problem:    Script to check a colorscheme can be improved.
Solution:   Match the whole group name. Don't warn for what is usually omitted.

patch 8.1.1673: cannot easily find the popup window at a certain position

Problem:    Cannot easily find the popup window at a certain position.
Solution:   Add popup_locate().

patch 8.1.1672: "make cmdidxs" doesn't work

Problem:    "make cmdidxs" doesn't work.
Solution:   Update macro names. (Naruhiko Nishino, closes #4660)

patch 8.1.1671: copying a blob may result in it being locked

Problem:    Copying a blob may result in it being locked.
Solution:   Reset v_lock. (Ken Takata, closes #4648)

patch 8.1.1670: sign column not always properly aligned

Problem:    Sign column not always properly aligned.
Solution:   Use "col" only after it was calculated. (Yee Cheng Chin,
            closes #4649)

patch 8.1.1669: Travis: test results section is closed even when failed

Problem:    Travis: test results section is closed even when some tests
            failed.
Solution:   Only close the section on success. (Daniel Hahler, closes #4659)

patch 8.1.1668: popup window test is a bit flaky on some systems

Problem:    Popup window test is a bit flaky on some systems.
Solution:   Clear the command line. (Naruhiko Nishino, closes #4656)

patch 8.1.1667: flags for Ex commands may clash with other symbols

Problem:    Flags for Ex commands may clash with other symbols.
Solution:   Prepend with EX_.

patch 8.1.1666: click in popup window scrollbar with border doesn't scroll

Problem:    Click in popup window scrollbar with border doesn't scroll.
Solution:   Correct column for the border. (Naruhiko Nishino, closes #4650)

patch 8.1.1665: crash when popup window with mask is below the screen

Problem:    Crash when popup window with mask is below the screen.
Solution:   Correct boundary check.

patch 8.1.1664: GUI resize may cause changing Rows at a bad time

Problem:    GUI resize may cause changing Rows at a bad time. (Dominique
            Pelle)
Solution:   Postpone resizing while updating the screen.

patch 8.1.1663: compiler warning for using size_t

Problem:    Compiler warning for using size_t.
Solution:   Add type cast. (Mike Williams)

patch 8.1.1662: cannot build uninstal.exe with some version of MinGW

Problem:    Cannot build uninstal.exe with some version of MinGW.
Solution:   Add -lole32. (Rene Nyffenegger, closes #4646)

patch 8.1.1661: cannot build with +textprop but without +balloon_eval

Problem:    Cannot build with +textprop but without +balloon_eval.
Solution:   Adjust #ifdefs. (closes #4645)

patch 8.1.1660: assert_fails() does not fail inside try/catch

Problem:    Assert_fails() does not fail inside try/catch.
Solution:   Set trylevel to zero. (Ozaki Kiichi, closes #4639)

patch 8.1.1659: popup window "mousemoved" values not correct

Problem:    Popup window "mousemoved" values not correct.
Solution:   Convert text column to mouse column.

patch 8.1.1658: debug statements included in patch

Problem:    Debug statements included in patch.
Solution:   Remove the debug statements.

patch 8.1.1657: Terminal: screen updates from 'balloonexpr' are not displayed

Problem:    Terminal: screen updates from 'balloonexpr' are not displayed.
Solution:   Update the screen if needed.  Fix the word position for
            "mousemoved".

patch 8.1.1656: popup window width is wrong when using Tabs

Problem:    Popup window width is wrong when using Tabs. (Paul Jolly)
Solution:   Count tabs correctly. (closes #4637)

patch 8.1.1655: popup window border drawn wrong with multi-byte char

Problem:    Popup window border drawn wrong with multi-byte char. (Marcin
            Szamotulski)
Solution:   Correct check in mb_fix_col(). (closes #4635)

patch 8.1.1654: GUI: screen updates from 'balloonexpr' are not displayed

Problem:    GUI: screen updates from 'balloonexpr' are not displayed.
Solution:   Update the screen if needed.  Also avoid the cursor being
            displayed in the wrong position.

patch 8.1.1653: ubsan warns for possibly passing NULL pointer

Problem:    Ubsan warns for possibly passing NULL pointer.
Solution:   Skip code when length is zero. (Dominique Pelle, closes #4631)

patch 8.1.1652: GUI: popup window doesn't close on mouse movement

Problem:    GUI: popup window doesn't close on mouse movement. (Paul Jolly)
Solution:   Generate mouse-move events when a popup window is visible.

patch 8.1.1651: suspend test is flaky on some systems

Problem:    Suspend test is flaky on some systems.
Solution:   Wait for the shell prompt to show. (Yee Cheng Chin, closes #4632)

patch 8.1.1650: warning for using uninitialized variable

Problem:    Warning for using uninitialized variable. (Tony Mechelynck)
Solution:   Simplify the code by always using the mouse coordinates.

patch 8.1.1649: Illegal memory access when closing popup window

Problem:    Illegal memory access when closing popup window.
Solution:   Get w_next before closing the window.

patch 8.1.1648: MS-Windows: build error with normal feaures

Problem:    MS-Windows: build error with normal feaures.
Solution:   Adjust #ifdef for find_word_under_cursor().

patch 8.1.1647: build error with GTK and hangulinput feature

Problem:    Build error with GTK and hangulinput feature, im_get_status()
            defined twice. (Dominique Pelle)
Solution:   Adjust im_get_status(). (closes #4628)

patch 8.1.1646: build failure

Problem:    build failure
Solution:   Add changes to structure.

patch 8.1.1645: cannot use a popup window for a balloon

Problem:    Cannot use a popup window for a balloon.
Solution:   Add popup_beval().  Add the "mousemoved" property.  Add the
            screenpos() function.

patch 8.1.1644: sound test does not work on Travis

Problem:    Sound test does not work on Travis.
Solution:   Use "sg" command to enable audio. (Ozaki Kiichi, closes #4624)

patch 8.1.1643: sign placement is wrong when 'foldcolumn' is set

Problem:    Sign placement is wrong when 'foldcolumn' is set.
Solution:   Adjust the column computation. (Yee Cheng Chin, closes #4627)

patch 8.1.1642: may use uninitialized variable

Problem:    May use uninitialized variable. (Patrick Palka)
Solution:   Initialize variables earlier. (closes #4623)

patch 8.1.1641: garbage collection may run at a wrong moment

Problem:    Garbage collection may run at a wrong moment. (Trygve Aaberge)
Solution:   Postpone garbage collection while parsing messages. (closes #4620)

patch 8.1.1640: the CursorHold autocommand takes down a balloon

Problem:    The CursorHold autocommand takes down a balloon. (Paul Jolly)
Solution:   Ignore the CursorHold pseudo-key.

patch 8.1.1639: changing an autoload name into a file name is inefficient

Problem:    Changing an autoload name into a script file name is inefficient.
Solution:   Remember the last replaced #. (Ozaki Kiichi, closes #4618)

patch 8.1.1638: running tests leaves some files behind

Problem:    Running tests leaves some files behind.
Solution:   Delete the files. (Ozaki Kiichi, closes #4617)

patch 8.1.1637: after running tests and clean the XfakeHOME directory remains

Problem:    After running tests and clean the XfakeHOME directory remains.
Solution:   Use "rm -rf". (Hirohito Higashi)

patch 8.1.1636: crash when popup has fitting scrollbar

Problem:    Crash when popup has fitting scrollbar. (Trygve Aaberge)
Solution:   Don't divide by zero if the scrollbar just fits. (closes #4615)

patch 8.1.1635: warnings for unused variables in small version

Problem:    Warnings for unused variables in small version. (John Marriott)
Solution:   Adjust #ifdefs.

patch 8.1.1634: terminal test fails when term_getansicolors() is missing

Problem:    Terminal test fails when term_getansicolors() is missing.
            Diff test fails without +rightleft.  (Dominique Pelle)
Solution:   Check if term_getansicolors() is supported. (closes #4597)

patch 8.1.1633: cannot generate prototypes with X11 but without GUI

Problem:    Cannot generate prototypes with X11 but without GUI.
Solution:   Include X11/Intrinsic.h.

patch 8.1.1632: build with EXITFREE but without +arabic fails

Problem:    Build with EXITFREE but without +arabic fails.
Solution:   Rename the function and adjust #ifdefs. (closes #4613)

patch 8.1.1631: displaying signs is inefficient

Problem:    Displaying signs is inefficient.
Solution:   Avoid making multiple calls to get information about a placed
            sign. (Yegappan Lakshmanan, closes #4586)

patch 8.1.1630: various small problems

Problem:    Various small problems.
Solution:   Various small improvements.

Update runtime files

patch 8.1.1629: terminal function help is in the wrong file

Problem:    Terminal function help is in the wrong file.
Solution:   Move the function details to terminal.txt.

patch 8.1.1628: popup window functions not in list of functions

Problem:    Popup window functions not in list of functions.
Solution:   Add popup window functins to the list of functions.  Reorganise
            the popup window help.

patch 8.1.1627: header file contains mixed comment style

Problem:    Header file contains mixed comment style.
Solution:   Use // style comments.

patch 8.1.1626: no test for closing a popup window with a modified buffer

Problem:    No test for closing a popup window with a modified buffer.
Solution:   Add a test.  Add "popups" to getbufinfo().

patch 8.1.1625: script line numbers are not exactly right

Problem:    Script line numbers are not exactly right.
Solution:   Handle heredoc and continuation lines better. (Ozaki Kiichi,
            closes #4611, closes #4511)

patch 8.1.1624: when testing in the GUI may try to run gvim in a terminal

Problem:    When testing in the GUI may try to run gvim in a terminal.
Solution:   Add the -v argument. (Yee Cheng Chin, closes #4605)  Don't skip
            tests that work now.

patch 8.1.1623: display wrong with signs in narrow number column

Problem:    Display wrong with signs in narrow number column.
Solution:   Increase the numbercolumn width if needed. (Yegappan Lakshmanan,
            closes #4606)

patch 8.1.1622: wrong width if displaying a lot of lines in a popup window

Problem:    Wrong width if displaying a lot of lines in a popup window.
Solution:   Accurately compute the line overflow.