aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | fix(nvim_open_win): crash if autocmds delete buffer/window #15549Sean Dewar2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | win_set_buf can trigger autocmds if noautocmd=false. If they close the window, code afterwards will dereference the freed win_T* wp pointer. This interaction became possible after commit 1def3d1542d6a65f057e743faea39a760b50db87. The reason deleting curbuf crashes, and not the buf passed to `nvim_open_win`, is because the float initially edits curbuf (`win_init`) until it's later set to edit buf (windows from `:new` and `:split <buf>` behave similiarly: approx. `:split`, then `:buffer <buf>`). `do_buffer` closes windows when their edited buffer is deleted (unless it's the only window; N/A for floats), so the float closes when curbuf is deleted, so we need to check `win_valid` after `win_set_buf` too. Closes #15548
* | | | | vim-patch:8.2.3460: some type casts are not needed #15868dundargoc2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some type casts are not needed. Solution: Remove unnecessary type casts. (closes vim/vim#8934) https://github.com/vim/vim/commit/dfa5e464d459f84200a73d178f1ecefe75bbe511
* | | | | docs(lsp): clarify parameters of some util functions (#15851)zeertzjq2021-10-02
| | | | | | | | | | | | | | | | | | | | `pad_left` and `pad_right` are unused List used keys of `opts` in `make_floating_popup_options`
* | | | | fix(float)!: always anchor to corner of window including border #15832zeertzjq2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | N, W, S, E are all inclusive, i.e., always anchor to the exact corner of the window (including border). This line may also need change in this case (change 0 to -1): This is most consistent and easiest to reason about, especially with GUIs whose border do not need to have width/height of 1/1 in cell units. Fix #15789
* | | | | Merge #15803 fix(lsp_spec): tests depend on previous sessionJustin M. Keyes2021-10-02
|\ \ \ \ \
| * | | | | fix(lsp_spec): tests depended on previous sessionJustin M. Keyes2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | - fix tests which accidentally depended on previous session
| * | | | | fix(release.sh): ignore failed "rm"Justin M. Keyes2021-10-02
| | | | | |
* | | | | | refactor: format with uncrustify #15872dundargoc2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | * refactor: format with uncrustify * refactor: fix function parameter comments
* | | | | | fix(docs): uncrustify.cfg location #15817kylo2522021-10-02
| | | | | |
* | | | | | docs: remove obsolete text on language #15875dundargoc2021-10-02
| | | | | |
* | | | | | fix(input): resolve isolated (non-ALT/META) mappings #13109erw72021-10-02
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Since 2f06413dfb36 #13042, "ESC+c" sequence is treated as "ESC c" instead of "M-c" (ALT/META+c) when not mapped, aka "fallthrough" behavior. But "isolated" (non-ALT/META) mappings to ESC and c were not resolved. This behavior is especially confusing for the TUI. Solution: Resolve isolated ESC, c mappings when there is no M-c mapping. Change ins_char_typebuf() to escape CSI, K_SPECIAL. fixes #13086 fixes #15869
* | | | | Merge pull request #15867 from bfredl/starpackBjörn Linse2021-10-02
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fix(runtime): add compressed {&packpath}/start/*/pack/*[/after] representation to &rtp by suggestion by at-tpope Summary: We can add XDG_DATA_DIR/nvim/site/pack/*/start/* (et al) as an unexpanded wildchar to &rtp which keeps it both short and explicit and still supporting globpath(&rtp, ...). ref #15101
| * | | | fix(runtime): add packages as "/pack/*/start/*" patterns to &rtpBjörn Linse2021-10-02
| |/ / / | | | | | | | | | | | | This makes `globpath(&rtp, ...)` work again for start packages
* | | | refactor: format with uncrustify #15842dundargoc2021-10-02
| | | | | | | | | | | | | | | | * refactor: format with uncrustify * refactor: convert function comments to doxygen
* | | | refactor: convert char_u to char #15824dundargoc2021-10-02
| | | |
* | | | feat(vim-patch.sh): better detection of remote name #15846zeertzjq2021-10-02
| | | |
* | | | feat(diagnostics): add vim.diagnostic.get_namespaces (#15866)Michael Lingelbach2021-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many vim.diagnostic functions expect the user to pass in a namespace id. This PR allows the user to list active diagnostic namespaces: ```lua :lua print(vim.inspect(vim.diagnostic.get_namespaces())) { [7] = { name = "vim.lsp.client-1", opts = {}, sign_group = "vim.diagnostic.vim.lsp.client-1" } } ```
* | | | docs(diagnostics): add "priority" option to signs table (#15860)Gregory Anders2021-10-01
| | | | | | | | | | | | | | | | This feature was added in #15785, but the docs for vim.diagnostic.config() weren't updated.
* | | | Merge pull request #15864 from jamessan/gitattributesJames McCoy2021-10-01
|\ \ \ \ | | | | | | | | | | fix: include ci/ in exported tarball
| * | | | fix: include ci/ in exported tarballJames McCoy2021-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | src/nvim/testdir/runnvim.sh re-uses the test suite code from ci/ to cleanup the output of legacy tests. Closes #15856 [skip ci]
* | | | | Merge pull request #15861 from dundargoc/refactor/xdiffJames McCoy2021-10-01
|\ \ \ \ \ | | | | | | | | | | | | refactor: update path to xdiff in comments
| * | | | | refactor: update path to xdiff in commentsDundar Göc2021-10-01
| | |/ / / | |/| | |
* | | | | Merge pull request #15862 from dundargoc/refactor/xdiff-pvsJames McCoy2021-10-01
|\ \ \ \ \ | | | | | | | | | | | | refactor: remove PVS comment on top of files
| * | | | | refactor: remove PVS comment on top of filesDundar Göc2021-10-01
| |/ / / / | | | | | | | | | | | | | | | | | | | | The xdiff directory is excluded from the PVS report so the comment isn't required.
* | | | | Merge pull request #15863 from dundargoc/docs/contributing/commit-messageJames McCoy2021-10-01
|\ \ \ \ \ | |/ / / / |/| | | | docs: update preferred commit message style for the static analyzer PRs
| * | | | docs: update preferred commit message style for the static analyzer PRsDundar Göc2021-10-01
|/ / / /
* | | | Merge pull request #15852 from bfredl/afteragainBjörn Linse2021-09-30
|\ \ \ \ | | | | | | | | | | fix(runtime): fix ordering of "after" packages
| * | | | fix(runtime): fix ordering of "after" packagesBjörn Linse2021-09-30
|/ / / / | | | | | | | | | | | | they must come after ordinary runtime dirs which are not "after"
* | | | ci: add type and scope from PR title as labels #15850dundargoc2021-09-30
| | | | | | | | | | | | This only works if the label exists in the first place of course.
* | | | Merge pull request #14937 from dstein64/vim-8.1.2304Jan Edmund Lazo2021-09-29
|\ \ \ \ | | | | | | | | | | vim-patch:8.1.2304,8.1.2309,8.1.2319,8.1.2321
| * | | | docs: add 'plus one' to correct getmousepos docsDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | This change was applied to Vim as part of 90df4b9. https://github.com/vim/vim/commit/90df4b9d423485f7db16e3a65cab4f14edc815ae
| * | | | chore: remove empty lineDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This update was part of vim-patch:4c29502. https://github.com/vim/vim/commit/4c295027a426986566cd7a76c47a6d3a529727e7 The other parts of that patch were applied to Neovim in ad6bb38.
| * | | | test: add a test for getmousepos()Daniel Steinberg2021-09-29
| | | | |
| * | | | vim-patch:8.1.2321: cannot select all text with the mouseDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot select all text with the mouse. (John Marriott) Solution: Move limiting the mouse column to f_getmousepos(). (closes https://github.com/vim/vim/issues/5242) https://github.com/vim/vim/commit/0a5aa7b28a39507260acb15c1ef698a33c855cc1
| * | | | vim-patch:8.1.2319: compiler warning for int sizeDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Compiler warning for int size. Solution: Add typecast. (Mike Williams) https://github.com/vim/vim/commit/07a63d86338476bafbd1a3ec462672df92666498
| * | | | vim-patch:8.1.2309: compiler warning for argument typeDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Compiler warning for argument type. Solution: Use linenr_T and cast to varnumber_T. (John Marriott) https://github.com/vim/vim/commit/abe12a1a4fce36bfa5dea3a0ce85603432d1905b
| * | | | vim-patch:8.1.2304: cannot get the mouse position when getting a mouse clickDaniel Steinberg2021-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot get the mouse position when getting a mouse click. Solution: Add getmousepos(). https://github.com/vim/vim/commit/db3a205147ce2c335d5c2181c1f789277f8775b0
* | | | | Refactor/uncrustify (#15790)dundargoc2021-09-29
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refactor: format with uncrustify * fixup(dundar): fix functions comments * fixup(dundar): remove space between variable and ++/-- * fixup(dundar): better workaround for macro attributes This is done to be able to better use uncrustify rules for macros * fixup(justin): make preprocessors follow neovim style guide
* | | | feat(lsp): add codeAction/resolve support (#15818)Mathias Fußenegger2021-09-28
| | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/15339 and https://github.com/neovim/neovim/issues/15828
* | | | Merge pull request #15812 from bfredl/tabaBjörn Linse2021-09-28
|\ \ \ \ | | | | | | | | | | fix(runtime): make a copy of runtime_search_path when iterating
| * | | | fix(runtime): make a copy of runtime_search_path when iteratingBjörn Linse2021-09-27
| | | | | | | | | | | | | | | | | | | | | | | | | This is to prevent concurrent modification, just like save_rtp in the vim 8 implementation
* | | | | feat(lsp): add client command support to codelens (#15820)Mathias Fußenegger2021-09-28
| | | | | | | | | | | | | | | | | | | | Also adds a check against the server capabilities to fix https://github.com/neovim/neovim/issues/15183
* | | | | refactor(lsp): remove json encode/decode wrappers (#15826)Mathias Fußenegger2021-09-28
| | | | |
* | | | | fix(ui): s/format_entry/format_item to match docs (#15819)Mathias Fußenegger2021-09-27
| | | | | | | | | | | | | | | Follow up to https://github.com/neovim/neovim/pull/15771
* | | | | feat(ui): add vim.ui.select and use in code actions (#15771)Mathias Fußenegger2021-09-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Continuation of https://github.com/neovim/neovim/pull/15202 A plugin like telescope could override it with a fancy implementation and then users would get the telescope-ui within each plugin that utilizes the vim.ui.select function. There are some plugins which override the `textDocument/codeAction` handler solely to provide a different UI. With custom client commands and soon codeAction resolve support, it becomes more difficult to implement the handler right - so having a dedicated way to override the picking function will be useful.
* | | | | chore: remove unused script finddeclarations.pl #15814dundargoc2021-09-27
|/ / / /
* | | | fix(lsp): avoid serializing boolean as key (#15810)Michael Lingelbach2021-09-27
| | | | | | | | | | | | | | | | | | | | | | | | In vim.lsp.buf.references, the key vim.type_idx (which evaluates to a boolean) was set to equal vim.types.dictionary. This resulted in a boolean key in json which is not allowed by the json spec, and which lua-cjson fails to serialize.
* | | | docs: document dependencies in MAINTAIN.md #15801Michael Lingelbach2021-09-26
| | | | | | | | | | | | Co-authored-by: Justin M. Keyes <justinkz@gmail.com>
* | | | refactor(diagnostic): use sign priority for severity_sort #15785Gregory Anders2021-09-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than relying on the order in which signs are placed to dictate the order in which they are displayed, explicitly set the priority of the sign according to the severity of the diagnostic and the value of severity_sort. If severity_sort is false or unset then all signs use the same priority.
* | | | docs: add lua-cjson to third-party licenses #15800Michael Lingelbach2021-09-26
| | | |