aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* refactor(mbyte.c): add const qualifierszeertzjq2022-02-09
| | | | | This only touches functions that do not return a pointer. Also add a note about the differences between mb_head_off() and utf_head_off().
* Merge pull request #17341 from zeertzjq/clang-suppresszeertzjq2022-02-09
|\ | | | | chore(clang): suppress "result of operation is garbage"
| * chore(clang): suppress "result of operation is garbage"zeertzjq2022-02-09
|/
* Merge pull request #17332 from cryptomilk/asn-vterm-output-callbackJames McCoy2022-02-08
|\ | | | | feat(term): Use vterm_output_set_callback()
| * feat(term): use vterm_output_set_callback()Andreas Schneider2022-02-08
| |
| * chore(deps): use https for downloading libvterm and libtermkeyAndreas Schneider2022-02-08
| |
* | Merge pull request #17330 from dundargoc/ci/add-reviewer-from-labelJames McCoy2022-02-08
|\ \ | | | | | | ci: add write permissions to add-reviewer job
| * | ci: add write permissions to add-reviewer jobDundar Göc2022-02-08
| | |
* | | Merge pull request #17326 from zeertzjq/vim-8.1.2297zeertzjq2022-02-09
|\ \ \ | | | | | | | | vim-patch:8.1.2297: the ex_vimgrep() function is too long
| * | | vim-patch:8.1.2297: the ex_vimgrep() function is too longzeertzjq2022-02-09
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: The ex_vimgrep() function is too long. Solution: Split it in three parts. (Yegappan Lakshmanan, closes vim/vim#5211) https://github.com/vim/vim/commit/d6a98a3a9768568b668f91a53267b36f86b84466 Including a missing change to ex_vimgrep() from patch 8.0.1831.
* | | Merge pull request #17328 from zeertzjq/vim-8.2.0197zeertzjq2022-02-09
|\ \ \ | | | | | | | | vim-patch:8.2.0197: some Ex commands not sufficiently tested
| * | | vim-patch:8.2.0197: some Ex commands not sufficiently testedzeertzjq2022-02-08
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Some Ex commands not sufficiently tested. Solution: Add more tests. (Yegappan Lakshmanan, closes vim/vim#5565) https://github.com/vim/vim/commit/ea3db914c0fa35797ad73f6d5bb3a4288d690065
* | | | Merge pull request #16553 from seandewar/vim-8.2.0878Sean Dewar2022-02-08
|\ \ \ \ | |_|/ / |/| | | vim-patch:8.2.{0882,1051,1083}: port `reduce()` function
| * | | vim-patch:8.2.1083: crash when using reduce() on a NULL listSean Dewar2022-01-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when using reduce() on a NULL list. Solution: Only access the list when not NULL. https://github.com/vim/vim/commit/fda20c4cc59008264676a6deb6a3095ed0c248e0 CHECK_LIST_MATERIALIZE hasn't been ported yet, but presumably if it is ported it'll use tv_list_first to check for range_list_item, which already checks for NULL, so this should need no extra changes and can be a full port. We didn't actually crash here due to the use of Nvim's tv_list functions checking for NULL, but apply these changes to match Vim better anyway.
| * | | vim-patch:8.2.1051: crash when changing a list while using reduce() on itSean Dewar2022-01-31
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when changing a list while using reduce() on it. Solution: Lock the list. (closes vim/vim#6330) https://github.com/vim/vim/commit/ca275a05d8b79f6a9101604fdede2373d0dea44e
| * | | 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.