aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | refactor: use kwargs parameter in vim.splitGregory Anders2021-09-25
| | | | |
| * | | | feat: add trimempty optional parameter to vim.splitGregory Anders2021-09-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `split()` VimL function trims empty items from the returned list by default, so that, e.g. split("\nhello\nworld\n\n", "\n") returns ["hello", "world"] The Lua implementation of vim.split does not do this. For example, vim.split("\nhello\nworld\n\n", "\n") returns {'', 'hello', 'world', '', ''} Add an optional parameter to the vim.split function that, when true, trims these empty elements from the front and back of the returned table. This is only possible for vim.split and not vim.gsplit; because vim.gsplit is an iterator, there is no way for it to know if the current item is the last non-empty item. Note that in order to preserve backward compatibility, the parameter for the Lua vim.split function is `trimempty`, while the VimL function uses `keepempty` (i.e. they are opposites). This means there is a disconnect between these two functions that may surprise users.
* | | | | vim-patch:8.2.3469: some files with json syntax are not recognized (#15891)dundargoc2021-10-03
| |_|_|/ |/| | | | | | | | | | | | | | | Problem: Some files with json syntax are not recognized. Solution: Add a few file patterns. (Emiliano Ruiz Carletti, closes vim/vim#8947) https://github.com/vim/vim/commit/50c56893423eb6ad2154a4151e67f7097f52efb6
* | | | vim-patch:8.2.3466: completion submode not indicated for virtual replace ↵zeertzjq2021-10-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (#15886) Problem: Completion submode not indicated for virtual replace. Solution: Add submode to "Rv". (closes vim/vim#8945) https://github.com/vim/vim/commit/cc8cd4453332276d55b4a1109eace5785a4f319d
* | | | vim-patch:8.2.3465: cannot detect insert scroll mode (#15885)zeertzjq2021-10-03
| |/ / |/| | | | | | | | | | | Problem: Cannot detect insert scroll mode. Solution: Add "scroll" to complete_info(). (closes vim/vim#8943) https://github.com/vim/vim/commit/27fef59dd1dd75f50c366f7f616ffa4451560452
* | | Merge pull request #15516 from bfredl/keysetBjörn Linse2021-10-03
|\ \ \ | | | | | | | | refactor(api): Represent option dicts as a structs in C and reduce conversion overhead from lua
| * | | refactor(api): handle option dicts properlyBjörn Linse2021-10-03
| | |/ | |/| | | | | | | | | | | | | | | | Do not copy a lot of lua strings (dict keys) to just strequal() them Just compare them directly to a dedicated hash function. feat(generators): HASHY McHASHFACE
* | | vim-patch:8.2.3464: nginx files are not recognized (#15883)Christian Clason2021-10-03
| | | | | | | | | | | | | | | Problem: nginx files are not recognized. Solution: Add several file patterns. (Chris Aumann, closes vim/vim#8922) https://github.com/vim/vim/commit/8b8c0ed657fabd88e610401ca8a12366f987db94
* | | Merge pull request #15786 from gpanders/diagnostic-signs-unique-severityGregory Anders2021-10-02
|\ \ \
| * | | refactor(diagnostics): always make 'set' go through 'show'Gregory Anders2021-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Always make calls to `vim.diagnostic.set` call `vim.diagnostic.show`. This creates an easier to reason about code path and is also less surprising when users wish to override override `vim.diagnostic.show` with custom behavior and `vim.diagnostic.set` is called with empty diagnostics. Functionally, the end result is the same: when `show` is called with an empty diagnostics list, it just calls `hide` and then returns, which is exactly what `reset` does right now.
| * | | docs(diagnostic): add blurb on how to replace builtin handlersGregory Anders2021-10-02
| | | |
| * | | test(diagnostic): add test case for signsGregory Anders2021-10-02
| | | |
* | | | 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