patch 8.2.0259: terminal in popup test sometimes fails

Problem:    Terminal in popup test sometimes fails.
Solution:   Clear the command line.

patch 8.2.0258: modifyOtherKeys cannot be temporarily disabled

Problem:    ModifyOtherKeys cannot be temporarily disabled.
Solution:   Add echoraw() with an example for modifyOtherKeys.

patch 8.2.0257: cannot recognize a terminal in a popup window

Problem:    Cannot recognize a terminal in a popup window.
Solution:   Add the win_gettype() function.

patch 8.2.0256: time and timer related code is spread out

Problem:    Time and timer related code is spread out.
Solution:   Move time and timer related code to a new file. (Yegappan
            Lakshmanan, closes #5604)

patch 8.2.0255: VMS: missing files in build

Problem:    VMS: missing files in build.
Solution:   Add the files. (Zoltan Arpadffy)

patch 8.2.0254: compiler warning for checking size_t to be negative

Problem:    Compiler warning for checking size_t to be negative.
Solution:   Only check for zero. (Zoltan Arpadffy)

patch 8.2.0253: crash when using :disassamble without argument

Problem:    Crash when using :disassamble without argument. (Dhiraj Mishra)
Solution:   Check for missing argument. (Dominique Pelle, closes #5635,
            closes #5637)

patch 8.2.0252: Windows compiler warns for using size_t

Problem:    Windows compiler warns for using size_t.
Solution:   Change to int. (Mike Williams)

patch 8.2.0251: a couple of function return types can be more specific

Problem:    A couple of function return types can be more specific.
Solution:   Use a better return type. (Ken Takata, closes #5629)

patch 8.2.0250: test_clear_search_pat() is unused

Problem:    test_clear_search_pat() is unused.
Solution:   Remove the function. (Yegappan Lakshmanan, closes #5624)

patch 8.2.0249: MS-Windows: various warnings

Problem:    MS-Windows: various warnings.
Solution:   Set the charset to utf-8. Add _WIN32_WINNT and _USING_V110_SDK71_.
            (Ken Takata, closes #5625)

patch 8.2.0248: MS-Windows: dealing with deprecation is too complicated

Problem:    MS-Windows: dealing with deprecation is too complicated.
Solution:   Use io.h directly. Move _CRT_SECURE_NO_DEPRECATE to the build
            file. Suppress C4091 warning by setting "_WIN32_WINNT". (Ken
            Takata, closes #5626)

patch 8.2.0247: misleading comment in NSIS installer script

Problem:    Misleading comment in NSIS installer script.
Solution:   Negate the meaning of the comment. (Ken Takata, closes #5627)

patch 8.2.0246: MSVC: deprecation warnings with Ruby

Problem:    MSVC: deprecation warnings with Ruby.
Solution:   Move _CRT_SECURE_NO_DEPRECATE to build file. (Ken Takata,
            closes #5622)

patch 8.2.0245: MSVC: error message if the auto directory already exists

Problem:    MSVC: error message if the auto directory already exists.
Solution:   Add "if not exists". (Ken Takata, closes #5620)

patch 8.2.0244: compiler warning in Lua interface

Problem:    Compiler warning in Lua interface.
Solution:   Add type cast. (Ken Takata, closes #5621)

patch 8.2.0243: insufficient code coverage for ex_docmd.c functions

Problem:    Insufficient code coverage for ex_docmd.c functions.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5618)

patch 8.2.0242: preview popup window test fails with long directory name

Problem:    Preview popup window test fails with long directory name. (Jakub
            Kądziołka)
Solution:   Use "silent cd". (closes #5615)

patch 8.2.0241: crash when setting 'buftype' to "quickfix"

Problem:    Crash when setting 'buftype' to "quickfix".
Solution:   Check that error list is not NULL. (closes #5613)

patch 8.2.0240: using memory after it was freed

Problem:    Using memory after it was freed. (Dominique Pelle)
Solution:   Do not mix converion buffer with other buffer.

patch 8.2.0239: MS-Windows: 'env' job option does not override existing vars

Problem:    MS-Windows: 'env' job option does not override existing
            environment variables.  (Tim Pope)
Solution:   Set the environment variables later. (Yasuhiro Matsumoto,
            closes #5485, closes #5608)

patch 8.2.0238: MS-Windows: job_stop() results in exit value zero

Problem:    MS-Windows: job_stop() results in exit value zero.
Solution:   Call TerminateJobObject() with -1 instead of 0. (Yasuhiro
            Matsumoto, closes #5150, closes #5614)

patch 8.2.0237: crash when setting 'wincolor' on finished terminal window

Problem:    Crash when setting 'wincolor' on finished terminal window.
            (Bakudankun)
Solution:   Check that the vterm is not NULL. (Yasuhiro Matsumoto, closes
            #5607, closes #5610)

patch 8.2.0236: MS-Windows unintall doesn't delete vimtutur.bat

Problem:    MS-Windows unintall doesn't delete vimtutur.bat.
Solution:   Change directory before deletion. (Ken Takata, closes #5603)

patch 8.2.0235: draw error when an empty group is removed from 'statusline'

Problem:    Draw error when an empty group is removed from 'statusline'.
Solution:   Do not use highlighting from a removed group.

patch 8.2.0234: message test fails on SunOS

Problem:    Message test fails on SunOS.
Solution:   Adjust expectation for printf "%p". (Ozaki Kiichi, closes #5595)

patch 8.2.0233: crash when using garbagecollect() in between rand()

Problem:    Crash when using garbagecollect() in between rand().
Solution:   Redesign the rand() and srand() implementation. (Yasuhiro
            Matsumoto, closes #5587, closes #5588)

patch 8.2.0232: the :compiler command causes a crash

Problem:    The :compiler command causes a crash. (Daniel Steinberg)
Solution:   Do not use the script index if it isn't set.

patch 8.2.0231: silent system command may clear the screen

Problem:    Silent system command may clear the screen.
Solution:   Do not clear the screen in t_te.

patch 8.2.0230: terminal popup test is flaky

Problem:    Terminal popup test is flaky.
Solution:   Increase wait time a bit.

patch 8.2.0229: compare instructions not tested

Problem:    Compare instructions not tested.
Solution:   Add test cases.  Fix disassemble with line continuation.

patch 8.2.0228: configure does not recognize gcc version on BSD

Problem:    Configure does not recognize gcc version on BSD.
Solution:   Do not use "\+" in the pattern matching the version number. (Ozaki
            Kiichi, closes #5590)

patch 8.2.0227: compiling a few instructions not tested

Problem:    Compiling a few instructions not tested.
Solution:   Add more test cases.

patch 8.2.0226: compiling for loop not tested

Problem:    Compiling for loop not tested.
Solution:   Add a test.  Make variable initialization work for more types.

patch 8.2.0225: compiling lambda not tested yet

Problem:    compiling lambda not tested yet.
Solution:   Add test for lambda and funcref. Drop unused instruction arg.

patch 8.2.0224: compiling :elseif not tested yet

Problem:    compiling :elseif not tested yet.
Solution:   Add test for :elseif.  Fix generating jumps.

patch 8.2.0223: some instructions not yet tested

Problem:    Some instructions not yet tested.
Solution:   Disassemble more instructions.  Move tests to a new file.  Compile
            call to s:function().

patch 8.2.0222: Vim9: optional function arguments don't work yet

Problem:    Vim9: optional function arguments don't work yet.
Solution:   Implement optional function arguments.

patch 8.2.0221: no test for Vim9 += and ..=

Problem:    No test for Vim9 += and ..=.
Solution:   Add tests.

patch 8.2.0220: terminal test did pass on Mac

Problem:    Terminal test did pass on Mac.
Solution:   Remove the skip again.

patch 8.2.0219: terminal test still fails on Mac

Problem:    Terminal test still fails on Mac.
Solution:   Skip part of the test on Mac.

patch 8.2.0218: several Vim9 instructions are not tested

Problem:    Several Vim9 instructions are not tested.
Solution:   Add more tests.

patch 8.2.0217: terminal test fails on Mac

Problem:    Terminal test fails on Mac.
Solution:   Add a short wait.

patch 8.2.0216: several Vim9 instructions are not tested

Problem:    Several Vim9 instructions are not tested.
Solution:   Add more tests. Fix :disassamble output. Make catch with pattern
            work.

patch 8.2.0215: wrong file name shortening

Problem:    Wrong file name shortening. (Ingo Karkat)
Solution:   Better check for path separator. (Yasuhiro Matsumoto,
            closes #5583, closes #5584)

patch 8.2.0214: a popup window with a terminal can be made hidden

Problem:    A popup window with a terminal can be made hidden.
Solution:   Disallow hiding a terminal popup.

patch 8.2.0213: configure does not recognize gcc 10.0 and later

Problem:    Configure does not recognize gcc 10.0 and later.
Solution:   Adjust the pattern matching the version number. (Sergei
            Trofimovich, closes #5580)

patch 8.2.0212: missing search/substitute pattern hardly tested

Problem:    Missing search/substitute pattern hardly tested.
Solution:   Add test_clear_search_pat() and tests. (Yegappan Lakshmanan,
            closes #5579)

patch 8.2.0211: test for ANSI colors fails without an "ls" command

Problem:    Test for ANSI colors fails without an "ls" command.
Solution:   Use "dir". (Ken Takata, closes #5582)

patch 8.2.0210: Coverity complains about uninitialized field

Problem:    Coverity complains about uninitialized field.
Solution:   Initialize the field.

Update runtime files.

patch 8.2.0209: function a bit far away from where it's used

Problem:    Function a bit far away from where it's used.
Solution:   Move function close to where it's used. (Ken Takata, closes #5569)

patch 8.2.0208: fnamemodify() does not apply ":~" when followed by ":."

Problem:    Fnamemodify() does not apply ":~" when followed by ":.".
Solution:   Don't let a failing ":." cause the ":~" to be skipped. (Yasuhiro
            Matsumoto, closes #5577)

patch 8.2.0207: crash when missing member type on list argument

Problem:    Crash when missing member type on list argument.
Solution:   Check for invalid type. (closes #5572)

patch 8.2.0206: calling Vim9 function using default argument fails

Problem:    Calling Vim9 function using default argument fails.
Solution:   Give an appropriate error. (closes #5572)

patch 8.2.0205: error code E899 used twice

Problem:    Error code E899 used twice.
Solution:   Use E863 for the terminal in popup error.

patch 8.2.0204: crash when using winnr('j') in a popup window

Problem:    Crash when using winnr('j') in a popup window.
Solution:   Do not search for neighbors in a popup window. (closes #5568)

patch 8.2.0203: :helptags and some other functionality not tested

Problem:    :helptags and some other functionality not tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5567)

patch 8.2.0202: when 'lazyredraw' is set the window title may not be updated

Problem:    When 'lazyredraw' is set the window title may not be updated.
Solution:   Set "do_redraw" before entering the main loop. (Jason Franklin)

patch 8.2.0201: cannot assign to an imported variable

Problem:    Cannot assign to an imported variable.
Solution:   Make it work.

patch 8.2.0200: Vim9 script commands not sufficiently tested

Problem:    Vim9 script commands not sufficiently tested.
Solution:   Add more tests.  Fix storing global variable.  Make script
            variables work.

patch 8.2.0199: Vim9 script commands not sufficiently tested

Problem:    Vim9 script commands not sufficiently tested.
Solution:   Add more tests.  Fix script-local function use.

patch 8.2.0198: no tests for y/n prompt

Problem:    No tests for y/n prompt.
Solution:   Add tests. (Dominique Pelle, closes #5564)

patch 8.2.0197: some Ex commands not sufficiently tested

Problem:    Some Ex commands not sufficiently tested.
Solution:   Add more tests. (Yegappan Lakshmanan, closes #5565)

patch 8.2.0196: blocking commands for a finished job in a popup window

Problem:    Blocking commands for a finished job in a popup window.
Solution:   Do not block commands if the job has finished.  Adjust test.

patch 8.2.0195: some tests fail when run in the GUI

Problem:    Some tests fail when run in the GUI.
Solution:   Make sure the window width is enough.  In the GUI run terminal Vim
            in the terminal, if possible.

patch 8.2.0194: some commands can cause problems in terminal popup

Problem:    Some commands can cause problems in terminal popup.
Solution:   Disallow more commands.

patch 8.2.0193: still build failure without +terminal feature

Problem:    Still build failure without +terminal feature.
Solution:   Add more #ifdefs.

patch 8.2.0192: build failure without +terminal feature

Problem:    Build failure without +terminal feature.
Solution:   Add #ifdefs.

patch 8.2.0191: cannot put a terminal in a popup window

Problem:    Cannot put a terminal in a popup window.
Solution:   Allow opening a terminal in a popup window.  It will always have
            keyboard focus until closed.

patch 8.2.0190: Kotlin files are not recognized

Problem:    Kotlin files are not recognized.
Solution:   Detect Kotlin files. (Alkeryn, closes #5560)

patch 8.2.0189: cd() with NULL argument crashes

Problem:    cd() with NULL argument crashes.
Solution:   Check for NULL. (Ken Takata, closes #5558)

patch 8.2.0188: Check commands don't work well with Vim9 script

Problem:    Check commands don't work well with Vim9 script.
Solution:   Improve constant expression handling.

patch 8.2.0187: reduntant code

Problem:    Reduntant code.
Solution:   Remove unused assignments. (Dominique Pelle, closes #5557)

patch 8.2.0186: a couple of tests may fail when features are missing

Problem:    A couple of tests may fail when features are missing.
Solution:   Check for features. (Dominique Pelle, closes #5561)

patch 8.2.0185: Vim9 script: cannot use "if has()" to skip lines

Problem:    Vim9 script: cannot use "if has()" to skip lines.
Solution:   Evaluate constant expression at runtime.

patch 8.2.0184: blob test fails

Problem:    Blob test fails.
Solution:   Check for different error when float feature is missing.

patch 8.2.0183: tests fail when the float feature is disabled

Problem:    Tests fail when the float feature is disabled.
Solution:   Skip tests that don't work without float support.

patch 8.2.0182: min() and max() materialize a range() list

Problem:    Min() and max() materialize a range() list.
Solution:   Compute the result without materializing the list. (#5541)

patch 8.2.0181: problems parsing :term arguments

Problem:    Problems parsing :term arguments.
Solution:   Improve parsing, fix memory leak, add tests. (Ozaki Kiichi,
            closes #5536)

patch 8.2.0180: test for wrapmargin fails if terminal is not 80 columns

Problem:    Test for wrapmargin fails if terminal is not 80 columns.
Solution:   Vertical split the window. (Ken Takata, closes #5554)

patch 8.2.0179: still a few places where range() does not work

Problem:    Still a few places where range() does not work.
Solution:   Fix using range() causing problems.

patch 8.2.0178: with VTP the screen may not be restored properly

Problem:    With VTP the screen may not be restored properly.
Solution:   Add another set of saved RGB values. (Nobuhiro Takasaki,
            closes #5548)

patch 8.2.0177: memory leak in get_tags()

Problem:    Memory leak in get_tags().
Solution:   Free matches when finding a pseudo-tag line. (Dominique Pelle,
            closes #5553)

patch 8.2.0176: generating os headers does not work for Swedish

Problem:    Generating os headers does not work for Swedish.
Solution:   Set the locale to C. (Christian Brabandt, closes #5258)

patch 8.2.0175: crash when removing list element in map()

Problem:    Crash when removing list element in map().
Solution:   Lock the list. (closes #2652)

patch 8.2.0174: various commands not completely tested

Problem:    Various commands not completely tested.
Solution:   Add more test cases. (Yegappan Lakshmanan, closes #5551)

patch 8.2.0173: build fails with old compiler

Problem:    Build fails with old compiler.
Solution:   Do not use anonymous unions. (John Marriott)

patch 8.2.0172: Coverity warning for not restoring character

Problem:    Coverity warning for not restoring character.
Solution:   Restore the character also in case of failure.

patch 8.2.0171: Coverity warning for using uninitialized buffer

Problem:    Coverity warning for using uninitialized buffer.
Solution:   Check the skip flag.

patch 8.2.0170: Coverity warning for ignoring return value

Problem:    Coverity warning for ignoring return value.
Solution:   Check the return value and return if failed.

patch 8.2.0169: Coverity warning for dead code

Problem:    Coverity warning for dead code.
Solution:   Check if inside try-finally.

patch 8.2.0168: Coverity warning for assigning NULL to an option

Problem:    Coverity warning for assigning NULL to an option.
Solution:   Use empty string instead of NULL.

patch 8.2.0167: Coverity warning for ignoring return value

Problem:    Coverity warning for ignoring return value.
Solution:   Check the return value and jump if failed.

patch 8.2.0166: Coverity warning for using uninitialized variable

Problem:    Coverity warning for using uninitialized variable.
Solution:   Check for failure.

patch 8.2.0165: Coverity warning for using NULL pointer

Problem:    Coverity warning for using NULL pointer.
Solution:   Add missing "else".

patch 8.2.0164: test_alot takes too long

Problem:    Test_alot takes too long.
Solution:   Run several tests individually.

patch 8.2.0163: test hangs on MS-Windows console

Problem:    Test hangs on MS-Windows console.
Solution:   use feedkeys() instead of test_feedinput(). (Ken Takata)

patch 8.2.0162: balloon test fails in the GUI

Problem:    Balloon test fails in the GUI.
Solution:   Skip test in the GUI.

patch 8.2.0161: not recognizing .gv file as dot filetype

Problem:    Not recognizing .gv file as dot filetype.
Solution:   Add *.gv to dot pattern. (closes #5544)