aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | open_buffer(): Do `BufEnter` for directories.Justin M. Keyes2017-01-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Abuse NOTDONE to give some nuance to the return value of readfile(), so that open_buffer() can distinguish between "failed, lol" and "failed because the path is a directory". Before this change, Vim *already* creates a new buffer when a directory is edited. So there is no reason it should not raise BufEnter, that was an implementation detail of ye olde readfile(). Most of the changes in this commit merely preserve the old semantics. The "implicit" change that we actually are interested in, is this line in `open_buffer()`, where `retval` being non-FAIL allows EVENT_BUFENTER to be applied: apply_autocmds_retval(EVENT_BUFENTER, NULL, NULL, FALSE, curbuf, &retval); References https://github.com/vim/vim/issues/1353
* | | | Merge pull request #5935 from jamessan/dictwatcher-crashJames McCoy2017-01-12
|\ \ \ \ | |/ / / |/| | | eval: Remove dictwatcher from watchers queue before freeing it
| * | | eval: Remove dictwatcher from watchers queue before freeing itJames McCoy2017-01-12
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a use-after-free noticed by ASAN which would occur when a dictwatcher was still active on a dictionary when the dictionary was freed. fun! MakeWatch() let d = {'foo': 'bar'} call dictwatcheradd(d, 'foo', function('...')) endfun Patch-by: oni-link Closes #5930
* | | Merge pull request #5932 from jamessan/icm-fixesJames McCoy2017-01-12
|\ \ \ | | | | | | | | inccommand: Ignore errors during preview and make cmd_can_preview stricter
| * | | inccommand: Preview :sub commands only after the delimiter is presentJames McCoy2017-01-11
| | | | | | | | | | | | | | | | Closes #5888
| * | | inccommand: Suppress error reporting when previewing commandsJames McCoy2017-01-11
|/ / / | | | | | | | | | Closes #5912
* | | Windows: vim_getenv(): Find runtime relative to nvim. #3303 (#5929)Justin M. Keyes2017-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Windows we cannot rely on absolute install paths to point to the location of the runtime. Vim uses the path of the current binary as a possible location for the runtime folder. In Neovim the install location places the runtime folder in ../share/nvim/runtime. In Vim this logic is guarded by USE_EXE_NAME, which is defined for win32 and macOS. TODO: We may need to incorporate similar logic for macOS: https://github.com/vim/vim/blob/0cdb72aa38c4a0140c94d56bf8bc17cb30260ebf/src/misc1.c#L4287-L4308
* | | Merge #5910 from justinmk/win32-jobstartJustin M. Keyes2017-01-11
|\ \ \ | | | | | | | | Windows: fix jobstart()
| * | | test: system([...]): v:shell_errorJustin M. Keyes2017-01-11
| | | |
| * | | system([...]): Set v:shell_error=-1 if not executable.Rui Abreu Ferreira2017-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do _not_ set v:shell_error on parameter validation error. system([...]) does not invoke a shell, so this change is somewhat questionable. But `:help v:shell_error` is sufficiently vague to allow -1 in this case.
| * | | Windows: ci/AppVeyor: Enable Python provider testsRui Abreu Ferreira2017-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Setup python2/3 and install the Neovim client in Appveyor to enable the python tests. - Use the Python installation provided by Appveyor, because dependencies pyuv, greenlet have issues compiling with MinGW-w64. And this way we avoid building those too. - Copy python.exe => python3.exe so that the python provider can find python3.
| * | | test: system([...])Rui Abreu Ferreira2017-01-11
| | | |
| * | | Windows: libuv_process_spawn(): Allow libuv argument quoting/escaping.Rui Abreu Ferreira2017-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #5360 References #3305 Reverts commit dc9652e68de163290abee880a74bf1727c715a1e. Disabling the quoting was does not solve the problem in general, and we would end up having to handle the quoting ourselves. See: https://github.com/JuliaLang/julia/issues/13776
* | | | clipboard: only check for pbcopy on macOS (#5927)Marco Hinz2017-01-11
|/ / / | | | | | | Fixes #5926.
* | | Merge pull request #5924 from jamessan/vim-7.4.2100James McCoy2017-01-10
|\ \ \ | | | | | | | | vim-patch:7.4.2100
| * | | lintJames McCoy2017-01-10
| | | |
| * | | vim-patch:7.4.2100James McCoy2017-01-10
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: "cgn" and "dgn" do not work correctly with a single character match and the replacement includes the searched pattern. (John Beckett) Solution: If the match is found in the wrong column try in the next column. Turn the test into new style. (Christian Brabandt) https://github.com/vim/vim/commit/6835dc61aebca2b602d85a9d63c449ace58683b4 Closes #5796
* | | Merge pull request #5863 from ZyX-I/more-clint-checksJames McCoy2017-01-10
|\ \ \ | | | | | | | | More clint brace checks
| * | | clint: Check for misplaced brace at function startZyX2017-01-03
| | | |
| * | | clint: Enable check for `{` positioned at the start of the line correctlyZyX2017-01-03
| | |/ | |/| | | | | | | For some reason that was incorrectly hidden by “file is *not* \*.c or \*.h file” check.
* | | Merge pull request #5919 from jamessan/vim-7.4.2008James McCoy2017-01-10
|\ \ \ | | | | | | | | vim-patch:7.4.2008,7.4.2009
| * | | lintJames McCoy2017-01-10
| | | |
| * | | vim-patch:7.4.2009James McCoy2017-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Messages test fails. Solution: Don't set redir_execute before returning. https://github.com/vim/vim/commit/ed59aa60d3905f935283727f4a7b33c81a00174b
| * | | vim-patch:7.4.2008James McCoy2017-01-10
|/ / / | | | | | | | | | | | | | | | | | | | | | Problem: evalcmd() has a confusing name. Solution: Rename to execute(). Make silent optional. Support a list of commands. https://github.com/vim/vim/commit/79815f1ec77406f2f21a618c053e5793b597db7a
* | | Merge pull request #5862 from jamessan/vim-aa3b15dJames McCoy2017-01-09
|\ \ \ | | | | | | | | vim-patch:aa3b15d,82af871,7.4.1925,c95a302
| * | | vim-patch:c95a302James McCoy2017-01-09
| | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/c95a302a4c42ec8230473cd4a5e0064d0a143aa8
| * | | vim-patch:7.4.1925James McCoy2017-01-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Viminfo does not merge file marks properly. Solution: Use a timestamp. Add the :clearjumps command. https://github.com/vim/vim/commit/2d35899721da0e9359a9fe1059554f8c4ea7f0c1
| * | | vim-patch:82af871James McCoy2017-01-09
| | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/82af8710bf8d1caeeceafb1370a052cb7d92f076
| * | | vim-patch:aa3b15dJames McCoy2017-01-09
|/ / / | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/aa3b15dbebf333282503d6031e2f9ba6ee4398ed
* | | Merge pull request #5860 from jamessan/vim-7.4.1889James McCoy2017-01-09
|\ \ \ | | | | | | | | vim-patch:7.4.1889
| * | | vim-patch:7.4.1889James McCoy2017-01-02
| |/ / | | | | | | | | | | | | | | | | | | Problem: When umask is set to 0177 Vim can't create temp files. (Lcd) Solution: Also correct umask when using mkdtemp(). https://github.com/vim/vim/commit/35d88f4e2ff5dcd9904f04612d5febede996137c
* | | process_wait(): Avoid dereference after LOOP_PROCESS_EVENTS. (#5917)Justin M. Keyes2017-01-09
| | |
* | | Merge #5519 from blueyed/improve-python-health-checkJustin M. Keyes2017-01-09
|\ \ \ | | | | | | | | Improve Python health check
| * | | healthcheck: s:download: do not report curl errors (twice)Daniel Hahler2017-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since `curl -s` is used, the actual error (e.g. "curl: (6) Could not resolve host: pypi.python.org") gets not reported anyway, and s:download returns an error message with `s:shell_error` already. This changes `s:download` to also include `a:url` in the error message now. It removes > ERROR: Command error (7) curl -sL https://pypi.python.org/pypi/neovim/json: And changes > ERROR: HTTP request failed: error: curl error with https://pypi.python.org/pypi/neovim/json: 6
| * | | healthcheck: s:check_python: only report latest with good responseDaniel Hahler2017-01-08
| | | |
| * | | healthcheck: python: prefer neovim.VERSIONDaniel Hahler2017-01-08
| | | |
| * | | healthcheck: python: include nvim path for unknown/outdated versionDaniel Hahler2017-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This helps to identify where the `neovim` module is coming from, e.g. - INFO: python3-neovim version: 0.1.10 (outdated; from ~/Vcs/neovim-python-client/neovim) - WARNING: Latest python3-neovim is NOT installed: 0.1.12
| * | | Improve error reporting for Python health checkDaniel Hahler2017-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - s:version_info: return errors from the `import` - skip report_ok for latest version, in case the current version was not found. status contains the error that has been reported in that case already.
| * | | s:check_python: handle 'pip install --user -e'Daniel Hahler2017-01-08
| | | |
* | | | health: completion for :CheckHealthMarco Hinz2017-01-08
|/ / /
* | | health: Windows: Handle backslash path separators. (#5914)Justin M. Keyes2017-01-08
| | | | | | | | | | | | | | | | | | | | | * health: Handle backslash path separators. * health: brevity, consistency * health: Do not set bufhidden=wipe. In the context of CheckHealth, bufhidden=wipe loses data for no reason.
* | | Merge #5909 from justinmk/win32-xdgJustin M. Keyes2017-01-08
|\ \ \ | | | | | | | | Windows: fix XDG fallback paths
| * | | test: Windows: Re-enable shada functional testsRui Abreu Ferreira2017-01-08
| | | | | | | | | | | | | | | | | | | | One of the tests remains disabled until we figure out the expected behaviour of mkdir() on Windows when `prot` is passed.
| * | | XDG: Windows: resolve $LOCALAPPDATA, $TEMP (#5278)Rui Abreu Ferreira2017-01-08
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | After #4964 environment variables in the XDG "fallback" table are no longer expanded. Fallback to correctly expanded $LOCALAPPDATA, $TEMP. If that fails (unlikely), fallback to hard-coded paths (e.g. ~/AppData/Local). Closes #5255
* | | ruby: old versions don't know `gem list -e`Marco Hinz2017-01-08
| | |
* | | Merge #5079 from ZyX-I/shada-save-current-posJustin M. Keyes2017-01-07
|\ \ \ | | | | | | | | shada: Save current cursor position before saving jumps
| * | | clint: Add more exceptions to “space after a cast” ruleZyX2017-01-06
| | | | | | | | | | | | | | | | Python does not allow branching here, complaining that look-behind is not fixed-width.
| * | | shada: Mark ALWAYS_INLINE function as `inline`ZyX2017-01-06
| | | |
| * | | shada: Move setpcmark()/cleanup_jumplist() out of the cycleZyX2017-01-04
| | | |
| * | | shada: Fix offset savingZyX2017-01-04
| | | |