aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | 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.
| * | | | | | terminal: Move re-edit detection to do_ecmd(). #5445Harm te Hennepe2016-10-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #4784
* | | | | | | docJustin M. Keyes2016-10-13
| | | | | | | | | | | | | | | | | | | | | yuck
* | | | | | | docJustin M. Keyes2016-10-12
| | | | | | |
* | | | | | | 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 #5467 from jszakmeister/prevent-in-tree-buildsJohn Szakmeister2016-10-12
|\ \ \ \ \ \ | | | | | | | | | | | | | | build: prevent in-tree builds for the time being, as it's not supported
| * | | | | | build: prevent in-tree builds for the time being, as it's not supportedJohn Szakmeister2016-10-11
| | | | | | |
* | | | | | | 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
| |/ / / / / /
* | | | | | | Merge #5463 from justinmk/te-skip-writesJustin M. Keyes2016-10-12
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | term_write(): Skip writes if stream was closed.
| * | | | | | 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
* | | | | | UpdateRemotePlugins: Discard duplicate paths. (#5464)Shougo2016-10-12
| | | | | |
* | | | | | Merge pull request #5457 from jamessan/wincmd-countJames McCoy2016-10-11
|\ \ \ \ \ \ | |/ / / / / |/| | | | | window: Fix cmd_with_count's formatting when sizeof(long) != 8
| * | | | | 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 pull request #5447 from mhinz/makefile/add-prefixJames McCoy2016-10-08
|\ \ \ \ \ | | | | | | | | | | | | Makefile: add PREFIX variable