aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | | 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
| * | | | | | Makefile: add PREFIX variableMarco Hinz2016-10-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We use a Makefile which in turn uses cmake. If we wanted to set the install prefix for cmake, we had to do this so far: make CMAKE_FLAGS="-DCMAKE_INSTALL_PREFIX=/tmp/nvim" That's long and hard to remember. Following the conventions of other Makefiles, this now works as well and is equivalent: make PREFIX=/tmp/nvim
* | | | | | | Merge pull request #5451 from jamessan/set-username-hostnameJames McCoy2016-10-08
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | config: Allow setting USERNAME/HOSTNAME from the environment
| * | | | | | | config: Allow setting USERNAME/HOSTNAME from the environmentJames McCoy2016-10-08
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allowing this to be controlled externally improves reproducibility, as well as provides a more useful address to report for "Compiled by". For example, I intend to set it to the packaging list when building the Debian package. Signed-off-by: James McCoy <jamessan@jamessan.com>
* | | | | | | Merge #5428 'vim-patch: 0c1ff16, 939a1ab, 8067a64'.Justin M. Keyes2016-10-08
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | vim-patch:0c1ff16Shougo Matsushita2016-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | updated runtime files. Add avra syntax. https://github.com/vim/vim/commit/0c1ff16b5467f97ce08134fdbc8198127bbe492a
| * | | | | | vim-patch:939a1abShougo Matsushita2016-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/939a1abe935a539f2d4c90a56cb0682cbaf3bbb0
| * | | | | | vim-patch:8067a64Shougo Matsushita2016-10-04
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing test file. https://github.com/vim/vim/commit/8067a64852d6d134b493c5674e404225ed4bbe7d
* | | | | | CheckHealth: choose correct path for the latest version (#5446)Marco Hinz2016-10-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If multiple versions of a package are installed, the provider health check could choose a wrong path: /usr/local/lib/python3.5/site-packages/neovim-0.1.10-py3.5.egg-info/PKG-INFO /usr/local/lib/python3.5/site-packages/neovim-0.1.9-py3.5.egg-info/PKG-INFO Prior to this change :CheckHealth could falsely show 0.1.9 as the installed version, since glob() doesn't enforce any predictable order. Now we sort all potential paths numerically in descending order and just look at the first path instead.
* | | | | | hardcopy.c: Fix -Wmaybe-uninitialized warning.Justin M. Keyes2016-10-08
| | | | | |
* | | | | | lint: Removing dead initializations #5410Patrick Jackson2016-10-08
| |_|/ / / |/| | | |
* | | | | Merge #5444 from malept/dont-hardcode-python3-in-checkhealthJustin M. Keyes2016-10-07
|\ \ \ \ \ | | | | | | | | | | | | Fix Python 2/3 health checks
| * | | | | health: fix Python 2 variable namesMark Lee2016-10-06
| | | | | |
| * | | | | health: remove duplicate nvim_path declarationMark Lee2016-10-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's the same as the declaration above it, but hardcoded to use python3 and does not redirect stderr.
* | | | | | tui/flush_buf: Don't toggle cursor when called from out() #5436Florian Larysch2016-10-06
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | unibi_format() calls out() multiple times for a given format string. When data->buf fills up during this process, flush_buf() gets called, which possibly calls unibi_out() again to toggle the cursor visibility. However, if we were halfway through outputting an escape sequence, doing this will clobber it, resulting in junk being displayed. Fix this by not toggling the cursor visibility when draining a full buffer in out().
* | | | | version.c: mark NA patches (#5380)Shougo2016-10-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NA list: 1653: matchit packadd 1750: channel 1770: termguicolors 1794: GUI Win32 1805: Makefile 1804, 1945: manpager.vim 1811: channel 2115, 2232, 2278, 2319: defaults.vim included: 1763: spell
* | | | | doc/vim_diff.txt (#5432)Justin M. Keyes2016-10-06
| |_|/ / |/| | |
* | | | Merge pull request #5366 from NovaDev94/vim-7.4.1740James McCoy2016-10-04
|\ \ \ \ | |/ / / |/| | | vim-patch:7.4.1740
| * | | vim-patch:7.4.1740Nova2016-09-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: syn-cchar defined with matchadd() does not appear if there are no other syntax definitions which matches buffer text. Solution: Check for startcol. (Ozaki Kiichi, haya14busa, closes vim/vim#757) https://github.com/vim/vim/commit/4d585022023b96f6507e8cae5ed8fc8d926f5140
* | | | tui/terminfo_start: Default to normal-mode cursor shape. (#5419)Justin M. Keyes2016-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | References #4867 For users who use a "bar" shape in the shell, it's annoying that Nvim starts with that same cursor shape, despite starting in normal-mode. So default to the normal-mode "block" shape instead. (Note: technically it's possible some user may set 'insertmode', and then the opposite problem occurs. But 'insertmode' is a silly option that shouldn't exist, and any user that uses it probably isn't fiddling with their cursor shape anyways.) Also rename the unibilium extensions: enter_insert_mode => set_cursor_shape_bar enter_replace_mode => set_cursor_shape_underline exit_insert_mode => set_cursor_shape_block to say explicitly what they do in the context of a terminal; it's irrelevant in this context what purpose they serve in Nvim.
* | | | ci: .travis.yml: Minimize scope of osx_image directive. (#5420)Justin M. Keyes2016-10-02
| | | |
* | | | event/multiqueue.c: Rename "queue" to "multiqueue".Justin M. Keyes2016-10-02
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `lib/queue.h` implements a basic queue. `event/queue.c` implements a specialized data structure on top of lib/queue.h; it is not a "normal" queue. Rename the specialized multi-level queue implemented in event/queue.c to "multiqueue", to avoid confusion when reading the code. Before this change one can eventually notice that "macros (uppercase symbols) are for the normal queue, lowercase operations are for the multi-level queue", but that is unnecessary friction for new developers (or existing developers just visiting this part of the codebase).
* | | Merge pull request #5409 from jamessan/toplevel-stateJames McCoy2016-09-30
|\ \ \ | | | | | | | | | | | | Correct logic for setting NormalState.toplevel
| * | | Correct logic for setting NormalState.toplevelJames McCoy2016-09-30
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Vim's main_loop function, the main loop is while (!cmdwin #ifdef FEAT_CMDWIN || cmdwin_result == 0 #endif ) { ... #ifdef FEAT_EVAL /* * May perform garbage collection when waiting for a character, but * only at the very toplevel. Otherwise we may be using a List or * Dict internally somewhere. * "may_garbage_collect" is reset in vgetc() which is invoked through * do_exmode() and normal_cmd(). */ may_garbage_collect = (!cmdwin && !noexmode); #endif /* * If we're invoked as ex, do a round of ex commands. * Otherwise, get and execute a normal mode command. */ if (exmode_active) { if (noexmode) /* End of ":global/path/visual" commands */ return; do_exmode(exmode_active == EXMODE_VIM); } else normal_cmd(&oa, TRUE); } cmdwin_result is set to 0 before calling main_loop to handle the cmdwin window and gets changed when the user causes a command to execute (either through pressing <CR> or <C-c>). This means that when the cmdwin isn't active OR the user is still editing their command, main_loop runs and main_loop calls normal_cmd with toplevel true as long as exmode isn't active. When the normal mode state was extracted in dae006a9, the conditions for toplevel and may_garbage_collect were combined. Since toplevel was set to always ignore cmdwin, the v:count(1) variables were no longer being updated when a command was prefixed with a count in the cmdwin. Closes #5404
* | | Merge pull request #5358 from jbradaric/vim-7.4.1679James McCoy2016-09-28
|\ \ \ | | | | | | | | vim-patch:7.4.{1648,1679,1681}
| * | | vim-patch:7.4.1681Jurica Bradaric2016-09-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Coverity warns for fixed size buffer length (false positive). Solution: Add a check for the name length. https://github.com/vim/vim/commit/ef9d9b94a8803c405884bb6914ed745ede57c596
| * | | vim-patch:7.4.1679Jurica Bradaric2016-09-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Coverity: copying value of v_lock without initializing it. Solution: Init v_lock in rettv_list_alloc() and rettv_dict_alloc(). https://github.com/vim/vim/commit/7d2a5796d39905a972e8f74af5f7b0a62e3de173
| * | | vim-patch:7.4.1648Jurica Bradaric2016-09-28
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: Compiler has a problem copying a string into di_key[]. (Yegappan Lakshmanan) Solution: Add dictitem16_T. https://github.com/vim/vim/commit/bee6c0cf86a75faa2aa893f2c9db82fd944a89a5
* | | Merge #5395 from justinmk/logJustin M. Keyes2016-09-28
|\ \ \
| * | | log.c: align/adjust layoutJustin M. Keyes2016-09-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New: 2016/09/27 18:00:49 INFO 28117/tui_flush:597: ... Old: 2016/09/27 18:00:49 [info @ tui_flush:597] 28117 - ...
| * | | tui: Move ui_bridge module to tui/ namespace.Justin M. Keyes2016-09-28
| | | |