aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | [RFC] vim-patch:7.4.1704 (#5487)Michael Ennen2016-10-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | vim-patch:7.4.1704 Problem: Using freed memory with "wincmd p". (Dominique Pelle) Solution: Also clear "prevwin" in other tab pages. https://github.com/vim/vim/commit/3dda7db4e1f7c4a8110a1f83001ec36b46693d27
* | | Merge pull request #5441 from Shougo/vim-7.4.1588James McCoy2016-10-20
|\ \ \ | | | | | | | | | | | | vim-patch:7.4.1588
| * | | vim-patch:7.4.1588Shougo Matsushita2016-10-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Old style test for quickfix. Solution: Turn test 96 into a new style test. https://github.com/vim/vim/commit/1ff2b64b11e7d263c6853745d5e594bd8f94b91e
* | | | Merge pull request #5440 from Shougo/vim-7.4.1565James McCoy2016-10-20
|\ \ \ \ | | | | | | | | | | vim-patch:7.4.1565
| * | | | vim-patch:7.4.1565Shougo Matsushita2016-10-15
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when assert_equal() runs into a NULL string. Solution: Check for NULL. (Dominique) Add a test. https://github.com/vim/vim/commit/f1551964448607f8222de2d8f0992ea43eb2fe67
* | | | 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.
* | | | | 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
| | | |
* | | | 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
* | | docJustin M. Keyes2016-10-16
| | |
* | | 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>
* | | | | 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.
* | | | | 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.
* | | | Merge #5257 from jbradaric/vim-7.4.1893Justin M. Keyes2016-10-13
|\ \ \ \ | | | | | | | | | | vim-patch:7.4.{1893,1895}
| * | | | 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.
| * | | | | | | terminal: Move re-edit detection to do_ecmd(). #5445Harm te Hennepe2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #4784
* | | | | | | | vim-patch:7.4.2274 (#5439)Harm te Hennepe2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Command line completion on "find **/filename" drops sub-directory. Solution: Handle this case separately. (Harm te Hennepe, closes vim/vim#932, closes vim/vim#939) https://github.com/vim/vim/commit/73d4e4c8922f6f4d256f910a18f47c0c3a48c28b
* | | | | | | | ui: Blank the next cell for any multi-cell characterJames McCoy2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Followup for #5461
* | | | | | | | Merge pull request #5461 from jamessan/emoji-ui-regressionJames McCoy2016-10-12
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | ui: Fix the call to utf_ambiguous_width
| * | | | | | | ui: Fix the call to utf_ambiguous_widthJames McCoy2016-10-11
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `utf_ambiguous_width` expects the Unicode character, but in 9e1c6596 I just passed the first UTF-8 byte to the function. This led to various display problems because now many multi-cell characters weren't falling into that part of the branch. Also, to better align with the existing Vim code, remove the forced cursor update. Setting the flag will cause it to happen in the next UI_CALL. Thanks to qvacua for all the help investigating the issue! Closes #5448
* | | | | | | Merge pull request #5466 from jszakmeister/fix-generated-files-for-in-tree-buildJohn Szakmeister2016-10-12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix generated files for in-tree build.
| * | | | | | | build: fix generated file paths when building in-treeJohn Szakmeister2016-10-11
| | | | | | | |
* | | | | | | | eval/term_write(): Skip writes if stream was closed.Justin M. Keyes2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the backing stream for a :terminal was closed (e.g. if the shell exits unexpectedly) there may be pending input on the loop which will be processed before the terminal close event (which is queued on the same loop). terminal_send checks term->closed but this does not reflect the state of the underlying streams. The terminal.c module in fact has no knowledge of the streams (this seems intentional: it is abstracted as TerminalOption.write_cb). The SIGCHLD handler (pty_process_unix.c) is executed immediately, and it triggers a stream teardown so Stream.closed=false (TerminalJobData.in.closed). When the pending writes are handled by eval.c:term_write, wstream_write() aborts because it sees the closed Stream. To avoid that, this commit checks Stream.closed in eval:term_write() before writing to the WStream. (As hinted above, we cannot do this in terminal:terminal_send() because that module cannot inspect the underlying streams.) References #5445 https://github.com/neovim/neovim/pull/5445#issuecomment-252529766
* | | | | | | | test/terminal: Cover race when :term shell process exits.Justin M. Keyes2016-10-12
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | References #5445 See https://github.com/neovim/neovim/pull/5445#issuecomment-252529766
* / / / / / / window: Fix cmd_with_count's formatting when sizeof(long) != 8James McCoy2016-10-11
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On architectures where `sizeof(long)` != 8, "%" PRId64 will read junk from memory. This was seen on various Debian builds where test/functional/legacy/close_count_spec.lua would fail due to `1<C-w>c` emitting an error like `E488: Trailing characters: close-87944975647104`. Changing the `Prenum` parameter to int64_t ensures it is safe to use `"%" PRId64`, and make another small step towards removal of the use of `long`.
* | | | | | Merge pull request #5411 from P4Cu/vim-7.4.1549James McCoy2016-10-08
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch:7.4.1549
| * | | | | | vim-patch:7.4.1549Andrzej Pacanowski2016-09-30
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test for syntax attributes fails in Win32 GUI. Solution: Use an existing font name. https://github.com/vim/vim/commit/c835293d54c223627c7d4516ee273c21a3506fa1
* | | | | | Merge #5428 'vim-patch: 0c1ff16, 939a1ab, 8067a64'.Justin M. Keyes2016-10-08
|\ \ \ \ \ \