aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | refactor(diff.c): allocate hunks directly in ga_arrayLewis Russell2022-10-24
| | | |
| * | | refactor(diff.c): remove char_u declarationsLewis Russell2022-10-24
| | | |
| * | | refactor(diff.c): break up ex_diffgetput()Lewis Russell2022-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ex_diffgetput is too big Solution: factor out main loop into function Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* | | | build: preprocess vim patches with uncrustify #20786dundargoc2022-10-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will enable a larger amount of chunks being automatically included due to fewer formatting differences between the vim and neovim files. The strategy is straightforward, if a bit tedious: - Get a list of all changed files. - Checkout parent commit. Copy all relevant files to a temporary location. - Checkout patch commit. Copy all relevant files to a temporary location. - Format .c and .h files with uncrustify. - Generate a diff from from these files. Closes https://github.com/neovim/neovim/issues/6226
* | | | fix(lua): pesc, tbl_islist result types #20751NAKAI Tsuyoshi2022-10-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: - pesc() returns multiple results, it should return a single result. - tbl_islist() returns non-boolean in some branches. - Docstring: @generic must be declared first Solution: Constrain docstring annotations. Fix return types. Co-authored-by: Justin M. Keyes <justinkz@gmail.com>
* | | | ci(lintcommit): enforce common scope names #20393dundargoc2022-10-24
| | | | | | | | | | | | | | | | This is useful to ensure related commits aren't spread out when generating the changelog.
* | | | Merge pull request #20791 from zeertzjq/vim-8.2.0908zeertzjq2022-10-24
|\ \ \ \ | | | | | | | | | | vim-patch:8.2.0908: crash when changing the function table while listing it
| * | | | test: add a test for a crash fixed by patch 8.2.0908zeertzjq2022-10-24
| | | | |
| * | | | vim-patch:8.2.0908: crash when changing the function table while listing itzeertzjq2022-10-24
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when changing the function table while listing it. Solution: Bail out when the function table changes. (closes vim/vim#6209) https://github.com/vim/vim/commit/3fffa97159a427067b60c80ed4645e168cc5c4bd Co-authored-by: Bram Moolenaar <Bram@vim.org>
* | | | build(deps): bump vimdoc parser and queries to v1.2.4 (#20788)Christian Clason2022-10-24
| | | |
* | | | docs: ":che" is ":checkhealth" #20147ii142022-10-23
| | | |
* | | | feat(vim-patch): mention original author #20772Maxime Brunet2022-10-23
| | | |
* | | | docs(maintain): CI strategy #20778dundargoc2022-10-22
| | | | | | | | | | | | | | | | It's only focused on GitHub Actions since that is the primary CI provider we use at the moment.
* | | | docs: fix typos (#20724)dundargoc2022-10-23
|/ / / | | | | | | Co-authored-by: Marco Lehmann <m99@posteo.de>
* | | refactor(diff.c): reduce scope of variables (#20781)Lewis Russell2022-10-23
| | | | | | | | | Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* | | fix(ui): send grid_resize events before triggering VimResized (#20760)zeertzjq2022-10-22
| | |
* | | Merge pull request #20770 from dundargoc/refactor/lintbfredl2022-10-21
|\ \ \ | |/ / |/| | refactor: fix uncrustify lint errors
| * | refactor: fix uncrustify lint errorsdundargoc2022-10-21
|/ /
* | vim-patch:9.0.0814: aws config files are not recognized (#20769)Christian Clason2022-10-21
| | | | | | | | | | | | | | | | vim-patch:436e5d395fd6 (since upstream tagged the wrong commit) Problem: Aws config files are not recognized. Solution: Use "confini" for aws config files. (Justin M. Keyes, closes vim/vim#11416) https://github.com/vim/vim/commit/436e5d395fd629c8d33b5cf7b373aad007f16851
* | Merge pull request #20664 from dundargoc/refactor/uncrustifydundargoc2022-10-21
|\ \ | | | | | | refactor(uncrustify): improved formatting rules
| * | docs(dev-style): remove rules covered by uncrustifydundargoc2022-10-21
| | | | | | | | | | | | | | | There's no reason for contributors to learn rules that can be automated away.
| * | refactor(uncrustify): improved formatting rulesdundargoc2022-10-21
| | |
| * | refactor(uncrustify): move macros definitions to enable formattingdundargoc2022-10-21
|/ / | | | | | | | | | | Uncrustify struggles to format function-like macros which are defined in deeply nested areas of the code. Un-nesting them unblocks useful formatting rules from uncrustify.
* | refactor: clang-tidy fixes to silence clangd warning (#20683)dundargoc2022-10-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refactor: readability-uppercase-literal-suffix * refactor: readability-named-parameter * refactor: bugprone-suspicious-string-compare * refactor: google-readability-casting * refactor: readability-redundant-control-flow * refactor: bugprone-too-small-loop-variable * refactor: readability-non-const-parameter * refactor: readability-avoid-const-params-in-decls * refactor: google-readability-todo * refactor: readability-inconsistent-declaration-parameter-name * refactor: bugprone-suspicious-missing-comma * refactor: remove noisy or slow warnings
* | fix(filetype): don't pass empty string to detect (#20766)Christian Clason2022-10-21
| | | | | | | | | | Problem: `*.db` files use empty string as default filetype, which is inconsistent with the rest of the code which uses `nil` instead. Solution: don't pass a default empty string
* | vim-patch: bump VIM_VERSION from 8.0 => 8.1 #20762Justin M. Keyes2022-10-21
|/ | | | There are 6 remaining 8.0.x patches, tracked in: https://github.com/neovim/neovim/issues/5431
* vim-patch:9.0.0806: 'langmap' works differently when there are modifiers ↵zeertzjq2022-10-21
| | | | | | | | (#20754) Problem: 'langmap' works differently when there are modifiers. Solution: Only apply 'langmap' to a character where modifiers have no effect. (closes vim/vim#11395, closes vim/vim#11404) https://github.com/vim/vim/commit/49660f5139d3fd55326a54eadf6bb31a3ffec2bf
* vim-patch:9.0.0808: jsonnet filetype detection has a typo (#20753)Christian Clason2022-10-20
| | | | | Problem: jsonnet filetype detection has a typo. Solution: Change "libjsonnet" to "libsonnet". (Maxime Brunet, closes vim/vim#11412) https://github.com/vim/vim/commit/6c8bc37a1083d17447156592f6f52da2d40b4855
* Merge pull request #20748 from zeertzjq/vim-9.0.0804zeertzjq2022-10-20
|\ | | | | vim-patch:partial:8.2.0418,9.0.0804: crash when trying to divide a number by -1
| * vim-patch:9.0.0804: crash when trying to divide a number by -1zeertzjq2022-10-20
| | | | | | | | | | | | Problem: Crash when trying to divice the largest negative number by -1. Solution: Handle this case specifically. https://github.com/vim/vim/commit/cdef1cefa2a440911c727558562f83ed9b00e16b
| * vim-patch:partial:8.2.0418: code in eval.c not sufficiently covered by testszeertzjq2022-10-20
|/ | | | | | | | | | Problem: Code in eval.c not sufficiently covered by tests. Solution: Add more tests. (Yegappan Lakshmanan, closes vim/vim#5815) https://github.com/vim/vim/commit/8b633135106dda8605463b780573c45b00c22afe Only port test_expr.vim and the first hunk of test_cmdline.vim. Add missing test from patch 7.4.1755. Cherry-pick test_expr.vim change from patch 8.2.2060.
* docs: update vimdoc parser #20747Justin M. Keyes2022-10-20
| | | | | | | | | Remove the user-manual ToC from help.txt, because: 1. it duplicates usr_toc.txt 2. it is not what most readers are looking for in the main help page. fix https://github.com/neovim/tree-sitter-vimdoc/issues/49 fix https://github.com/neovim/tree-sitter-vimdoc/issues/50 fix https://github.com/neovim/tree-sitter-vimdoc/issues/51
* fix(build): duplicate version string "v0.8.0-v0.8.0" #20578dundargoc2022-10-20
| | | | | | | | | | - Prevent duplicate version strings such as v0.8.0-v0.8.0. - Change the format for git releases from v0.9.0-dev-67-g625ba79be to v0.9.0-dev-67+g625ba79be. Nvim versions are now: release : v0.9.0 prerelease without git info: v0.9.0-dev prerelease with git info : v0.9.0-dev-67+g625ba79be
* fix(docs-html): misaligned tabs after conceal #20690Yee Cheng Chin2022-10-20
| | | | | | | | | | | | | | | | Problem: `gen_help_html.lua` does not properly handle tab characters after "concealed" text (tags, taglinks, codespans). This causes misaligned layout in "old" (preformatted) docs. For text like `*tag*`, |tag_link|, and `code_span`, Vim hides the "*", "|", "`" characters, but Vim still counts those characters for "virtual column" when a tab character follows it. So if you have a tag of say 6 characters long, those two concealed character would lead to the tab character after it start at column 8. gen_help_html.lua doesn't account for that which leads to formatting flaws in the generated output. Solution: Add two spaces after concealed nodes that are followed by a tab char.
* docs(news): mention 'splitkeep' #20744zeertzjq2022-10-20
|
* vim-patch:9.0.0798: clang format configuration files are not recognized (#20741)ObserverOfTime2022-10-19
| | | | | | Problem: Clang format configuration files are not recognized. Solution: Use yaml for Clang format configuration files. (Marwin Glaser, closes vim/vim#11398) https://github.com/vim/vim/commit/3c708c43908ba44f075bbaa7daf584c6b46d9723
* vim-patch:9.0.0739: mouse column not correctly used for popup_setpos (#20729)zeertzjq2022-10-19
| | | | | | | | Problem: Mouse column not correctly used for popup_setpos. Solution: Adjust off-by-one error and handle Visual line selection properly. (Yee Cheng Chin, closes vim/vim#11356) https://github.com/vim/vim/commit/17822c507c03d509037c9ee5eee5cfbb201b3f01 The test_termcodes.vim test cannot be used. Use a Lua test instead.
* revert: "oldtests: wait 200ms on mac for timer test" (#20728)zeertzjq2022-10-19
| | | This reverts commit 3bad76008e1c98724eca7d986a6340eff1de8193.
* Merge pull request #20727 from zeertzjq/vim-9.0.0789zeertzjq2022-10-19
|\ | | | | vim-patch:9.0.{0789,0790}
| * vim-patch:9.0.0790: test for dummy buffer does not always produce the E86 errorzeertzjq2022-10-19
| | | | | | | | | | | | Problem: Test for dummy buffer does not always produce the E86 error. Solution: Do not check if the error is produced. https://github.com/vim/vim/commit/53c5c9f50ca68d3ed559eebb2c5f7d23f39a768c
| * vim-patch:9.0.0789: dummy buffer ends up in a windowzeertzjq2022-10-19
|/ | | | | | Problem: Dummy buffer ends up in a window. Solution: Disallow navigating to a dummy buffer. https://github.com/vim/vim/commit/8f3c3c6cd044e3b5bf08dbfa3b3f04bb3f711bad
* vim-patch:9.0.0788: ModeChanged autocmd not executed when Visual ends with ↵zeertzjq2022-10-19
| | | | | | | | | | CTRL-C (#20722) Problem: ModeChanged autocmd not executed when Visual mode is ended with CTRL-C. Solution: Do not trigger the autocmd when got_int is set. (closes vim/vim#11394) https://github.com/vim/vim/commit/61c4b04799bf114cadc3bbf212ae8b2ad22a6980 Cherry-pick removal of cmdwin feature check from patch 9.0.0663.
* refactor(window.c): reduce scope of locals (#20301)Lewis Russell2022-10-18
|
* vim-patch:9.0.0779: lsl and lm3 file extensions are not recognized (#20704)Christian Clason2022-10-18
| | | | | Problem: lsl and lm3 file extensions are not recognized. Solution: Add *.lsl and *.lm3 patterns. (Doug Kearns, closes vim/vim#11384) https://github.com/vim/vim/commit/4ac8e7948cb3e07bc4598ede8b274891d14dfa7c
* fix(highlight): link more treesitter groups by default (#20711)Christian Clason2022-10-18
| | | | | | | | | | | Problem: Captures used by bundled parsers are not highlighted by default Solution: Add links to default groups A link is added for a capture if * there is a default group of the same name (e.g., `@tag` -> `Tag`) * it's used in a bundled query and doesn't have a reasonable fallback (e.g., `@text.literal`) Also add all linked groups to the treesitter docs.
* feat(docs): update parser, HTML gen #20720Justin M. Keyes2022-10-18
| | | | | | | | | | Note: although the tolerance in help_spec.lua increased, the actual error count with the new parser decreased by about 20%. The difference is that the old ignore_parse_error() ignored many more errors with the old parser. fix https://github.com/neovim/tree-sitter-vimdoc/issues/37 fix https://github.com/neovim/tree-sitter-vimdoc/issues/44 fix https://github.com/neovim/tree-sitter-vimdoc/issues/47
* vim-patch:9.0.0786: user command does not get number from :tab modifier (#20716)zeertzjq2022-10-18
| | | | | Problem: User command does not get number from :tab modifier. Solution: Include the number. (closes vim/vim#11393, closes vim/vim#6901) https://github.com/vim/vim/commit/208567e9d744ef7b89bed1f62e951ae4ee2f6f5f
* fix(intro): omit patch version in ":help news" item #20713zeertzjq2022-10-18
| | | | Because maintenance releases share the same news.txt as the last non-maintenance release.
* ci: add reviewers based on labels (#20703)dundargoc2022-10-18
|
* fix(qflist): avoid read of uninitialized memory (#20709)Mahmoud Al-Qudsi2022-10-18
| | | | | | | | | | | | | | | | | | | | | | | | | | If the call to `qf_setup_state()` in `qf_init_ext()` fails, control flow jumps to label `qf_init_end` where a call to `qf_update_buffer()` is made with `old_last` as a function call argument. Prior to this patch, `old_last` would not yet have been initialized to its default value of `NULL`, resulting in `qf_update_buffer()` attempting to compare against its uninitialized value (quickfix.c:3882) then later forwarding it to `qf_fill_buffer()` where the address is dereferenced and repeatedly read from/ written to for performing core qflist operations. Depending on what the default value of `old_last` was, the results may range from a best case scenario of neovim terminating with SIGSEGV from an attempt to dereference an invalid pointer (quickfix.c:4056) to memory corruption if it contained some other value that results in `qfp` being initialized from `old_last->qf_next` (after which it is subsequently written to and read from in a fairly arbitrary fashion). Though extremely unlikely, it's possible for there to be security considerations as a user can ensure that the next call to `qf_setup_state()` fails. This patch ensures that `old_last` is NULL-assigned before control flow jumps to `qf_init_end`. Closes #20639.