aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * vim-patch:8.2.1728: compiler warning for using uninitialized variablezeertzjq2022-08-25
| | | | | | | | | | | | Problem: Compiler warning for using uninitialized variable. (John Marriott) Solution: Initialize "neighbor". https://github.com/vim/vim/commit/c53e9c57a9846655c2d3169788f4beefa6d22d90
| * vim-patch:8.1.1085: compiler warning for possibly uninitialized variablezeertzjq2022-08-25
|/ | | | | | | Problem: Compiler warning for possibly uninitialized variable. (Tony Mechelynck) Solution: Make conditions more logical. https://github.com/vim/vim/commit/bd9bf266fccbf7b7f09e476e09b61f0133e914db
* refactor: ins_compl_show_pum() (#19938)zeertzjq2022-08-25
| | | | | | | | | | | This marks the remaining part of Vim patch 8.2.3953 as ported, because ins_compl_upd_pum() has been removed. vim-patch:8.2.3953: insert completion code is too complicated Problem: Insert completion code is too complicated. Solution: More refactoring. Move function arguments into a struct. (Yegappan Lakshmanan, closes vim/vim#9437) https://github.com/vim/vim/commit/6ad84ab3e48d9490e4139df04f2c55b136f5501d
* Merge pull request #19937 from zeertzjq/vim-8.2.3942zeertzjq2022-08-25
|\ | | | | vim-patch:8.2.{3942,partial:4001}
| * vim-patch:partial:8.2.4001: insert complete code uses global variableszeertzjq2022-08-25
| | | | | | | | | | | | | | | | | | | | Problem: Insert complete code uses global variables. Solution: Make variables local to the file and use accessor functions. (Yegappan Lakshmanan, closes vim/vim#9470) https://github.com/vim/vim/commit/d94fbfc74a8b8073e7a256c95fa6f39fc527c726 Skip changes in comments for callback-related functions (not ported). Also make compl_busy static again.
| * vim-patch:8.2.3942: Coverity reports a possible memory leakzeertzjq2022-08-25
|/ | | | | | Problem: Coverity reports a possible memory leak. Solution: Free the array if allocation fails. https://github.com/vim/vim/commit/8e7cc6b920ddea37deaa5e6b7b3bdfff2222d137
* Merge pull request #19936 from zeertzjq/vim-8.2.3989zeertzjq2022-08-25
|\ | | | | vim-patch:8.2.{2534,3989,4037}: Insert mode completion tests and fixes
| * vim-patch:8.2.4037: Insert mode completion is insufficiently testedzeertzjq2022-08-25
| | | | | | | | | | | | | | Problem: Insert mode completion is insufficiently tested. Solution: Add more tests. Fix uncovered memory leak. (Yegappan Lakshmanan, closes vim/vim#9489) https://github.com/vim/vim/commit/370791465e745354d66696de8cbd15504cf958c0
| * vim-patch:8.2.3989: some insert completion code is not testedzeertzjq2022-08-25
| | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some insert completion code is not tested. Solution: Add a few tests. Refactor thesaurus completion. (Yegappan Lakshmanan, closes vim/vim#9460) https://github.com/vim/vim/commit/e982586f8eebf2b055987218f6d3f7a084c4bf69 vim-patch:9.0.0254: typo in function name Problem: Typo in function name. Solution: Rename the function. (closes vim/vim#10971) https://github.com/vim/vim/commit/5fb3aabc2b0edd5573e107bb3bc103c348771f61
| * vim-patch:8.2.2534: missing test coveragezeertzjq2022-08-25
|/ | | | | | | | | | Problem: Missing test coverage. Solution: Improve test coverage for completion with different encodings, mapset(), and term function failures. (Dominique Pellé, closes vim/vim#7877) https://github.com/vim/vim/commit/a1070eae77f635f08b6f2612726b905796baaa58 Cherry-pick E716 -> E715 change from patch 8.2.4861.
* Merge pull request #19927 from zeertzjq/vim-9.0.0045zeertzjq2022-08-25
|\ | | | | vim-patch:9.0.{0045,0046,0060,0102}: Insert mode completion crash fixes
| * vim-patch:9.0.0102: reading past end of line with insert mode completionzeertzjq2022-08-25
| | | | | | | | | | | | Problem: Reading past end of line with insert mode completion. Solution: Check text length. https://github.com/vim/vim/commit/a6f9e300161f4cb54713da22f65b261595e8e614
| * vim-patch:9.0.0060: accessing uninitialized memory when completing long linezeertzjq2022-08-25
| | | | | | | | | | | | Problem: Accessing uninitialized memory when completing long line. Solution: Terminate string with NUL. https://github.com/vim/vim/commit/b9e717367c395490149495cf375911b5d9de889e
| * vim-patch:9.0.0046: reading past end of completion with duplicate matchzeertzjq2022-08-25
| | | | | | | | | | | | Problem: Reading past end of completion with duplicate match. Solution: Check string length https://github.com/vim/vim/commit/baefde14550231f6468ac2ed2ed495bc381c0c92
| * vim-patch:9.0.0045: reading past end of completion with a long linezeertzjq2022-08-25
|/ | | | | | | | | | | | | | | | | Problem: Reading past end of completion with a long line and 'infercase' set. Solution: Allocate the string if needed. https://github.com/vim/vim/commit/caea66442d86e7bbba3bf3dc202c3c0d549b9853 Cherry-pick the deletion of a blank line from patch 9.0.0027. N/A patches for version.c: vim-patch:9.0.0054: compiler warning for size_t to int conversion Problem: Compiler warning for size_t to int conversion. Solution: Add type cast. (Mike Williams, closes vim/vim#10741) https://github.com/vim/vim/commit/c7bd2f08e531f08723cdc677212a3633d11c9a97
* vim-patch:fd999452adaf (#19929)Christian Clason2022-08-25
| | | | Update runtime files https://github.com/vim/vim/commit/fd999452adaf529a30d78844b5fbee355943da29
* fix(treesitter): free memory on removing parser (#19933)zeertzjq2022-08-25
| | | This fixes the ASAN failure.
* Merge pull request #19830 from lewis6991/hlgroup_namebfredl2022-08-24
|\ | | | | feat(highlight)!: error on invalid names and allow '.' and '@'
| * feat(highlight)!: error on invalid names and allow '.' and '@'Lewis Russell2022-08-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously if a highlight group with a name outside the regexp [a-zA-Z0-9_] was defined, Nvim would emit an "invalid character" warning message. This was annoying for Lua scripts, as it was very hard to debug what line of code was triggering this message since it didn't produce a stack trace. This has now been promoted to an error with the code E5248. Additionally the ASCII character period ('.') and at-sign ('@') have been added to the allowed list of characters of a highlight group name to support the application of defining hierarchical highlight groups, e.g. 'TS.keyword'. Co-authored-by: Christian Clason <christian.clason@uni-due.de>
* | Merge pull request #19928 from vigoux/ts-fix-c-parserThomas Vigouroux2022-08-24
|\ \ | | | | | | test(treesitter): make internal lang test pending when necessary
| * | test(treesitter): make internal lang test pending when necessaryThomas Vigouroux2022-08-24
|/ /
* | feat(lua): add vim.iconv (#18286)Lewis Russell2022-08-24
| | | | | | Co-authored-by: Justin M. Keyes <justinkz@gmail.com>
* | Merge pull request #19496 from vigoux/ts_internal_langThomas Vigouroux2022-08-24
|\ \ | | | | | | feat(treesitter): allow customizing language symbol name
| * | feat(treesitter): allow customizing language symbol nameThomas Vigouroux2022-08-22
| | |
* | | Merge pull request #19906 from bfredl/bigstagebfredl2022-08-24
|\ \ \ | | | | | | | | perf(api): allow to use an arena for return values
| * | | refactor(api): provide a temporary copy solution for nvim_call_atomicbfredl2022-08-24
| | | | | | | | | | | | | | | | | | | | | | | | Make the copy_object() family accept an optional arena. More than half of the callsites should be refactored to use an arena later anyway.
| * | | refactor(arena): use a shared block freelistbfredl2022-08-24
| | | | | | | | | | | | | | | | | | | | This is both simpler in client code and more effective (always reuse block hottest in cache)
| * | | perf(api): allow to use an arena for return valuesbfredl2022-08-23
| | | |
* | | | feat(eval)!: make Vim functions return inner window width and height (#19743)zeertzjq2022-08-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In non-multigrid UI the only change is that the returned height now excludes winbar, and this is compatible with Vim. In multigrid UI this means the return value of these functions now reflect the space available for buffer lines in a window. No change in nvim_win_get_height() and nvim_win_get_width().
* | | | vim-patch:partial:8.2.3953: insert completion code is too complicated (#19923)zeertzjq2022-08-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insert completion code is too complicated. Solution: More refactoring. Move function arguments into a struct. (Yegappan Lakshmanan, closes vim/vim#9437) https://github.com/vim/vim/commit/6ad84ab3e48d9490e4139df04f2c55b136f5501d Skip most pum-related refactoring. Cherry-pick rename to match_at_original_text() from patch 8.2.4001.
* | | | vim-patch:9.0.0020: with some completion reading past end of string (#19922)zeertzjq2022-08-24
| | | | | | | | | | | | | | | | | | | | Problem: With some completion reading past end of string. Solution: Check the length of the string. https://github.com/vim/vim/commit/f12129f1714f7d2301935bb21d896609bdac221c
* | | | Merge pull request #19921 from zeertzjq/vim-8.2.3937zeertzjq2022-08-24
|\ \ \ \ | |_|_|/ |/| | | vim-patch:8.2.{3937,3944}: Insert mode completion refactoring
| * | | vim-patch:8.2.3944: insert mode completion functions are too longzeertzjq2022-08-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insert mode completion functions are too long. Solution: Split up into multiple functions. (Yegappan Lakshmanan, closes vim/vim#9431) https://github.com/vim/vim/commit/5d2e007ccbfbd749a1f201d06965b8811ff50e6e Cherry-pick can_cindent_get() -> get_can_cindent() from patch 8.1.2062.
| * | | vim-patch:8.2.3937: Insert mode completion function is too longzeertzjq2022-08-24
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Insert mode completion function is too long. Solution: Refactor into multiple functions. (Yegappan Lakshmanan, closes vim/vim#9423) https://github.com/vim/vim/commit/edc6f103907a004b9e2265e232dc8be8bc594601 Cherry-pick a typo fix from patch 8.2.3637.
* | | fix(path): path_is_url returns false for "foo:/" #19797sigmaSd2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: path_to_url() returns false for single-slash URIs ("foo:/" vs "foo://"). This is not compliant with the URI spec. https://url.spec.whatwg.org/#url-representation LSP in particular allows single-slash URIs. Solution: Relax path_to_url() to accept single-slash URIs. This is not fully compliant (only ":" is required by the spec), but it is hopefully good enough without causing false-positives in typical text files. ref https://url.spec.whatwg.org/#windows-drive-letter ref https://github.com/neovim/neovim/pull/19773 ref https://github.com/neovim/neovim/pull/19773#issuecomment-1214763769
* | | fix(cmdline): do not trigger completion at wrong time (#19920)zeertzjq2022-08-24
| | | | | | | | | Cherry-picked from Vim patches 8.2.4339, 9.0.0238.
* | | vim-patch:9.0.0248: duplicate code in finding a script in the execution ↵zeertzjq2022-08-24
| | | | | | | | | | | | | | | | | | | | | stack (#19917) Problem: Duplicate code in finding a script in the execution stack. Solution: Reduce duplicate code. (closes vim/vim#10961) https://github.com/vim/vim/commit/a247142ae45308087b25f91c8af48399c8ac2943
* | | vim-patch:9.0.0206: redraw flags are not named specifically (#19913)zeertzjq2022-08-23
| | | | | | | | | | | | | | | Problem: Redraw flags are not named specifically. Solution: Prefix "UPD_" to the flags, for UPDate_screen(). https://github.com/vim/vim/commit/a4d158b3c839e96ed98ff87c7b7124ff4518c4ff
* | | refactor(spell.c): resolve clint errors (#19875)Lewis Russell2022-08-23
| | | | | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* | | doc(CONTRIBUTING): update #19891Lewis Russell2022-08-23
| | | | | | | | | | | | | | | | | | - add section for managing includes via IWYU - recommend clangd over ctags - tidy up links - remove ./scripts/check-includes.py
* | | fix(inspect): escape identifiers that are lua keywords (#19898)Simon Wachter2022-08-23
| | | | | | | | | A lua keyword is not a valid table identifier
* | | Merge pull request #19905 from zeertzjq/vim-8.2.4726zeertzjq2022-08-23
|\ \ \ | | | | | | | | vim-patch:8.2.{4726,4740,4741,4749,4841,4842}: expand('<script>')
| * | | vim-patch:8.2.4842: expand("%:p") is not empty when there is no buffer namezeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | Problem: expand("%:p") is not empty when there is no buffer name. Solution: When ignoring errors still return NULL. (closes vim/vim#10311) https://github.com/vim/vim/commit/211a5bb2353c66684f38527184a258921f95c9d9
| * | | vim-patch:8.2.4841: empty string considered an error for expand()zeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Empty string considered an error for expand() when 'verbose' is set. (Christian Brabandt) Solution: Do not give an error for an empty result. (closes vim/vim#10307) https://github.com/vim/vim/commit/a96edb736d4274fc4aea460800780e06e1510812
| * | | vim-patch:8.2.4749: <script> is not expanded in autocmd contextzeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: <script> is not expanded in autocmd context. Solution: Add the context to the pattern struct. (closes vim/vim#10144) Rename AutoPatCmd to AutoPatCmd_T. https://github.com/vim/vim/commit/eca7c60d68e63001dbe3c8e5d240b0895e607fc3 Omit AutoPatCmd -> AutoPatCmd_T rename as it is inconsistent. Use `.sn_name` instead of `->sn_name` as v8.2.0154 hasn't been ported. Omit acp_script_stx(), use member directly.
| * | | vim-patch:8.2.4741: startup test failszeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Startup test fails. Solution: Avoid an error for verbose expansion. Fix that the "0verbose" command modifier doesn't work. https://github.com/vim/vim/commit/60895f3e36def9beb7d5463e792e5154ad9a7a0a Most code changes has already been ported.
| * | | vim-patch:8.2.4740: when expand() fails there is no error messagezeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | | | | | | | Problem: When expand() fails there is no error message. Solution: When 'verbose' is set give an error message. https://github.com/vim/vim/commit/575445200bd35283191ecd7a0d596b37c5b477a4
| * | | vim-patch:8.2.4726: cannot use expand() to get the script namezeertzjq2022-08-23
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot use expand() to get the script name. Solution: Support expand('<script>'). (closes vim/vim#10121) https://github.com/vim/vim/commit/6013d0045dec7ca7c0068fbe186c42d754a7368b Use `.sn_name` instead of `->sn_name` as v8.2.0154 hasn't been ported. Cherry-pick builtin.txt expand() doc from latest Vim.
* | | test: add test for using :normal to stop Visual mode in cmdwin (#19904)zeertzjq2022-08-23
| | |
* | | vim-patch:8.2.1046: insufficient tests for src/buffer.c (#19903)zeertzjq2022-08-23
| | | | | | | | | | | | | | | | | | Problem: Insufficient tests for src/buffer.c. Solution: Add more tests. Move comments related tests to a separate file. (Yegappan Lakshmanan, closes vim/vim#6325) https://github.com/vim/vim/commit/b7e2483655d9b68df0c7349918027d800051a28a