aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | vim-patch:8.2.0882: leaking memory when using reduce()Sean Dewar2022-01-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Leaking memory when using reduce(). Solution: Free the intermediate value. https://github.com/vim/vim/commit/48b1c21809553d3463b5ed6c2b3bc6d335663bb6
| * | | | | | feat(eval): partially port v8.2.0878Sean Dewar2022-01-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No reduce() function. Solution: Add a reduce() function. (closes vim/vim#5481) https://github.com/vim/vim/commit/85629985b71035608a37ba3bde86968481490d46 Needs CHECK_LIST_MATERIALIZE from v8.2.0751 (and range_list_materialize from 8.2.0149). Move e_reduceempty to funcs.c, as it's only used there. Make it static. Use tv_blob_len, tv_list_len == 0 for empty checks. Replace vim_memset(&funcexe, 0, ...) with FUNCEXE_INIT. Leave li initially undefined (tv_list_first returns NULL if list is NULL). This patch has a memory leak fixed by v8.2.0882.
* | | | | | | Merge pull request #17334 from zeertzjq/vim-8.2.4326zeertzjq2022-02-08
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | vim-patch:8.2.4326: "o" and "O" copying comment not sufficiently tested
| * | | | | | vim-patch:8.2.4326: "o" and "O" copying comment not sufficiently testedzeertzjq2022-02-08
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: "o" and "O" copying comment not sufficiently tested. Solution: Add a test case. (closes vim/vim#9718) https://github.com/vim/vim/commit/51ab7c7d0da08aac796acff22a6c075dac579e76 Fix a mistake when porting Vim patch 8.2.3934
* | | | | | Merge pull request #17327 from zeertzjq/test-ex-modezeertzjq2022-02-08
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | test: add Lua functional tests for Ex mode
| * | | | | test: add Lua functional tests for Ex modezeertzjq2022-02-08
|/ / / / /
* | | | | Merge pull request #17324 from zeertzjq/vim-patch-sh-readJames McCoy2022-02-07
|\ \ \ \ \ | | | | | | | | | | | | chore(vim-patch.sh): use piping instead of here string for `while read`
| * | | | | chore(vim-patch.sh): use piping instead of here string for `while read`zeertzjq2022-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using a here string can cause an error if there are no missing patches: `./scripts/vim-patch.sh: line 580: runtime_commits: bad array subscript` Using piping doesn't cause the error.
* | | | | | Merge pull request #17323 from dundargoc/ci/add-reviewer-from-labelJames McCoy2022-02-07
|\ \ \ \ \ \ | |/ / / / / |/| | | | | ci: add reviewers based on label
| * | | | | ci: add reviewers based on labelDundar Göc2022-02-07
| | |/ / / | |/| | |
* | | | | Merge pull request #17311 from zeertzjq/vim-8.2.4303zeertzjq2022-02-08
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.2.{4303,4307}: a few messages should not be translated
| * | | | | vim-patch:8.2.4307: a few more messages should not be translatedzeertzjq2022-02-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: A few more messages should not be translated. Solution: Remove _(). https://github.com/vim/vim/commit/0c1550d9e94046d3fc9a8ad70b895eaa1e53fca5
| * | | | | vim-patch:8.2.4303: a few messages should not be translatedzeertzjq2022-02-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: A few messages should not be translated. Solution: Remove _(). (Dominique Pellé, closes vim/vim#9702) https://github.com/vim/vim/commit/cd53eed2c55f2256008962965b1de1d1df76d535
* | | | | | chore: fix typos (#17250)dundargoc2022-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com> Co-authored-by: Dani Dickstein <daniel.dickstein@gmail.com> Co-authored-by: Axel Dahlberg <git@valleymnt.com>
* | | | | | Merge pull request #16873 from seandewar/vim-8.2.1665Sean Dewar2022-02-07
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | vim-patch:8.2.{1665,1726,1872,1893,1921,2280,2813}: `matchfuzzy` and friends
| * | | | | docs(fuzzy-match): cherry-pick latest changesSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/vim/vim/commit/53f7fccc9413c9f770694b56f40f242d383b2d5f https://github.com/vim/vim/commit/1b884a0053982335f644eec6c71027706bf3c522 https://github.com/vim/vim/commit/4c295027a426986566cd7a76c47a6d3a529727e7 https://github.com/vim/vim/commit/3ec3217f0491e9ba8aa8ea02f7e454cd19a287ef
| * | | | | vim-patch:8.2.2813: cannot grep using fuzzy matchingSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot grep using fuzzy matching. Solution: Add the "f" flag to :vimgrep. (Yegappan Lakshmanan, closes vim/vim#8152) https://github.com/vim/vim/commit/bb01a1ef3a093cdb36877ba73474719c531dc8cb
| * | | | | vim-patch:8.2.2280: fuzzy matching doesn't give access to the scoresSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Fuzzy matching doesn't give access to the scores. Solution: Return the scores with a third list. (Yegappan Lakshmanan, closes vim/vim#7596) https://github.com/vim/vim/commit/9d19e4f4ba55f8bef18d4991abdf740ff6472dba Remove seemingly useless NULL checks. assert that removing the li one wasn't dumb.
| * | | | | vim-patch:8.2.1921: fuzzy matching does not recognize path separatorsSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Fuzzy matching does not recognize path separators. Solution: Add a bonus for slash and backslash. (Yegappan Lakshmanan, closes vim/vim#7225) https://github.com/vim/vim/commit/dcdd42a8ccb9bafd857735d694b074269f337333
| * | | | | vim-patch:8.2.1893: fuzzy matching does not support multiple wordsSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Fuzzy matching does not support multiple words. Solution: Add support for matching white space separated words. (Yegappan Lakshmanan, closes vim/vim#7163) https://github.com/vim/vim/commit/8ded5b647aa4b3338da721b343e0bce0f86655f6
| * | | | | vim-patch:8.2.1872: matchfuzzy() does not prefer sequential matchesSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Matchfuzzy() does not prefer sequential matches. Solution: Give sequential matches a higher bonus. (Christian Brabandt, closes vim/vim#7140) https://github.com/vim/vim/commit/e9f9f16387554e5b34ba42ce00c42c28dd66af58
| * | | | | vim-patch:8.2.1726: fuzzy matching only works on stringsSean Dewar2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Fuzzy matching only works on strings. Solution: Support passing a dict. Add matchfuzzypos() to also get the match positions. (Yegappan Lakshmanan, closes vim/vim#6947) https://github.com/vim/vim/commit/4f73b8e9cc83f647b34002554a8bdf9abec0a82f Also remove some N/A and seemingly useless NULL checks -- Nvim allocs can't return NULL. I'm not sure why the retmatchpos stuff in match_fuzzy checks for NULL too, given that Vim checks for NULL alloc in do_fuzzymatch; assert that the li stuff is not NULL as that's the one check I'm ever-so-slightly unsure about. Adjust tests. Note that the text_cb tests actually throw E6000 in Nvim, but we also can't assert that error due to v8.2.1183 not being ported yet.
| * | | | | vim-patch:8.2.1665: cannot do fuzzy string matchingSean Dewar2022-02-07
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot do fuzzy string matching. Solution: Add matchfuzzy(). (Yegappan Lakshmanan, closes vim/vim#6932) https://github.com/vim/vim/commit/635414dd2f3ae7d4d972d79b806348a6516cb91a Adjust Test_matchfuzzy's 2nd assert to expect the last error thrown, as v8.2.1183 hasn't been ported yet (to be honest, the error message is kinda weird if the 2nd argument is not convertible to string). We can still port this fully as porting v8.2.1183 would require removing this change to pass CI.
* | | | | chore(typval): return NULL over false for pointer return type (#17316)Sean Dewar2022-02-07
| | | | | | | | | | | | | | | While we're at it, abort() for an unhandled v_type.
* | | | | Merge pull request #17263 from zeertzjq/vim-8.2.0208zeertzjq2022-02-07
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.2.{0208,0215,0942,3824,3939}: fnamemodify() patches
| * | | | | vim-patch:8.2.3939: MS-Windows: fnamemodify('', ':p') does not workzeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: MS-Windows: fnamemodify('', ':p') does not work. Solution: Do not consider an empty string a full path. (Yegappan Lakshmanan, closes vim/vim#9428, closes vim/vim#9427) https://github.com/vim/vim/commit/5a664fe57fe7ba65a771bc95ef1c205e4db193b7
| * | | | | vim-patch:8.2.3824: no ASAN support for MSVCzeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No ASAN support for MSVC. Solution: Add ASAN support and fix a coupld of uncovered problems. (Yegappan Lakshmanan, closes vim/vim#9357) https://github.com/vim/vim/commit/6df0f2759d12ec0bc999b2963ecea4387a2bc516
| * | | | | vim-patch:8.2.0942: expanding to local dir after homedir keeps "~/"zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Expanding to local dir after homedir keeps "~/". Solution: Adjust modify_fname(). (Christian Brabandt, closes vim/vim#6205, closes vim/vim#5979) https://github.com/vim/vim/commit/0e390f40e944036fb558a63b91238cfda128d95f
| * | | | | vim-patch:8.2.0215: wrong file name shorteningzeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Wrong file name shortening. (Ingo Karkat) Solution: Better check for path separator. (Yasuhiro Matsumoto, closes vim/vim#5583, closes vim/vim#5584) https://github.com/vim/vim/commit/a78e9c61a0ded9c5302bc77e889aa1b3d3467f61
| * | | | | vim-patch:8.2.0208: fnamemodify() does not apply ":~" when followed by ":."zeertzjq2022-02-07
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Fnamemodify() does not apply ":~" when followed by ":.". Solution: Don't let a failing ":." cause the ":~" to be skipped. (Yasuhiro Matsumoto, closes vim/vim#5577) https://github.com/vim/vim/commit/d816cd94d87afb73c505bf1e5cd5e07522482113
* | | | | Merge pull request #17299 from zeertzjq/vim-8.1.0711zeertzjq2022-02-07
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:7.4.{1163,1164,1167,1173,1178,1181,1228},8.1.0711
| * | | | | vim-patch:8.1.0711: test files still use function!zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test files still use function!. Solution: Remove the exclamation mark. Fix overwriting a function. https://github.com/vim/vim/commit/1e1153600c0377472d62cc553173fe555ddcf5a7 Some of the changes were already applied previously.
| * | | | | vim-patch:7.4.1228zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: copy() and deepcopy() fail with special variables. (Nikolai Pavlov) Solution: Make it work. Add a test. Closes vim/vim#614. https://github.com/vim/vim/commit/155500077c80cdb5d9c63996000c011b66a676bf Code is N/A. This only ports the tests. Comment out tests involving v:none as Nvim has removed it.
| * | | | | vim-patch:7.4.1181zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: free_tv() can't handle special variables. (Damien) Solution: Add the variable type. https://github.com/vim/vim/commit/6650a694547eb744afa060ec62dd8270e99db9f2 Code is N/A. This only ports the tests.
| * | | | | vim-patch:7.4.1178zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: empty() doesn't work for the new special variables. Solution: Make empty() work. (Damien) https://github.com/vim/vim/commit/767d8c1a1ae762ecf47297c168b8c23caf05d30a Code is N/A. This only ports the tests. Comment out tests involving v:none as Nvim has removed it.
| * | | | | vim-patch:7.4.1173zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for new behavior of v:true et al. Solution: Add a test. https://github.com/vim/vim/commit/65591001e405cbaaf9772c9375d0bb6049cf9a3a Comment out tests involving v:none as Nvim has removed it.
| * | | | | vim-patch:7.4.1167zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No tests for "is" and "isnot" with the new variables. Solution: Add tests. https://github.com/vim/vim/commit/04369229657f182d35b471eb8b38f273a4d9ef65 Comment out tests involving v:none as Nvim has removed it.
| * | | | | vim-patch:7.4.1164zeertzjq2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No tests for comparing special variables. Error in jsondecode() not reported. test_json does not work Japanse system. Solution: Set scriptencoding. (Ken Takata) Add a few more tests. Add error. https://github.com/vim/vim/commit/6039c7f05376f0e470cf62bf2757e653aea357f3 Code is N/A. This only ports the tests. Comment out tests involving v:none as Nvim has removed it.
| * | | | | vim-patch:7.4.1163zeertzjq2022-02-07
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Expressions "0 + v:true" and "'' . v:true" cause an error. Solution: Return something sensible when using a special variable as a number or as a string. (suggested by Damien) https://github.com/vim/vim/commit/17a13437c9414a8693369a97f3be2fc8ad48c12e Code is N/A. This only ports the tests. Comment out tests involving v:none as Nvim has removed it.
* | | | | Merge pull request #17305 from zeertzjq/vim-8.2.1741zeertzjq2022-02-07
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:8.2.1741: pathshorten() only supports using one character
| * | | | | vim-patch:8.2.1741: pathshorten() only supports using one characterzeertzjq2022-02-06
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: pathshorten() only supports using one character. Solution: Add an argument to control the length. (closes vim/vim#7006) https://github.com/vim/vim/commit/6a33ef0deb5c75c003a9f3bd1c57f3ca5e77327e Cherry-pick a line in test from patch 8.2.0634. Use Nvim's config paths in docs. shorten_dir() returning a pointer looks a bit confusing here, as it is actually the same pointer passed to it, and it doesn't really reduce much code, so change it back to void. Assigning rettv->vval.v_string = NULL is not needed if a pointer is within 64 bits. While this is usually the case, I'm not sure if it can be taken for granted.
* | | | | Merge pull request #17279 from zeertzjq/state-enter-vpeekcbfredl2022-02-06
|\ \ \ \ \ | | | | | | | | | | | | fix(event-loop): call vpeekc() directly first to check for character
| * | | | | fix(event-loop): call vpeekc() directly first to check for characterzeertzjq2022-02-03
| | | | | | | | | | | | | | | | | | | | | | | | Expand mappings first by calling `vpeekc()` directly.
* | | | | | Merge pull request #17302 from lewis6991/pkg_ldrbfredl2022-02-06
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix(lua): restore priority of the preloader
| * | | | | | fix(lua): restore priority of the preloaderLewis Russell2022-02-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neovim currently places its own loader for searching runtime files at the front of `package.loaders`. This prevents any preloaders in `package.preload` from being used. This change fixes that by moving the default package preloader to run before Neovim's loader. For example, LuaJIT provides preloaders for the built-in modules `ffi` and `bit`, so this optimisation will improve the loading of those.
* | | | | | | Merge pull request #17306 from clason/vim-8.2.4305Christian Clason2022-02-06
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * Vim 8.2.4305: tex filetype detection fails * Vim 8.2.4306: no test for fixed perl filetype check
| * | | | | | | vim-patch:8.2.4306: no test for fixed perl filetype checkChristian Clason2022-02-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for fixed perl filetype check. Solution: Add a test. Sort test functions. https://github.com/vim/vim/commit/500761b1cf666f030009d2dcdacfdce28f68f43d
| * | | | | | | vim-patch:8.2.4305: tex filetype detection failsChristian Clason2022-02-06
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Tex filetype detection fails. Solution: Check value to be positive. (closes vim/vim#9704) https://github.com/vim/vim/commit/e5b7897585eccec84431d8b23df5cde2e283828c
* | | | | | | Merge pull request #17277 from laomaiweng/windows-debug-symbolsJames McCoy2022-02-06
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | build(win): package the debug symbols (PDB file) along with nvim.exe
| * | | | | | | build(win): don't try to package a PDB if not building with MSVC…Quentin Minster2022-02-03
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com>