aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | 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
| | |
* | | feat(lsp): allow subset of CodeActionContext as arg to code_action methods ↵Mathias Fußenegger2021-09-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | (#15793) This makes it easier to filter the code actions. For example: vim.lsp.buf.code_action { only = 'refactor' }
* | | feat(lsp): use cjson for lsp rpc (#15759)Michael Lingelbach2021-09-26
| | |
* | | Merge pull request #14871 from mjlbach/feature/lua-cjson-embeddedMichael Lingelbach2021-09-26
|\ \ \ | | | | | | | | feat(lua): expose lua-cjson as vim.json
| * | | test: add tests for vim.jsonMichael Lingelbach2021-09-26
| | | |
| * | | feat(lua): expose lua-cjson as vim.jsonMichael Lingelbach2021-09-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * add vim.json.encode and vim.json.decode * use vim.NIL instead of cjson.null * resolve strict-prototypes warnings * The following benchmark shows an approximately 2.5x (750 ms vs 300 ms) improvement in deserialization performance over vim.fn.json_decode on a medium package.json ```lua local uv = vim.loop local function readfile(path) return end local json_url = "https://raw.githubusercontent.com/rust-analyzer/rust-analyzer/b24c8d5c89ee93d1172b4127564f5da3b0c88dad/editors/code/package.json" io.popen(string.format('curl -v -f -L -O %q &> /dev/null', json_url)) local json_string = io.open('package.json'):read '*a' uv.update_time() local start = uv.hrtime() for i = 1,1000 do vim.fn.json_decode(json_string) end uv.update_time() print(string.format("Deserialization time vim.fn.json_decode: %s ms", (uv.hrtime() - start) * (1e-6))) uv.update_time() local start = uv.hrtime() for i = 1,1000 do vim.json.decode(json_string) end uv.update_time() print(string.format("Deserialization time vim.json.decode: %s ms", (uv.hrtime() - start) * (1e-6))) ``` Co-Authored-By: Björn Linse <bjorn.linse@gmail.com>
| * | | feat(lua): add lua-cjson as vendored dependencyMichael Lingelbach2021-09-26
| | | | | | | | | | | | | | | | Derived from the openresty lua-cjson fork at commit https://github.com/openresty/lua-cjson/commit/3d93d297092172eac3d52a1b3b6c1d479da5341a
* | | | ci(lintcommit.lua): replace third-party commitlint #15747dundargoc2021-09-26
| | | |
* | | | Merge pull request #15797 from smolck/ui-stuffBjörn Linse2021-09-26
|\ \ \ \ | | | | | | | | | | fix(ui_bridge): set bridge width and height on attach
| * | | | fix(ui_bridge): set bridge width and height on attachsmolck2021-09-26
| | |_|/ | |/| |
* | | | fix(tui): remove obsolete $NVIM detection #15791erw72021-09-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Initially, we planned to set the NVIM environment variable to detect that neovim is running in the neovim built-in terminal. At the time this code was written, there was no way for a parent to set environment variables for a program running in an embedded terminal. Later it was implemented in #12937, but the code to set the NVIM was not added. #11390 now uses ConPTY instead of winpty when possible, so it is no longer necessary to force the use of win32con even when running inside an embedded terminal. Therefore, we now do not need this code.
* | | | Merge pull request #15351 from bfredl/virt_lineBjörn Linse2021-09-26
|\ \ \ \ | | | | | | | | | | feat(screen): virtual lines
| * | | | feat(decorations): support virtual lines (for now: only one block at a time)Björn Linse2021-09-26
| | | | |
* | | | | refactor: replace sprintf with snprintf #15794dundargoc2021-09-26
| | | | |
* | | | | Merge pull request #15721 from bfredl/electricboogaloBjörn Linse2021-09-26
|\ \ \ \ \ | |/ / / / |/| | | | fix(runtime): some ordering issues (and start work on cache mechanism)
| * | | | fix(runtime): ordering of loading packages with user configBjörn Linse2021-09-26
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | site packages must be sourced before user config NOTE: we only consider dirs exactly matching "after" to be an AFTER dir. vim8 considers all dirs like "foo/bar_after", "Xafter" etc, as an "after" dir in SOME codepaths not not in ALL codepaths.
* | | | docs(CONTRIBUTING.md): how to use uncrustify #15780Dundar Göc2021-09-25
| | | | | | | | | | | | | | | | close #15780
* | | | fix(api): fix crash after set_option_value_for() #15390gmntroll2021-09-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: This crashes Nvim: tabedit call nvim_win_set_option(1000, 'statusline', 'status') split wincmd J wincmd j Solution: - Change `no_display` parameter value to be the same as in matching `restore_win_noblock` call. In case of different values `topframe` isn't restored to `curtab->tp_topframe`. - Call `restore_win_noblock` if `switch_win_noblock` returns `FAIL` (`switch_win` must always have matching `restore_win`) - Change `switch_win`/`restore_win` to `_noblock` versions to allow autocommands. fixes #14097 fixes #13577
* | | | refactor: format with uncrustify #15778dundargoc2021-09-25
|/ / / | | | | | | * fixup: force exactly one whitespace between type and variable
* | | Merge pull request #15776 from jamessan/macos-11-ciJames McCoy2021-09-25
|\ \ \ | | | | | | | | Add macos-11 CI
| * | | ci: use runner, instead of os, for job name and cache keyJames McCoy2021-09-24
| | | |
| * | | ci: add macOS 11 buildJames McCoy2021-09-24
| | | |
* | | | test(normal): CA_COMMAND_BUSY in visual select mode #15292Thomas Vigouroux2021-09-24
|/ / / | | | | | | | | | | | | | | | | | | | | | Fix was already applied in 5f144efefa66c9694123b679bab52992279a6015 #15688, but this commit adds another dimension to the test. Test correctly fails after reverting 5f144efefa66c9694123b679bab52992279a6015. ref #15288
* | | Merge #15774 fix(pvs): fix warnings, scriptJustin M. Keyes2021-09-24
|\ \ \
| * | | fix(pvs): Exclude xdiff from analysisJustin M. Keyes2021-09-24
| | | | | | | | | | | | | | | | | | | | ref 088161a9459a ref fbe88ef8f5a4
| * | | fix(PVS): disable "typo" warnings V1051, V1074Justin M. Keyes2021-09-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PVS is worried about typos. Now we need it to stop worrying... Disable these checks entirely, they are all false positives. tui.c:1873 V1074 Boundary between escape sequence and string is unclear. The escape sequence ends with a letter and the next character is also a letter. Check for typos. tui.c:1983 V1074 Boundary between escape sequence and string is unclear. The escape sequence ends with a letter and the next character is also a letter. Check for typos. regexp_nfa.c:6189 V1051 Consider checking for misprints. It's possible that the 'pim->result' should be checked here. screen.c:2928 V1051 Consider checking for misprints. It's possible that the 'vcol_sbr' should be checked here. screen.c:3187 V1051 Consider checking for misprints. It's possible that the 'line_attr' should be checked here. screen.c:3267 V1051 Consider checking for misprints. It's possible that the 'multi_attr' should be checked here. screen.c:4747 V1051 Consider checking for misprints. It's possible that the 'redraw_next' should be checked here. syntax.c:3448 V1051 Consider checking for misprints. It's possible that the 'arg_end' should be checked here. syntax.c:3625 V1051 Consider checking for misprints. It's possible that the 'arg_end' should be checked here. tui.c:1836 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here. tui.c:1863 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here. tui.c:1882 V1051 Consider checking for misprints. It's possible that the 'data->unibi_ext.set_cursor_style' should be checked here.