patch 8.0.1849: compiler warning for unused arguments, missing prototype

Problem:    compiler warning for unused arguments and missing prototype
Solution:   Add UNUSED.  Add static.

patch 8.0.1848: 'termwinscroll' does not work properly

Problem:    'termwinscroll' does not work properly. (Dominique Pelle)
Solution:   Subtract removed scrollback from the scrollback count.  Add a test
            for 'termwinscroll'. (closes #2909)

patch 8.0.1847: some build options don't have an example

Problem:    Some build options don't have an example.
Solution:   Add a couple more examples and compiler flags.

patch 8.0.1846: Python interface is incompatible with lldb

Problem:    Python interface is incompatible with lldb.
Solution:   For OutputType set the base to be PyFile_Type. (Boxu Zhang)
            Partly disabled to avoid a crash.

patch 8.0.1845: various comment updates needed, missing white space

Problem:    Various comment updates needed, missing white space.
Solution:   Update comments, add white space.

patch 8.0.1844: superfluous quickfix code, missing examples

Problem:    Superfluous quickfix code, missing examples.
Solution:   Remove unneeded code.  Add a few examples.  Add a bit more
            testing. (Yegappan Lakshmanan, closes #2916)

patch 8.0.1843: entry for 'wrap' in options window is wrong

Problem:    Entry for 'wrap' in options window is wrong. (John Little)
Solution:   Make the change apply locally.

patch 8.0.1842: popup menu inside terminal window isn't cleared

Problem:    Popup menu inside terminal window isn't cleared.
Solution:   Use NOT_VALID in pum_undisplay(). (suggested by Christian
            Brabandt, closes #2908)

patch 8.0.1841: HP-UX does not have setenv()

Problem:    HP-UX does not have setenv().
Solution:   Use vim_setenv(). (John Marriott)

patch 8.0.1840: getwinpos() is not tested

Problem:    getwinpos() is not tested.
Solution:   Add a test. (Dominique Pelle, closes #2911)

patch 8.0.1839: script to check .po file doesn't check for plural header

Problem:    Script to check .po file doesn't check for plural header.
Solution:   Add a check that the plural header is present when needed.

Updated runtime files

patch 8.0.1838: cursor in wrong pos when switching to Terminal-Normal mode

Problem:    Cursor in wrong position when switching to Terminal-Normal mode.
            (Dominique Pelle)
Solution:   Move to the end of the line if coladvance() fails.  Do not take a
            snapshot a second time.

patch 8.0.1837: one character cmdline abbreviation not triggered after '<,'>

Problem:    One character cmdline abbreviation not triggered after '<,'>.
Solution:   Skip over the special range. (Christian Brabandt, closes #2320)

patch 8.0.1836: buffer-local window options may not be recent

Problem:    Buffer-local window options may not be recent if the buffer is
            still open in another window.
Solution:   Copy the options from the window instead of the outdated window
            options. (Bjorn Linse, closes #2336)

patch 8.0.1835: print document name does not support multi-byte

Problem:    Print document name does not support multi-byte.
Solution:   Use StartDocW() if needed. (Yasuhiro Matsumoto, closes #2478)

patch 8.0.1834: GUI: find/replace dialog does not handle some chars

Problem:    GUI: find/replace dialog does not handle some chars properly.
Solution:   Escape '?' when needed.  Always escape backslash. (closes #2418,
            closes #2435)

patch 8.0.1833: X11: ":echo 3.14" gives E806

Problem:    X11: ":echo 3.14" gives E806.
Solution:   set LC_NUMERIC to "C". (Dominique Pelle, closes #2368)

patch 8.0.1832: cannot use :unlet for an environment variable

Problem:    Cannot use :unlet for an environment variable.
Solution:   Make it work.  Use unsetenv() if available. (Ken Takata,
            closes #2855)

patch 8.0.1831: sometimes the quickfix title is incorrectly prefixed with ':'

Problem:    Sometimes the quickfix title is incorrectly prefixed with ':'.
Solution:   Prepend the colon in another way. (Yegappan Lakshmanan, closes
            #2905)

patch 8.0.1830: switching to Terminal-Normal mode does not redraw

Problem:    Switching to Terminal-Normal mode does not redraw. (Dominique
            Pelle)
Solution:   Also redraw when not updating the snapshot. (closes #2904)

patch 8.0.1829: MS-Windows: script for vimdiff can't handle ! chars

Problem:    MS-Windows: script for vimdiff can't handle ! chars.
Solution:   Escape the ! chars. (Hans Ginzel, closes #2896)

patch 8.0.1828: get no clue why :gui does not fork

Problem:    Get no clue why :gui does not fork.
Solution:   Add a channel log message.

patch 8.0.1827: compiler warning for signed/unsigned char pointers

Problem:    Compiler warning for signed/unsigned char pointers. (Cesar Romani)
Solution:   Change the type of jv_argv.

Update runtime files.

patch 8.0.1826: configure uses old compiler flag

Problem:    Configure uses old compiler flag.
Solution:   Remove _DARWIN_C_SOURCE. (Kazunobu Kuriyama)

patch 8.0.1825: might use NULL pointer when out of memory

Problem:    Might use NULL pointer when out of memory. (Coverity)
Solution:   Handle NULL pointer better.

patch 8.0.1824: Coverity warns for variable that may be uninitialized

Problem:    Coverity warns for variable that may be uninitialized.
Solution:   Initialize the variable.

patch 8.0.1823: test for terminal stdout redirection is flaky

Problem:    Test for terminal stdout redirection is flaky.
Solution:   Wait for the job to finish.

patch 8.0.1822: make uninstall does not remove colors/tools

Problem:    Make uninstall does not remove colors/tools.
Solution:   Add a line to delete the tools directory. (Kazunobu Kuriyama)

patch 8.0.1821: cursor in terminal window moves when pressing CTRL-W

Problem:    Cursor in terminal window moves when pressing CTRL-W. (Dominique
            Pelle)
Solution:   Do not more the cursor or redraw when not in Terminal-Normal mode.
            (closes #2904)

patch 8.0.1820: terminal window redirecting stdout does not show stderr

Problem:    Terminal window redirecting stdout does not show stderr. (Matéo
            Zanibelli)
Solution:   When stdout is not connected to pty_master_fd then use it for
            stderr. (closes #2903)

patch 8.0.1819: swap file warning for file with non-existing directory

Problem:    Swap file warning for a file in a non-existing directory, if there
            is another with the same file name. (Juergen Weigert)
Solution:   When expanding the file name fails compare the file names.

patch 8.0.1818: lines remove from wrong buffer when using terminal window

Problem:    Lines remove from wrong buffer when using terminal window.
Solution:   Make sure to use tl_buffer.

patch 8.0.1817: a timer may change v:count unexpectedly

Problem:    A timer may change v:count unexpectedly.
Solution:   Save and restore v:count and similar variables when a timer
            callback is invoked. (closes #2897)

patch 8.0.1816: no test for setcmdpos()

Problem:    No test for setcmdpos().
Solution:   Add a test. (Dominique Pelle, closes #2901)

patch 8.0.1815: crash with terminal window and with 'lazyredraw' set

Problem:    Still a crash with terminal window and with 'lazyredraw' set.
            (Antoine)
Solution:   Do not wipe out the buffer when updating the screen.

patch 8.0.1814: crash with terminal window and with 'lazyredraw' set

Problem:    Crash with terminal window and with 'lazyredraw' set. (Antoine)
Solution:   Check the terminal still exists after update_screen().

patch 8.0.1813: Windows installer doesn't install terminal debugger

Problem:    Windows installer doesn't install terminal debugger.
Solution:   Add the package to the list of files to install.

patch 8.0.1812: the qf_jump_to_usable_window() function is too long

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

patch 8.0.1811: no test for winrestcmd()

Problem:    No test for winrestcmd().
Solution:   Add a test. (Dominique Pelle, closes #2894)

patch 8.0.1810: buffer of a terminal only updated in Terminal-Normal mode

Problem:    Buffer of a terminal only updated in Terminal-Normal mode.
Solution:   Copy the terminal window content to the buffer when in
            Terminal-Job mode.

patch 8.0.1809: various typos

Problem:    Various typos.
Solution:   Correct the mistakes, change "cursur" to "cursor". (closes #2887)

patch 8.0.1808: can't build without TGETENT

Problem:    Can't build without TGETENT.
Solution:   Add #ifdef

patch 8.0.1807: function to set terminal name is too long

Problem:    Function to set terminal name is too long.
Solution:   Refactor the function.  Fix typo in test.

patch 8.0.1806: InsertCharPre causes problems for autocomplete

Problem:    InsertCharPre causes problems for autocomplete. (Lifepillar)
Solution:   Check for InsertCharPre before calling vpeekc(). (Christian
            Brabandt, closes #2876)

patch 8.0.1805: qf_parse_line() is too long

Problem:    qf_parse_line() is too long.
Solution:   Split it in parts.  Properly handle vim_realloc() failing.
            (Yegappan Lakshmanan, closes #2881)

patch 8.0.1804: using :normal in terminal window causes problems

Problem:    Using :normal in terminal window causes problems. (Dominique
            Pelle)
Solution:   Don't call terminal_loop() for :normal. (closes #2886)

patch 8.0.1803: warning for uninitialized variable

Problem:    Warning for uninitialized variable. (Tony Mechelynck)
Solution:   Initialize it.

patch 8.0.1802: MS-Windows: terminal test fails

Problem:    MS-Windows: terminal test fails.
Solution:   Close redirected output file earlier.

patch 8.0.1801: MS-Windows: redirecting terminal output does not work

Problem:    MS-Windows: redirecting terminal output does not work.
Solution:   Intercept the text written to the terminal and write it to the
            file.

patch 8.0.1800: X11: getting color is slow

Problem:    X11: getting color is slow.
Solution:   Avoid using sprintf() and XParseColor(), put the RGB values in
            XColor directly.

Update runtime files.

patch 8.0.1799: no test for :registers command

Problem:    No test for :registers command.
Solution:   Add a test. (Dominique Pelle, closes #2880)

patch 8.0.1798: MS-Windows: file considered read-only too often

Problem:    MS-Windows: file considered read-only when another program has
            opened it.
Solution:   Pass file sharing flag to CreateFile(). (Linwei, closes #2860)

patch 8.0.1797: terminal window is redrawn too often

Problem:    Terminal window is redrawn too often and scrolling is repeated.
Solution:   Don't scroll immediately but only when redrawing.  Avoid redrawing
            the whole terminal window on every change.

patch 8.0.1796: GUI: click on tab fails when the focus is in a terminal

Problem:    GUI: click on tab fails when the focus is in a terminal window.
Solution:   Handle K_TABLINE.

patch 8.0.1795: lose contact with jobs when :gui forks

Problem:    Lose contact with jobs when :gui forks.
Solution:   Don't fork when there is a running job. Make log message for a
            died job clearer.  Also close the terminal when stderr and stdout
            are the same FD.

patch 8.0.1794: duplicate term options after renaming

Problem:    Duplicate term options after renaming.
Solution:   Remove the old names 'termkey', 'termsize' and 'terminalscroll'.

patch 8.0.1793: no test for "vim -g"

Problem:    No test for "vim -g".
Solution:   Add a test for "-g" and "-y".

patch 8.0.1792: MS-Windows users expect -? to work like --help

Problem:    MS-Windows users expect -? to work like --help.
Solution:   Add -?. (Christian Brabandt, closes #2867)

patch 8.0.1791: using uint8_t does not work everywhere

Problem:    Using uint8_t does not work everywhere.
Solution:   Use char_u instead.

patch 8.0.1790: 'winfixwidth' is not always respected by :close

Problem:    'winfixwidth' is not always respected by :close.
Solution:   Prefer a frame without 'winfixwidth' or 'winfixheight'. (Jason
            Franklin)

patch 8.0.1789: BufWinEnter does not work well for a terminal window

Problem:    BufWinEnter does not work well for a terminal window.
Solution:   Do not trigger BufWinEnter when opening a terminal window.

patch 8.0.1788: tool to check a color scheme is not installed

Problem:    Tool to check a color scheme is not installed.
Solution:   Update the install rule. (Christian Brabandt)

patch 8.0.1787: cannot insert the whole cursor line

Problem:    Cannot insert the whole cursor line.
Solution:   Make CTRL-R CTRL-L work. (Andy Massimino, closes #2857)

patch 8.0.1786: no test for 'termwinkey'

Problem:    No test for 'termwinkey'.
Solution:   Add a test.  Make feedkeys() handle terminal_loop() returning
            before characters are consumed.

patch 8.0.1785: missing symbol in Win32 small build

Problem:    Missing symbol in Win32 small build.
Solution:   Define VTERM_ANSI_INDEX_NONE without the terminal feature. Also
            fix unused function with #ifdef.

patch 8.0.1784: gvim test gets stuck in dialog

Problem:    Gvim test gets stuck in dialog.
Solution:   Rename the file used.

patch 8.0.1783: cannot use 256 colors in a MS-Windows console

Problem:    Cannot use 256 colors in a MS-Windows console.
Solution:   Add 256 color support. (Nobuhiro Takasaki, closes #2821)

patch 8.0.1782: no simple way to label quickfix entries

Problem:    No simple way to label quickfix entries.
Solution:   Add the "module" item, to be used instead of the file name for
            display purposes. (Martin Szamotulski, closes #1757)

patch 8.0.1781: file names in quickfix window are not shortened

Problem:    File names in quickfix window are not always shortened.
Solution:   Shorten the file name when opening the quickfix window. (Yegappan
            Lakshmanan, closes #2851, closes #2846)

Update runtime files and translations

patch 8.0.1780: test fails because Vim in a terminal uses wrong 'encoding'

Problem:    Test fails because Vim in a terminal uses wrong 'encoding'.
Solution:   Set encoding in the test where it matters. (James McCoy,
            closes #2847)

patch 8.0.1779: deleting in a block selection causes problems

Problem:    Deleting in a block selection causes problems.
Solution:   Check the length of the line before adding bd.textcol and
            bd.textlen. (Christian Brabandt, closes #2825)

patch 8.0.1778: script to check translations does not always work

Problem:    Script to check translations does not always work.
Solution:   Go to first line before searching for MIME.

patch 8.0.1777: cannot cleanup before loading another colorscheme

Problem:    Cannot cleanup before loading another colorscheme.
Solution:   Add the ColorSchemePre autocommand event.

patch 8.0.1776: in tests, when WaitFor() fails it doesn't say why

Problem:    In tests, when WaitFor() fails it doesn't say why.
Solution:   Turn a few more WaitFor() into WaitForAssert().

patch 8.0.1775: MS-Windows: warning for unused variable

Problem:    MS-Windows: warning for unused variable.
Solution:   Move declaration inside #ifdef. (Mike Williams)

patch 8.0.1774: reading very long lines can be slow

Problem:    Reading very long lines can be slow.
Solution:   Read up to 1 Mbyte at a time to avoid a lot of copying.  Add a
            check for going over the column limit.

patch 8.0.1773: dialog messages are not translated

Problem:    Dialog messages are not translated.
Solution:   Add N_() and _() where needed. (Sergey Alyoshin)

patch 8.0.1772: quickfix: mixup of FALSE and FAIL, returning -1

Problem:    Quickfix: mixup of FALSE and FAIL, returning -1.
Solution:   Use FAIL and INVALID_QFIDX. (Yegappan Lakshmanan)

patch 8.0.1771: in tests, when WaitFor() fails it doesn't say why

Problem:    In tests, when WaitFor() fails it doesn't say why. (James McCoy)
Solution:   Add WaitForAssert(), which produces an assert error when it fails.

patch 8.0.1770: assert functions don't return anything

Problem:    Assert functions don't return anything.
Solution:   Return non-zero when the assertion fails.

patch 8.0.1769: repeated saving and restoring viewstate for 'incsearch'

Problem:    Repeated saving and restoring viewstate for 'incsearch'.
Solution:   Use a structure.

patch 8.0.1768: SET_NO_HLSEARCH() used in a wrong way

Problem:    SET_NO_HLSEARCH() used in a wrong way.
Solution:   Make it a function. (suggested by Dominique Pelle,
            closes #2850)

patch 8.0.1767: with 'incsearch' text may jump up and down

Problem:    With 'incsearch' text may jump up and down. ()
Solution:   Besides w_botline also save and restore w_empty_rows.
            (closes # 2530)

patch 8.0.1766: expanding abbreviation doesn't work

Problem:    Expanding abbreviation doesn't work. (Tooth Pik)
Solution:   Return OK instead of FALSE and FAIL instead of TRUE. (Christian
            Brabandt)

patch 8.0.1765: CTRL-G j in Insert mode is incorrect when 'virtualedit' set

Problem:    CTRL-G j in Insert mode is incorrect when 'virtualedit' is set.
Solution:   Take coladd into account. (Christian Brabandt, closes #2743)

patch 8.0.1764: lgtm considers tutor.es to be EcmaScript

Problem:    Lgtm considers tutor.es to be EcmaScript.
Solution:   Add a config file for lgtm. (Bas van Schaik, closes #2844)

patch 8.0.1763: :argedit does not reuse an empty unnamed buffer

Problem:    :argedit does not reuse an empty unnamed buffer.
Solution:   Add the BLN_CURBUF flag and fix all the side effects. (Christian
            Brabandt, closes #2713)

patch 8.0.1762: terminal debug logging is a bit complicated

Problem:    Terminal debug logging is a bit complicated.
Solution:   Make log_tr() use variable arguments (Ozaki Kiichi, closes #2730)

patch 8.0.1761: job in terminal window with no output channel is killed

Problem:    Job in terminal window with no output channel is killed.
Solution:   Keep the job running when the input is a tty. (Ozaki Kiichi,
            closes #2734)

patch 8.0.1760: wrong number of arguments to vms_read()

Problem:    Wrong number of arguments to vms_read().
Solution:   Drop the first argument. (Ozaki Kiichi)

patch 8.0.1759: memory leak from duplicate options

Problem:    Memory leak from duplicate options. (Yegappan Lakshmanan)
Solution:   Don't set the default value twice.

patch 8.0.1758: open_line() returns TRUE/FALSE for success/failure

Problem:    open_line() returns TRUE/FALSE for success/failure.
Solution:   Return OK or FAIL.

patch 8.0.1757: unnecessary changes in libvterm

Problem:    Unnecessary changes in libvterm.
Solution:   Bring back // comments and trailing comma in enums.

patch 8.0.1756: GUI: after prompting for a number the mouse shape is wrong

Problem:    GUI: after prompting for a number the mouse shape is sometimes
            wrong.
Solution:   Call setmouse() after setting "State". (Hirohito Higashi,
            closes #2709)

patch 8.0.1755: MS-Windows: high unicode char received as two utf-16 words

Problem:    MS-Windows GUI: high unicode char received as two utf-16 words.
Solution:   Keep the first word until the second word is received. (Chris
            Morgan, closes #2800)

patch 8.0.1754: ex_helpgrep() is too long

Problem:    ex_helpgrep() is too long.
Solution:   Refactor the function. (Yegappan Lakshmanan, closes #2766)