aboutsummaryrefslogtreecommitdiff
path: root/src/nvim
Commit message (Collapse)AuthorAge
...
| * | | process.c: Prevent data loss for process output streamsoni-link2016-05-15
| | | | | | | | | | | | | | | | | | | | For a terminating process, it's output streams could be closed, before all data is read.
* | | | version: State that 1960 was includedZyX2016-06-26
| | | |
* | | | Merge #4965 from justinmk/fixup4453Justin M. Keyes2016-06-25
|\ \ \ \ | | | | | | | | | | ex_cmds2.c: lint
| * | | | ex_cmds2.c: cleanupJustin M. Keyes2016-06-25
| | | | |
| * | | | ex_cmds2.c:style: Silence all clint warningsJ Phani Mahesh2016-06-24
| | | | |
* | | | | option: Make all pointers in set_string_option constantZyX2016-06-24
| | | | |
* | | | | *: Fix linter errorsZyX2016-06-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also adds one exception to linter rules: typedef struct { kvec_t(Object) stack; } EncodedData; is completely valid (from the style guide point of view) code.
* | | | | kvec: Do not bother with making capacity a power of 2ZyX2016-06-24
| | | | | | | | | | | | | | | This avoids gcc warnings about undefined behaviour.
* | | | | msgpack_rpc: Fix crash in log_server_msgZyX2016-06-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It appears that used msgpack library is not able to parse back message created by msgpack_rpc_from_object() if nesting level is too high, so log_server_msg now cares about msgpack_unpack_next() return value. Also error message from server_notifications_spec.lua is not readable if something is wrong (though at least now it does not crash when parsing deeply nested structures). log_server_msg() in the test reports [msgpack-rpc] nvim -> client(1) [error] "parse error"
* | | | | option: Handle NULL string in set_option_valueZyX2016-06-24
| | | | |
* | | | | eval: Also make clear_tv non-recursiveZyX2016-06-24
| | | | |
* | | | | kvec,typval_encode: Add new vector: the one with preallocated arrayZyX2016-06-24
| | | | |
* | | | | msgpack_rpc: Also make msgpack_from_*/msgpack_to_* functions not recurZyX2016-06-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes some stack overflows in new test regarding deeply nested variables. Now in place of crashing vim_to_object/msgpack_rpc_from_object/etc it crashes clear_tv with stack overflow.
* | | | | api/helpers: Use typval_encode.h for vim_to_objectZyX2016-06-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ought to prevent stack overflow, but I do not see this actually working: *lua* code crashes with stack overflow when trying to deserialize msgpack from Neovim, Neovim is fine even if nesting level is increased 100x (though test becomes very slow); not sure how recursive function may survive this. So it looks like there are currently only two positive effects: 1. NULL lists are returned as empty (#4596). 2. Functional tests are slightly more fast. Very slightly. Checked for Release build for test/functional/eval tests because benchmarking of debug mode is not very useful.
* | | | | os/fileio: Use readv oftenZyX2016-06-24
| | | | |
* | | | | *: Satisfy linter (newest type casts rule)ZyX2016-06-24
| | | | |
* | | | | file: Move src/nvim/file.* to src/nvim/os/fileio.*ZyX2016-06-24
| | | | |
* | | | | unittests: Add tests for file.cZyX2016-06-24
| | | | | | | | | | | | | | | | | | | | Also fixes some errors found.
* | | | | shada: Do not forget to close ShaDa readerZyX2016-06-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously there was file descriptor leak, not detected by sanitizers. Now it is file descriptor leak with a small memory leak which is detected by ASAN what fails one of the tests (actually, “ShaDa support code leaves .tmp.z in-place when there is error in original ShaDa and it has .tmp.a … .tmp.x”, but error is reported at the next test because leaks are not detected until Neovim exit and Neovim exit happens when clear()/reset() is called which happens in before_each only).
* | | | | file: Use own constants, do not rely on fcntl.hZyX2016-06-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One of the reasons is that O_RDONLY is zero, which makes checking whether file is opened read- or write-only harder. It is not guaranteed that on other system O_WRONLY will not be zero (e.g. because file can only be opened in read-write mode).
* | | | | file: Add buffered reading and writingZyX2016-06-23
| | | | | | | | | | | | | | | | | | | | Still no busted tests. Not tested without HAVE_PREADV.
* | | | | file,os/fs,shada: Separate opening, closing, writing and reading filesZyX2016-06-23
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moves low-level functions handling to os/fs.c. Adds file.c with a proxy interface. Target: while leaving syscalls handling is os.c (partially handled by libuv), add buffering for reading and writing to file.c.
* | | | cmake: remove unused includes (#4947)Nicolai Skogheim2016-06-22
| | | |
* | | | lintJames McCoy2016-06-22
| | | |
* | | | vim-patch:7.4.1592James McCoy2016-06-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Quickfix code using memory after being freed. (Dominique Pelle) Solution: Detect that the window was closed. (Hirohito Higashi) https://github.com/vim/vim/commit/0899d698030ec076eb26352cda1ea334ab0819d9
* | | | Always resize the embedded vterm properlyMarco Hinz2016-06-20
| | | |
* | | | Merge #4697 'capture() function'.Justin M. Keyes2016-06-20
|\ \ \ \
| * | | | Port capture() functionShougo Matsushita2016-05-27
| | | | | | | | | | | | | | | | | | | | https://groups.google.com/forum/#!msg/vim_dev/H3Z3ChSUh_4/beZs6KzYdBsJ
* | | | | coverity/149459: CHECKED_RETURN (false positive)Justin M. Keyes2016-06-18
| | | | |
* | | | | synIDattr(): return RRGGBB value for [fg|bg|sp]# #4851Rom Grk2016-06-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add tests for synIDattr() with [fg|bg|sp]# add tests for synIDattr and various #RGB colors synIDattr: test for ui_rgb_attached() test: fix tests for synIDattr fg/bg/sp
* | | | | eval: add api_info()Björn Linse2016-06-17
| | | | | | | | | | | | | | | | | | | | | | | | | Previously, the api metadata was only accessible frow within nvim as msgpackparse(systemlist('nvim --api-info'))[0]
* | | | | Merge #4923 from justinmk/docJustin M. Keyes2016-06-16
|\ \ \ \ \ | | | | | | | | | | | | doc: cleanup
| * | | | | doc: consolidate nvim.txtJustin M. Keyes2016-06-15
| | | | | |
* | | | | | vim-patch:7.4.1126James McCoy2016-06-15
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Can only get the directory of the current window. Solution: Add window and tab arguments to getcwd() and haslocaldir(). (Thinca, Hirohito Higashi) https://github.com/vim/vim/commit/c970330676eaae7ba7cd05cfa46df5a413853ef9
* | | | | Add new functionality to the `=` marker in the STLTJ DeVries2016-06-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new functionality is explained in the documentation. Also, many tests have been added to the buffer_spec.lua file
* | | | | Merge #3745 from cacplate/ops_WconversionJustin M. Keyes2016-06-13
|\ \ \ \ \ | | | | | | | | | | | | Enable -Wconversion in ops.c
| * | | | | remove some asserts and lintCharles Joachim2016-05-30
| | | | | |
| * | | | | ops.c: enable -Wconversion warningCharles Joachim2016-05-30
| | | | | |
* | | | | | vim-patch:7.4.1352 (#4914)Michael Ennen2016-06-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The test script lists all functions before executing them. Solution: Only list the function currently being executed. https://github.com/vim/vim/commit/93bf558caef2d507ef6baf56eaf6025b63da1e34
* | | | | | tui/input.c: Handle Ctrl-Z (suspend) for TERM=linux (#3100) (#4911)Mateusz Czapliński2016-06-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix #3100. On virtual consoles (Alt-F1, etc.), the Ctrl-Z combination was lost.
* | | | | | Merge #4908 from ZyX-I/clint-checks-2Justin M. Keyes2016-06-11
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add more clint checks
| * | | | | | func_attr: Fix other new linter errorsZyX2016-06-11
| | | | | | |
| * | | | | | *: Fix new linter errorsZyX2016-06-11
| | | | | | |
| * | | | | | *: Also fix the adjacent errorsZyX2016-06-11
| | | | | | |
| * | | | | | *: Fix errors from new linter checksZyX2016-06-11
| | | | | | |
* | | | | | | Remove some unnecessary function attributes (#4909)oni-link2016-06-11
|/ / / / / / | | | | | | | | | | | | | | | | | | This removes attribute FUNC_ATTR_NONNULL_ALL for functions without a pointer parameter.
* | | | | | Merge #4813 'runtime: clipboard: start daemons in /'.Justin M. Keyes2016-06-10
|\ \ \ \ \ \
| * | | | | | eval: allow setting cwd in {jobstart,termopen}()Aleksa Sarai2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Processes in vim are always started in the current directory, which causes issues when the process is a daemon and the current directory is a mountpoint. Fix this by adding an option to set the cwd of the new process with jobstart(). In addition, fix termopen() so that it actually uses the cwd option from the dict (it couldn't previously set the cwd value due to dead code). Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
* | | | | | | Merge pull request #4875 from shotat/vim-7.4.1130James McCoy2016-06-09
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:7.4.1130
| * | | | | | | vim-patch:7.4.1130shotat2016-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Memory leak in :vimgrep. Solution: Call FreeWild(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/61ff4dd6a4d47bd32383fe28087be2b37dec53f4 mark 1130 as NA