aboutsummaryrefslogtreecommitdiff
path: root/src
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
* | | | | | | 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
* | | | | | 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
* | | | | | 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: 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
* | | | | 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.
* | | | | 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
* | | | | 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
* | | | | 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
| | | | |
* | | | | 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
* | | | | 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
| | | | | |
| * | | | | shada: Silence linterZyX2017-01-04
| | | | | |
| * | | | | shada: Save current cursor position before saving jumpsZyX2017-01-04
| | | | | |
* | | | | | unittest,memory: Fix testsZyX2017-01-07
| | | | | |
* | | | | | memory: Document new additions to memory.hZyX2017-01-07
| | | | | |
* | | | | | memory: Restore entered_free_all_mem functionalityZyX2017-01-07
| | | | | |
* | | | | | eval: Fix failing testZyX2017-01-07
| | | | | |
* | | | | | unittest: Add dict_items functionZyX2017-01-07
| | | | | |
* | | | | | eval: Make sure that copyID is reset when neededZyX2017-01-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Works by making value pushed on stack represent the exhausted list. Fixes #5901, except for dictionaries which need similar adjustment.
* | | | | | eval/typval_encode.h: Use only a single macros with _INNER[_2] hackZyX2017-01-07
| | | | | |
* | | | | | unittest: Allow mocking allocator callsZyX2017-01-07
| |_|/ / / |/| | | |
* | | | | Merge pull request #5826 from ZyX-I/fix-typval_encodeJames McCoy2017-01-06
|\ \ \ \ \ | | | | | | | | | | | | Refactor eval/typval_encode.h
| * | | | | eval: Fix typoZyX2017-01-07
| | | | | |
| * | | | | eval/typval_encode: Dump empty dictionary before checking for refcycleZyX2017-01-06
| | | | | | | | | | | | | | | | | | Otherwise copyID will stay forever on empty dictionaries.
| * | | | | eval: Work with reference cycles in partials (self) properlyZyX2017-01-06
| | | | | |
| * | | | | eval/typval_encode: Use TYPVAL_ENCODE_CONV_EMPTY_DICT for partialsZyX2017-01-06
| | | | | |
| * | | | | eval/typval_encode.h: Remove documentation that is located in *.c.hZyX2017-01-06
| | | | | |
| * | | | | *: Remove `// fname()` comments near typval_encode includesZyX2017-01-04
| | | | | |
| * | | | | eval/encode: Always check the return value of encode_vim_to_\*ZyX2017-01-04
| | | | | |
| * | | | | tests: Add tests for partials dumpingZyX2017-01-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also fixed dumping of partials by encode_vim_to_object and added code which is able to work with partials and dictionaries to test/unit/eval/helpers.lua (mostly copied from #5119, except for partials handling).
| * | | | | eval: Fix errorneous early exit when converting lists and dictionariesZyX2017-01-03
| | | | | |
| * | | | | eval/encode: Fail when stringifying NULL functionsZyX2017-01-03
| | | | | |
| * | | | | eval/typval_encode: Handle NULL partials properlyZyX2017-01-03
| | | | | |
| * | | | | eval/typval_encode: Provide proper values as dict argumentZyX2017-01-03
| | | | | |
| * | | | | eval: Fix unused variable error in release buildsZyX2017-01-03
| | | | | |
| * | | | | eval/encode: Fix 4 new linter failuresZyX2017-01-03
| | | | | |