aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | Merge pull request #5503 from jamessan/vim-7.4.1494James McCoy2016-10-20
|\ \ \ \ | | | | | | | | | | vim-patch:7.4.1494
| * | | | hist_char2type: Return '/' for NUL.James McCoy2016-10-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During -S execution, ccline.cmdfirstc is NULL, so calling histadd() with a hist-name of '' would fail. Mimic Vim's behavior and fallback to '/' in this scenario, since '' is a documented hist-name as of vim/vim@3e496b0e.
| * | | | vim-patch:7.4.1494Patrick2016-10-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: clr_history() does not work properly. Solution: Increment hisptr. Add a test. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/119d4693e06e68d4f099aa7287e375ae3d265fd0 Note: Only added a test. The change to `histptr` had already been made.
* | | | | man.vim: silence `file` call (#5509)Colin Caine2016-10-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a file is opened by nvim with ft=man already set, and "has('vim_starting')", ftplugin/man.vim calls 'execute 'file man://'.ref', this causes nvim to display something like this: ```` "<name of original file>" 977, 41017C "man://foo(1)" [Not edited] 977 lines --0%-- Press ENTER or type command to continue ```` This is annoying, because nothing of note has actually happened. Use cases why you might want to read a man page from a file: `MANPAGER='bash -c "nvim -c \"set ft=man\" </dev/tty <(col -bx)"' man git` `nvim -c 'set ft=man' <(man -P cat git)`
* | | | | Merge pull request #5507 from bfredl/terminal_apiBjörn Linse2016-10-19
|\ \ \ \ \ | | | | | | | | | | | | tests: fix racyness in terminal/api_spec.lua
| * | | | | tests: fix racyness in terminal/api_spec.luaBjörn Linse2016-10-19
|/ / / / / | | | | | | | | | | | | | | | | | | | | Previously, the nvim_input from the socket channels could be processed before the input from stdin in rare cases.
* | | | | Merge pull request #5393 from bfredl/dispatchfixBjörn Linse2016-10-19
|\ \ \ \ \ | |_|_|/ / |/| | | | refactor gendispatch.lua to move verbatim c code to .c files
| * | | | api: move verbatim c code out of gendispatch.lua and into c filesBjörn Linse2016-10-19
|/ / / / | | | | | | | | | | | | | | | | Remove max_fname_len check, which caused false successful lookups, and was an optimization for a very rare case.
* | | | folds: Do not auto-update folds for some foldmethods. #5426Shougo Matsushita2016-10-19
| | | |
* | | | Merge #5502 from justinmk/error-write-to-bg-processJustin M. Keyes2016-10-19
|\ \ \ \ | | | | | | | | | | system('foo &', 'bar'): Show error, don't crash.
| * | | | system('foo &', 'bar'): Show error, don't crash.Justin M. Keyes2016-10-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #3529 Closes #5241 In Vim, :echo system('cat - &', 'foo') works because for both system() and :! Vim writes input to a temp file and uses shell syntax to redirect the file to the backgrounded `cat` (get_cmd_output() .. make_filter_cmd()). In Nvim, :echo system('cat - &', 'foo') fails because we write the input directly via pipes (shell.c:do_os_system()), but (per POSIX[1]) backgrounded process input stream is redirected from /dev/null (unless overridden by shell redirection; supported only by some shells [2]), so our writes are ignored, the process exits quickly, and if we are writing data larger than the buffer size we'll see EPIPE. This still works: :%w !tee > foo1358.txt & but this does not: :%w !tee foo1358.txt & though it *should* (why doesn't it?) because we still do the temp file dance in do_bang() .. do_filter(). [1] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_09_03_02 [2] http://unix.stackexchange.com/a/71218
| * | | | test: system(): backgrounded shell commandJustin M. Keyes2016-10-19
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These tests are essentially affirming a regression vs Vim. In Vim, :echo system('cat - &', 'foo') returns "foo", because Vim internally wraps the command with shell-specific syntax to redirect the streams from /dev/null[1]. That can't work in Nvim because we use pipes directly (instead of temp files) and don't wrap the command with shell-specific redirection syntax. References #3529 References #5241 [1] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_09_03_02
* / / / rplugin: resolve paths in manifest file (#5501)Marco Hinz2016-10-18
|/ / / | | | | | | | | | | | | | | | | | | `:CheckHealth nvim` would always report an outdated manifest if symlinks were used, because the manifest file contains unresolved paths that get compared against resolved paths. Now we resolve paths before they get written to the manifest file.
* | | Doc: Ruby omni-completion works (#5497)Marco Hinz2016-10-17
| | |
* | | Merge #5493 from justinmk/checkhealthJustin M. Keyes2016-10-16
|\ \ \ | | | | | | | | CheckHealth: check tmux, terminfo
| * | | CheckHealth: check terminfoJustin M. Keyes2016-10-16
| | | |
| * | | CheckHealth: check tmux configurationJustin M. Keyes2016-10-16
| | | |
| * | | docJustin M. Keyes2016-10-16
| | | |
* | | | Revert "Makefile: add PREFIX variable"Marco Hinz2016-10-16
|/ / / | | | | | | | | | | | | | | | This reverts commit c13f72ee0a6600963a10e57155d2e814d56599e2. References #5447.
* | | test: execute_spec: Adjust screen assertion.Justin M. Keyes2016-10-15
| | | | | | | | | | | | | | | | | | The previous form was passing because it happens immediately before this form; but on a very fast(?) server the screen check might miss that form. It's also not really want we want to assert anyways.
* | | vim-patch: 7.4.1619 (#5475)Michael Ennen2016-10-16
| | | | | | | | | | | | | | | | | | | | | | | | Problem: When 'fileformats' is set in the vimrc it applies to new buffers but not the initial buffer. Solution: Set 'fileformat' when starting up. (Mike Williams) https://github.com/vim/vim/commit/364fa5c7ec2a99a791c8f8b66fe70b0bf1dd9a41
* | | vim-patch:7.4.1833 (#5484)Shougo2016-10-15
| | | | | | | | | | | | | | | | | | Problem: Cannot use an Ex command for 'keywordprg'. Solution: Accept an Ex command. (Nelo-Thara Wallus) https://github.com/vim/vim/commit/2ff8b64679242e73248774a388d54931c9ce49bd
* | | Merge pull request #5424 from jamessan/win-tab-nrJames McCoy2016-10-15
|\ \ \ | | | | | | | | api: Support getting the number of a window/tabpage
| * | | api: Support getting the number of a window/tabpageJames McCoy2016-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to provide better compatibility with the classic bindings, the API needs to provide the ability to query the number (really index) of the window/tabpage. This is needed for neovim/python-client#87, as discussed in neovim/neovim#1898. Signed-off-by: James McCoy <jamessan@jamessan.com>
* | | | Merge pull request #5456 from jamessan/char-sign-conversionJames McCoy2016-10-15
|\ \ \ \ | | | | | | | | | | Enable CI testing of unsigned char and fix fallout
| * | | | lintJames McCoy2016-10-09
| | | | |
| * | | | if_cscope: Fix conversion warnings when char defaults to unsignedJames McCoy2016-10-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ../src/nvim/if_cscope.c: In function 'cs_read_prompt': ../src/nvim/if_cscope.c:1771:47: warning: comparison is always true due to limited range of data type [-Wtype-limits] while ((ch = (char)getc(csinfo[i].fr_fp)) != EOF && ch != CSCOPE_PROMPT[0]) ^~ ../src/nvim/if_cscope.c:1804:14: warning: comparison is always false due to limited range of data type [-Wtype-limits] if (ch == EOF) { ^~ ../src/nvim/if_cscope.c:1816:14: warning: negative integer implicitly converted to unsigned type [-Wsign-conversion] ch = EOF; ^~~ ../src/nvim/if_cscope.c:1821:12: warning: comparison is always false due to limited range of data type [-Wtype-limits] if (ch == EOF) ^~ Since EOF is -1, it will be converted to a large unsigned value to compare with unsigned char and never match. Use an int to store the return from getc so we can safely compare it and, once known to be valid, cast it to char when storing it into buf. Signed-off-by: James McCoy <jamessan@jamessan.com>
| * | | | Change constack.cs_flags from char to intJames McCoy2016-10-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the -Wconversion warning when char's type is unsigned. ../src/nvim/ex_eval.c: In function 'ex_while': ../src/nvim/ex_eval.c:1000:28: warning: conversion to 'char' from 'int' may alter its value [-Wconversion] cstack->cs_lflags &= ~CSL_HAD_LOOP; ^ Signed-off-by: James McCoy <jamessan@jamessan.com>
| * | | | ci: Force unsigned char usage for the 32bit buildJames McCoy2016-10-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since C leaves whether char is signed or unsigned up to the implementer, there are different defaults on different architectures. Forcing unsigned char for one of our CI builds should help catch these issues moving forward.
* | | | | Merge pull request #5384 from bfredl/getchar_eventBjörn Linse2016-10-15
|\ \ \ \ \ | |_|_|/ / |/| | | | allow event processing in getchar()
| * | | | events: allow event processing in getchar()Björn Linse2016-10-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this is consistent with vim, and is necessary for plugins that implement their own input modes using "getchar()" and still want to do async event processing.
* | | | | startup: Remove 'N files to edit' message. #5481Linda_pp2016-10-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When '--embed' passed to command line arguments, stdin and stdout are used for IPC. But when multiple files are passed as arguments, nvim wrongly sends 'N files to edit' message to its stdout. As the result, attaching to process from frontend failed. Solution: Remove the message because it doesn't fill any purpose.
* | | | | health/provider.vim: Pass errors through. Give more feedback.Justin M. Keyes2016-10-13
| | | | |
* | | | | Revert "tui: Move ui_bridge module to tui/ namespace."Justin M. Keyes2016-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ui_bridge.c is useful for libnvim consumers, not just the built-in TUI. This reverts commit beb2e4f095583af09ebe9c66e3bf453b61511f23.
* | | | | health/provider.vim: Fix system([]) invocations.Justin M. Keyes2016-10-13
| | | | | | | | | | | | | | | | | | | | Closes #5435
* | | | | test/tui_spec: Force LANG=CJustin M. Keyes2016-10-13
|/ / / / | | | | | | | | | | | | References #5455
* | | | Merge #5257 from jbradaric/vim-7.4.1893Justin M. Keyes2016-10-13
|\ \ \ \ | | | | | | | | | | vim-patch:7.4.{1893,1895}
| * | | | server_requests_spec: Expect correct window ID.Jurica Bradaric2016-10-07
| | | | |
| * | | | eval: Exit early if argument is invalid.Jurica Bradaric2016-10-07
| | | | |
| * | | | vim-patch:7.4.1895Jurica Bradaric2016-10-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot use a window ID where a window number is expected. Solution: Add LOWEST_WIN_ID, so that the window ID can be used where a number is expected. https://github.com/vim/vim/commit/888ccac8902cee186fbd47e971881f6d9b19c068
| * | | | vim-patch:7.4.1893Jurica Bradaric2016-10-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot easily get the window ID for a buffer. Solution: Add bufwinid(). https://github.com/vim/vim/commit/b3619a90eae2702553ff9494ecc4c9b20c13c224
* | | | | Merge #5418 from Shougo/vim-7.4.2158Justin M. Keyes2016-10-13
|\ \ \ \ \ | | | | | | | | | | | | vim-patch: 7.4.2158, 2162, 2205
| * | | | | vim-patch:7.4.2205Shougo Matsushita2016-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: 'wildignore' always applies to getcompletion(). Solution: Add an option to use 'wildignore' or not. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/e9d58a6459687a1228b5aa85bd7b31f8f1e528a8
| * | | | | vim-patch:7.4.2162Shougo Matsushita2016-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Result of getcompletion('', 'sign') depends on previous completion. Solution: Call set_context_in_sign_cmd(). (Dominique Pelle) https://github.com/vim/vim/commit/7522f6982197f83a5c0f6e9af07fb713934f824a
| * | | | | vim-patch:7.4.2158Shougo Matsushita2016-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Result of getcompletion('', 'cscope') depends on previous completion. (Christian Brabandt) Solution: Call set_context_in_cscope_cmd(). https://github.com/vim/vim/commit/b650b9878e9f0ac6bb1b61230095ad9ab3850a33
* | | | | | Merge #5427 from Shougo/vim-7.4.1714Justin M. Keyes2016-10-13
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch: 7.4.1765, 7.4.1714
| * | | | | | vim-patch:7.4.1765Shougo Matsushita2016-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Undo options are not together in the options window. Solution: Put them together. (Gary Johnson) https://github.com/vim/vim/commit/4694a17d1ec08382f996990a7fac1ac60197ec81
| * | | | | | vim-patch:7.4.1714Shougo Matsushita2016-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Non-GUI specific settings in the gvimrc_example file. Solution: Move some settings to the vimrc_example file. Remove setting 'hlsearch' again. (suggested by Hirohito Higashi) https://github.com/vim/vim/commit/54f1b7abf8c48b1dd997202258d1d0673ed4bd29
* | | | | | | Merge #5470 from justinmk/pr5445Justin M. Keyes2016-10-13
|\ \ \ \ \ \ \
| * | | | | | | main.c: "BufReadCmd term://": Skip existing terminal.Justin M. Keyes2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check `exists('b:term_title')` to avoid the BufReadCmd for already-initialized :terminal buffers. Move the test for `:argadd`. Add a test for `:edit<CR>`. Tweak comments and code style.
generated by cgit (git 2.34.1) at 2025-07-19 06:10:23 +0000