aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | | Merge pull request #15062 from seandewar/vim-8.1.0829Jan Edmund Lazo2021-07-30
|\ \ \ \ | | | | | | | | | | vim-patch:8.1.0829: when 'hidden' is set session creates extra buffers
| * | | | vim-patch:8.1.0829: when 'hidden' is set session creates extra buffersSean Dewar2021-07-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When 'hidden' is set session creates extra buffers. Solution: Move :badd commands to the end. (Jason Franklin) https://github.com/vim/vim/commit/d39e275b57493f9e25e1b62f84810571eee30cf4 Adjust some tests in ex_cmds/mksession_spec.lua: - 'restores same :terminal buf in splits': Buffers aren't always :badded in the same order as they're :edited, :balted, etc, so the order of buffers in the buffer list may change slightly now that :badd happens afterwards. - 'restores buffers with tab-local CWD': This is explained in a comment.
* | | | | fixup! fix(highlight): remove syncolor.vimJan Edmund Lazo2021-07-29
| | | | | | | | | | | | | | | | | | | | Fix lint errors from lines with 80+ characters.
* | | | | vim-patch:8.2.2957: using getchar() in Vim9 script is problematicJan Edmund Lazo2021-07-29
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Using getchar() in Vim9 script is problematic. Solution: Add getcharstr(). (closes vim/vim#8343) https://github.com/vim/vim/commit/3a7503c34c65ed15cc08deb5b54aaf2ea51525b4 Cherry-pick Test_getchar() changes from patch v8.1.2304 to sync with upstream. Port f_getcharstr() to src/nvim/eval/funcs.c, not src/nvim/getchar.c. Patch v8.1.2042 is not ported yet.
* | | | Merge pull request #15207 from gpanders/bye-syncolorBjörn Linse2021-07-29
|\ \ \ \ | | | | | | | | | | fix(highlight): remove syncolor.vim
| * | | | fixup! fix(highlight): remove syncolor.vimGregory Anders2021-07-27
| | | | |
| * | | | fix(highlight): remove syncolor.vimGregory Anders2021-07-27
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove syncolor.vim in favor of defining the default highlight groups directly in `init_highlight`. This approach provides a number of advantages: 1. The highlights are always defined, regardless of whether or not the syntax regex engine is enabled. 2. Redundant sourcing of syntax files is eliminated (syncolor.vim was often sourced multiple times based on how the user's colorscheme file was written). 3. The syntax highlighting regex engine and the highlight groups themselves are more fully decoupled. 4. Removal of the confusing `:syntax on` / `:syntax enable` dichotomy (they now both do the same thing). This approach also correctly solves a number of bugs related to highlighting (#15176, #12573, #15205).
* | | | Merge pull request #15184 from bfredl/sunjonhlBjörn Linse2021-07-29
|\ \ \ \ | | | | | | | | | | feat(decorations): allow more than one stacked highlight in a virt_text
| * | | | feat(decorations): deprecate nvim_buf_set_virtual_textBjörn Linse2021-07-29
| | | | | | | | | | | | | | | | | | | | Full virt_text functionality is provided by nvim_buf_set_extmark
| * | | | refactor(decorations): merge the two different code paths for virt_textBjörn Linse2021-07-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test(bufhl): CHANGE of tested behaviour (inb4 a proper priority mechanism) test(decoration): change of test; previous behavior was buggy (ghost buffer text)
| * | | | feat(decorations): allow more than one stacked highlight in a virt_textBjörn Linse2021-07-29
| | | | |
* | | | | Merge pull request #15213 from zeertzjq/vim-8.2.3236Thomas Vigouroux2021-07-29
|\ \ \ \ \ | |/ / / / |/| | | | vim-patch:8.2.3236: mode() does not indicate using CTRL-O in Select mode
| * | | | vim-patch:8.2.3236: mode() does not indicate using CTRL-O in Select modezeertzjq2021-07-29
| |/ / / | | | | | | | | | | | | | | | | | | | | Problem: mode() does not indicate using CTRL-O in Select mode. Solution: Use "vs" and similar. (closes vim/vim#8640) https://github.com/vim/vim/commit/eaf3f36168f85c8e0ab7083cd996b9fbe937045d
* / / / fix(decorations): crash when :bdelete (extmark_free_all) after clear_namespaceBjörn Linse2021-07-29
|/ / / | | | | | | | | | fixes #15212
* | | Merge pull request #15197 from ↵Björn Linse2021-07-26
|\ \ \ | | | | | | | | | | | | | | | | dundargoc/refactor/a-song-of-true-and-false/global-variables refactor: replace TRUE/FALSE with true/false
| * | | refactor: replace TRUE/FALSE with true/falseDundar Göc2021-07-26
| | | | | | | | | | | | | | | | Focus is on global variables.
* | | | Merge pull request #15180 from gpanders/xcallocThomas Vigouroux2021-07-26
|\ \ \ \ | |/ / / |/| | | fix: fix incorrect call sites of xcalloc
| * | | fix: fix incorrect call sites of xcallocGregory Anders2021-07-24
| | | | | | | | | | | | | | | | The number of elements comes first and the size of each element second.
* | | | refactor: replace TRUE/FALSE macros with C99 true/falseDundar Göc2021-07-25
| | | |
* | | | Merge pull request #14311 from matveyt/nomode_ceBjörn Linse2021-07-25
|\ \ \ \ | | | | | | | | | | refactor(state): Remove EXMODE_NORMAL
| * | | | Remove EXMODE_NORMALmatveyt2021-07-23
| | | | |
* | | | | chore: PVS/V751: ignore warning #15153dundargoc2021-07-25
| | | | | | | | | | | | | | | | | | | | There's already an ignore directive but it's placed on the wrong line. This commit should correct it.
* | | | | fix: source syncolors.vim after startup scriptsGregory Anders2021-07-24
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue introduced in #14771 (fix: source syncolors.vim before startup scripts) that affected highlights for users who set 'background' to light in their startup script. Because syncolor.vim checks for the value of &background, it was always setting up the 'dark' background colors, which looked wrong for users using light backgrounds. The primary benefit of #14771 is that it decoupled highlighting from the syntax engine. This is useful for e.g. treesitter, which still makes use of highlights even if the syntax engine is disabled. For this reason, it is still worthwhile to source syncolor.vim separately from synload.vim, which #14771 accomplishes. However, we should still source syncolor.vim after the user startup scripts, to ensure that we are respecting the options the user sets. Another corollary benefit is that this reduces some redundancy in highlight definitions, since we now only source syncolors.vim if the user did not already enable a colorscheme.
* | | | Merge pull request #14771 from gpanders/colorschemeBjörn Linse2021-07-22
|\ \ \ \ | |/ / / |/| | | fix: source syncolors.vim before startup scripts
| * | | fix: source syncolors.vim before startup scriptsGregory Anders2021-07-14
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue (#12573) where colorscheme files are sourced twice upon startup. This occurs when the startup script calls `:colorscheme`, which sets the `g:colors_name` global variable. When syntax highlighting is enabled in `syn_maybe_enable()` the `syntax.vim` script is sourced which in turn sources `synload.vim`. This script checks to see if `g:colors_name` is set and, if so, runs exe "colors " . colors_name This is done to ensure that highlight groups are defined before enabling the syntax highlighting engine. Instead, source syncolors.vim before the startup scripts which sets up default highlights and only load the full syntax engine after the startup scripts or when the user runs `:syntax on`. Add a guard variable `did_syncolor` to prevent syncolor.vim from being sourced twice and remove the line mentioned above from synload.vim so that the colorscheme file is not re-sourced when the syntax engine is loaded.
* | | Merge pull request #15127 from bfredl/nlua_initBjörn Linse2021-07-19
|\ \ \ | | | | | | | | refactor(lua): initialize lua state at startup instead of dynamically
| * | | refactor(lua): initialize lua state at startup instead of dynamicallyBjörn Linse2021-07-19
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | lua is used as part of implementation for more core features. As an example, every user keypress will invoke a lua function to check for keypress handlers (regardless if they are registered or not). Thus not starting lua until it is first used doesn't make much sense anymore. nlua_enter was also needed due to the earlier stateful &rtp translation, which by now have been made stateless.
* | | Merge pull request #15070 from dundargoc/PVS/V768James McCoy2021-07-19
|\ \ \ | | | | | | | | [RDY] chore(PVS/V768): don't use enum as bool.
| * | | chore(PVS/V768): don't use enum as bool.Dundar Göc2021-07-19
| |/ /
* / / chore(PVS/V751): tui.c, Parameter is not usedJustin M. Keyes2021-07-18
|/ / | | | | | | | | False positive. Documentation for grid_scroll says "`cols` is always zero, reserved for future use".
* | refactor(options): remove obsolete distinction of "vi" vs "vim" defaultsBjörn Linse2021-07-14
| | | | | | | | | | It might come as a schocking surprise, but the defaults we support are the NEOVIM defaults.
* | refactor(options): delet unused P_VIM flagBjörn Linse2021-07-14
| |
* | Fix line length clint errorGregory Anders2021-07-12
| |
* | Rename stdin to stdin_modeGregory Anders2021-07-12
| | | | | | | | stdin is a macro in Windows builds.
* | feat(job): add parameter to close stdinGregory Anders2021-07-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some programs behave differently when they detect that stdin is being piped. This can be problematic when these programs are used with the job control API where stdin is attached, but not typically used. It is possible to run the job using a PTY which circumvents this problem, but that includes a lot of overhead when simply closing the stdin pipe would suffice. To enable this behavior, add a new parameter to the jobstart options dict called "stdin" with two valid values: "pipe" (the default) implements the existing behavior of opening a channel for stdin and "null" which disconnects stdin (or, if you prefer, connects it to /dev/null). This is extensible so that other modes can be added in the future.
* | chore(clang): Fix 'Array subscript is undefined' #15057ansimita2021-07-11
| |
* | window.c: address Coverity failure #14996Daniel Steinberg2021-07-10
| | | | | | | | | | | | | | | | | | Check that `wip2` does not point to the same address as `wip`, to address the Coverity test failure from PR #14884. Based on the `if` clauses, `free_wininfo(wip2, ...)` is only called when `wip2->wi_win == NULL` and `wip->wi_win == wp`. I think `wip2` would only point to the same address as `wip` in scenarios where `wp` were `NULL`, which can be assumed otherwise based on the earlier code.
* | refactor(undo): don't assume curbuf in u_compute_hashThomas Vigouroux2021-07-09
| |
* | Merge pull request #12971 from vigoux/decurbufBjörn Linse2021-07-09
|\ \ | | | | | | Decrease reliance on curbuf in BUFEMPTY and `undo.c`
| * | refactor(undo): remove last occurence of save_curbufThomas Vigouroux2021-07-08
| | |
| * | refactor: remove all mentions to `curbuf_lock`Thomas Vigouroux2021-07-08
| | |
| * | style(undo): remove unnecessary commentThomas Vigouroux2021-07-06
| | |
| * | fix(qf): use correct buffer fieldThomas Vigouroux2021-07-06
| | |
| * | buffer: move BUFEMPTY to a functionThomas Vigouroux2021-07-06
| | |
| * | undo: reduce reliance on curbufThomas Vigouroux2021-07-06
| | |
| * | buffer: don't rely on curbuf in BUFEMPTYThomas Vigouroux2021-07-06
| | |
* | | vim-patch:8.2.3125: variables are set but not used #15028dundargoc2021-07-08
| | | | | | | | | | | | | | | | | | Problem: Variables are set but not used. Solution: Move the declarations to the block where they are used. (closes vim/vim#8527) https://github.com/vim/vim/commit/09f688c33aad9692276dfb68842cf0621a0e2002
* | | vim-patch:8.2.3089: garbage collection has useless code #15027dundargoc2021-07-08
| | | | | | | | | | | | | | | Problem: Garbage collection has useless code. Solution: Bail out when aborting. (closes vim/vim#8504) https://github.com/vim/vim/commit/20cc52832067d22a3cd47dfb0805251228c32e7d
* | | chore: use codespell to spell check #15016dundargoc2021-07-07
| | |
* | | clang/'Dead initialization': don't assign unused value to variable "string" ↵dundargoc2021-07-07
| | | | | | | | | | | | #15015