Update to Vim 8.1.1486 #901
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Problem: Compiler warnings for use of STRNCPY(). (John Marriott) Solution: Use mch_memmove() instead of STRNCPY().
Problem: Click on WinBar of other window not tested. Solution: Add a test case.
Problem: Compiler warning in direct write code. Solution: Add a type cast.
Problem: After :unmenu can still execute the menu with :emenu. Solution: Do not execute a menu that was disabled for the specified mode.
Problem: Cannot navigate to errors before/after the cursor.
Solution: Add the :cbefore and :cafter commands. (Yegappan Lakshmanan,
closes #4340)
Problem: Cannot combine text properties with syntax highlighting. Solution: Add the "combine" field to prop_type_add(). (closes #4343)
Problem: Missing screenshot update. Solution: Update the screenshot.
Problem: Missing change for "combine" field. Solution: Also change the textprop implementation.
Problem: Cannot set 'spellang' to "sr@latin". (Bojan Stipic) Solution: Allow using '@' in 'spellang'. (closes #4342)
Problem: Remarks about functionality not in Vi clutters the help.
Solution: Move all info about what is new in Vim or already existed in Vi to
vi_diff.txt. Remove {not in Vi} remarks. (closes #4268) Add
"noet" to the help files modeline. Also include many other help
file improvements.
Problem: Cannot specify a count with :chistory.
Solution: Add a count to :chistory and :lhistory. (Yegappan Lakshmanan,
closes #4344)
Problem: Running make in src/po leaves LINGUAS file behind. (Ken Takata) Solution: Delete LINGUAS after running msgfmt.
Problem: Delaying half a second after the top-bot message. Solution: Instead of the delay add "W" to the search count.
Problem: Detecting *.tmpl as htmlcheetah is outdated. Solution: Use the generic name "template". (closes #4348)
Problem: Test17 is old style. Solution: Turn into new style test. (Yegappan Lakshmanan, closes #4347)
Problem: Running tests leaves XTest_tabpage_cmdheight file behind. Solution: Delete the right file. (closes #4350)
Problem: Cannot build with +eval but without +mouse. Solution: Add #ifdefs around f_test_setmouse(). (John Marriott)
Problem: Search stats don't show for mapped command.
Solution: Remove SEARCH_PEEK from searchit flags. Add a test. (Christian
Brabandt)
Problem: May not have enough space to add "W" to search stats. Solution: Reserve a bit more space. (Christian Brabandt)
… git
Problem: .hgignore and .gitignore are either distributed or in git, not
both.
Solution: Add .gitignore to the distribution and .hgignore to git. Update
the entries. (Christian Brabandt, Ken Takata)
Problem: Not easy to change directory and restore. Solution: Add the chdir() function. (Yegappan Lakshmanan, closes #4358)
Problem: Invalid command line arguments not tested. Solution: Add a test. (Dominique Pelle, closes #4346)
Problem: MSVC files are no longer useful for debugging. Newer Visual
Studio versions cannot read them.
Solution: Delete the files. (Ken Takata, closes #4357)
Problem: MS-Windows: Some fonts return wrong average char width. Solution: Compute the average ourselves. (Ken Takata, closes #4356)
Problem: When vimrun.exe does not exist external command may fail.
Solution: Use "cmd /c" twice to get the same behavior. (Ken Takata,
closes #4355)
Problem: Crash when using invalid command line argument. Solution: Check for options not being initialized.
Problem: Invalid argument test fails without GTK. Solution: Test -display and --display separately.
Problem: Invalid argument test fails without X clipboard. Solution: Test -display only with the +xterm_clipboard feature.
Problem: "extends" from 'listchars' is used when 'list' is off. (Hiroyuki
Yoshinaga)
Solution: Only use the "extends" character when 'list' is on. (Hirohito
Higashi, closes #4360)
Problem: In a terminal 'ballooneval' does not work right away.
Solution: Flush output after drawing the balloon. Add the <Ignore> key
code. Add a test.
Problem: MS-Windows: using special character requires quoting. Solution: Add quotes. (Ken Takata)
Problem: Gcc warns for uninitialized variable. Solution: Put usage inside "if". (Ken Takata)
Problem: Only 4-digit rgb termresponse is recognized. Solution: Also recognize 2-digit rgb response. (closes #4486)
Problem: Allocating wrong amount of memory. (Yegappan Lakshmanan) Solution: Use sizeof() for right type of struct.
Problem: Not updating priority on existing sign. Solution: Set the sign priority. Add a test. (Yegappan Lakshmanan)
Problem: Cscope test fails. Solution: Update expected text.
Problem: The generated desktop files may be invalid.
Solution: Check validity with desktop-file-validate. (Christian Brabandt,
Will Thompson, closes #4480)
Problem: No test for checking the cursor style response. Solution: Add a simple test. Also include the missing part of 8.1.1464.
… table Problem: New Unicode character U32FF missing from double-width table. Solution: Add the character.
…sponse Problem: 'background' not correctly set for 2-digit rgb termresponse. Solution: Adjust what digit to use. (closes #4495)
Problem: Add_termcap_entry() is not tested. Solution: Add a simple test.
Problem: New resolve() implementation causes problem for plugins.
Solution: Only resolve a resparse point after checking it is needed. (Ken
Takata, closes #4492)
Problem: 'ttybuiltin' is not tested. Solution: At least test that it doesn't break things.
Problem: Search string not displayed when 'rightleft' is set. Solution: Clear the right part of the old text. (closes #4488, closes #4489)
Problem: No statistics displayed after running tests.
Solution: Summarize the test results. (Christian Brabandt, closes #4391)
Also make it possible to report a skipped file.
Problem: Test summary fails in the tiny version. Solution: set 'nocompatible'.
Problem: Still an error when running tests with the tiny version. Solution: Do not try reading test.log
Problem: Change included for debugging only. Solution: Restore the REDIR_TEST_TO_NULL line.
Problem: Desktop file check doesn't run on CI.
Solution: Install the desktip-file-utils packages. (Christian Brabandt,
closes #4498)
Problem: Length for two-digit rgb termresponse is off by one. Solution: Adjust the length. (closes #4494)
Problem: No test for wincol() depending on the 'number' option. Solution: Add a couple of tests. (Christian Brabandt, closes #4500)
Problem: Skipped tests are not properly listed. Solution: Throw a "Skipped" exception instead of using ":finish" or ":return".
Problem: Some tests are slow.
Solution: Add timing to the test messages. Fix double free when quitting in
VimLeavePre autocmd.
…mand Problem: Double free when garbage_collect() is used in autocommand. Solution: Have garbage collection also set the copyID in funccal_stack.
Problem: A listener change is merged even when it adds a line. (Paul Jolly) Solution: Do not merge a change that adds or removes a line. (closes #4490)
|
Sorry but for merging from master I prefer to resolve them myself so I can tell what’s going in. There are some balloon testing issues that is blocking this but I should just turn off that test. I’ll merge this weekend. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To resolve issues #900 and #898.