aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | object_to_vim: Recognize buffers, windows, tabpages.Scott Prager2015-04-13
|/ /
* | deps: Add jemalloc as an optional dependencyThiago de Arruda2015-04-13
| | | | | | | | | | | | | | | | | | | | | | | | Jemalloc will be used if the cmake option `USE_JEMALLOC` is enabled(which is the default). To avoid trouble with clang's ASAN, it is disabled by default if the `SANITIZE` option is enabled. Since jemalloc has thread cache for small objects, it fills the gap created by removing klib memory pools. The `xstrdup` funciton(memory.c) had to be reimplemented on top of `xmalloc` to make it work with a custom allocator.
* | memory: Add `free` wrapper and refactor project to use itThiago de Arruda2015-04-13
| | | | | | | | | | | | We already use wrappers for allocation, the new `xfree` function is the equivalent for deallocation and provides a way to fully replace the malloc implementation used by Neovim.
* | memory: Replace klib memory pools by malloc/freeThiago de Arruda2015-04-12
| | | | | | | | | | | | Klib pools were used to improve allocation efficiency for some small objects, but it is not a thread-safe approach. Thread safety in allocations will be required for implementing #2371).
* | eval: Fix memory deallocation of JobEventThiago de Arruda2015-04-12
| | | | | | | | This causes a "read after free" error when kmp_free is replaced by `free`.
* | remove char_u: vim_version_dir #2414Mark Bainter2015-04-12
| | | | | | | | See: #459
* | refactor remove_tail() function to no longer use char_u #2413Mark Bainter2015-04-12
| | | | | | | | See #459
* | Remove obsolete CENT macro #2397Marco Hinz2015-04-12
| | | | | | | | | | | | | | | | | | | | | | | | The CENT macro was set condionally depending on the FEAT_GUI constant that was removed a long time ago. Other small refactorings: - remove obsolete TERM= flags - sort and indent lines in array for readability - 'Conceal' and 'WildMenu' were moved from highlight_init_{dark,light}[] to highlight_init_both[] since the same values were used anyway
* | Refactor default_vim{,runtime}_dir to use char type. #2375Mark Bainter2015-04-12
| | | | | | | | | | | | See: #459 Reviewed-by: Justin M. Keyes <justinkz@gmail.com> Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
* | fix: segfault in ml_recover() if swap file not foundJustin M. Keyes2015-04-12
| | | | | | | | | | - To reproduce: ":recover non_existent_swapfile" - Regression: 00f18e2e09af35494d0b94ae76a9dbc3e0319d89
* | fix: segfault in recover_names(): must not concat full pathJustin M. Keyes2015-04-12
| | | | | | | | | | - regression by 42f1bd9b2228aaca4fb8a5597a3b5774f7ef6876 - closes #1836
* | test: migrate legacy test 78Justin M. Keyes2015-04-12
| |
* | passing-by: mark non-null return; remove invalid OOM check.Justin M. Keyes2015-04-12
| | | | | | | | remove some #ifdef noise
* | Expand :profile's filename #2390Marco Hinz2015-04-11
| | | | | | | | | | | | | | | | | | vim_strsave() is replaced by expand_env_save_opt(), which expands ~ for convenience: :profile start ~/.nvim/prof.log Prior to this change you had to specify an absolute path.
* | Merge pull request #2404 from Pyrohh/coverity-108870Michael Reed2015-04-11
|\ \ | | | | | | [RFC] coverity/108870: Missing break in switch
| * | coverity/108870: Missing break in switchMichael Reed2015-04-10
| | |
* | | coverity/{13738,13739,68853,13717,13720,13716,68854,13718,13721}: FP.Eliseo Martínez2015-04-11
| | | | | | | | | | | | | | | | | | | | | All these issues are false positives that result from coverity's inability to properly follow arithmetic implications in expressions using some macros. Redefining macros another way to make arithmetic implications clearer fixes the issues.
* | | Remove all references to JobActivityMarco Hinz2015-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | The JobActivity event got replaced by callback functions provided to jobstart() or termopen(). It got removed here: https://github.com/neovim/neovim/commit/6e7757ad51dfe3b2de857ff8a8688718ff6115ac
* | | eval: Fix `jobwait()`Thiago de Arruda2015-04-11
| | | | | | | | | | | | | | | | | | - Properly save job event deferring state for recursive calls - Disable breakcheck while running. Breakcheck can invoke job callbacks in unexpected places.
* | | eval: Fix segfault caused by passing invalid callback to jobstartThiago de Arruda2015-04-11
| | |
* | | ops.c: cleanup of `get_yank_register`Björn Linse2015-04-11
| | |
* | | clipboard: fix "" register not updated when clipboard=unnamedBjörn Linse2015-04-11
| | | | | | | | | | | | Helped-By: Scott Prager <splinterofchaos@gmail.com>
* | | clipboard: adjust v:register when clipboard=unnamedBjörn Linse2015-04-11
|/ / | | | | | | | | Helped-By: Nicolas Hillegeer <nicolas@hillegeer.com> Helped-By: Michael Reed <m.reed@mykolab.com>
* | Remove unused 'if_[language].txt': ReviewMichael Reed2015-04-10
| |
* | Remove unused 'if_[language].txt': Clean upMichael Reed2015-04-10
| | | | | | | | | | | | Regarding |script-here|: despite being a language agnostic piece of advice, it was in `if_perl.txt`. Regardless, we now only have one support for one legacy plugin interface, so put it in `if_pyth.txt`
* | Bring E319 (cmd not available) in line with E519 (opt not available)Michael Reed2015-04-10
| |
* | Clarify that nvim has no built-in GUIMichael Reed2015-04-10
| |
* | coverity/{108271,108272,108273}: add fallthrough annotation #2362Alexey Shmalko2015-04-10
| | | | | | | | Signed-off-by: Michael Reed <m.reed@mykolab.com>
* | mark.c: Remove JUMPLIST_ROTATE #2376David Bürgin2015-04-09
| | | | | | | | JUMPLIST_ROTATE is never defined, and the feature is not popular.
* | Add new highlight groups TermCursor/TermCursorNCMarco Hinz2015-04-09
| | | | | | | | | | | | | | These highlight groups replace the old mechanism of setting: - {g,b}:terminal_focused_cursor_highlight - {g,b}:terminal_unfocused_cursor_highlight
* | event: Only process if event_init has been run.Scott Prager2015-04-08
| | | | | | | | | | | | | | | | | | Reported by @fourjay, a codepath that causes event_poll() to run before event_init() will trigger a segfault as the events list will not have been initialized. Exiting immediately from event_init() causes nvim to hang, so just exit before running the events. fixes #2339
* | eval: do not assume a vval.v_string is nonnull.Scott Prager2015-04-08
| | | | | | | | | | | | | | | | | | | | | | In the fallowing functions, use get_tv_string() to safely avoid a NULL pointer dereference. * rpcstart * rpcrequest * rpcnotify * jobstart fixes #2321
* | system(): Return an empty string if no output.Scott Prager2015-04-08
| | | | | | | | fixes #2286
* | messages: Update common instances of Vim to Nvim #2031Michael Reed2015-04-08
| |
* | Merge pull request #1805 from fwalch/vim-7.4.399Florian Walch2015-04-09
|\ \ | | | | | | vim-patch: Port parts of Vim 7.4.399.
| * | vim-patch: Port parts of Vim 7.4.399.Florian Walch2015-04-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolves #1753. https://code.google.com/p/vim/source/detail?r=v7-4-399 Reviewed-by: Felipe Oliveira Carvalho <felipekde@gmail.com> Helped-by: David Bürgin <676c7473@gmail.com> Helped-by: Scott Prager <splinterofchaos@gmail.com> Helped-by: Michael Reed <m.reed@mykolab.com> Helped-by: Felipe Oliveira Carvalho <felipekde@gmail.com>
* | | Merge pull request #2346 from splinterofchaos/fix-terminalScott Prager2015-04-08
|\ \ \ | | | | | | | | [RFC] terminal: Handle loss of focus in event loop.
| * | | term: ensure term->buf is validScott Prager2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fallowing test (reduced), submitted by @mhinz may free term->buf, leaving the pointer dangling. ```vim let s:buf = -1 function! s:exit_handler() execute 'bdelete!' s:buf endfunction vnew let s:buf = bufnr('%') let id = termopen('sleep 1', { 'on_exit': function('s:exit_handler') }) call s:test() ``` When the buffer is known to be closing, set term->buf to NULL, and check buf_valid() in on_refresh(). Helped-by: Marco Hinz (@mhinz)
| * | | term: use window col offset to calculate widthScott Prager2015-04-05
| | | | | | | | | | | | | | | | fixes #2317
| * | | term: after <C-\>, resume normal input loopScott Prager2015-04-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pressing <C-\> and then a mouse click will insert the click into the terminal as if a keyboard button had been pressed. Keep track of whether the last input was <C-\> and only call terminal_send_key() if the next input is a key press.
| * | | terminal: Handle loss of focus in event loop.Scott Prager2015-04-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While in a terminal and insert mode, if an event caused loss of focus, nvim would stay in the terminal event loop causing an inconsistent view of internal state and/or segfault. Remove the "term" argument from terminal_enter() as it only makes sense to call it with curbuf->terminal. Terminate the loop when switched to a different buffer. fixes #2301
* | | | event.c: Simplify timer use in event_poll()oni-link2015-04-08
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | event_poll() always leaves the libuv I/O loop after one iteration. The duration for how long the loop polls for I/O is given by the shortest timeout of all active timers. Is no timer active, I/O is/could be polled indefinitely. To make sure our timer is still active when I/O polling begins, a prepare handle is used to start the timer right before the polling. But by only using a timer that restarts after its timeout was reached, we also can assure that polling is done with (nearly) the same finite timeout. For a short explanation how the I/O loop is working, see http://docs.libuv.org/en/latest/design.html#the-i-o-loop.
* | | Fix warnings: window.c: close_last_window_tabpage(): Np deref: RI.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 1769. Diagnostic : Real issue. Rationale : It seems buffer could be null. Not sure, though. Resolution : Check for buffer null. This resolution was chosen as it will always work. But it could be that buffer can't really be null at that point. autocmd_win is ruled out by close_window, so that can't be the case. I'm not sure if other windows without buffers are possible, so leaving it this way. If it's confirmed buffer can't be null, resolution through an assert would be possible and this would be FP, not RI.
* | | Fix warnings: terminal.c: get_config_int(): Dead init: RI.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dead initialization @ 1119. Diagnostic : Real issue. Rationale : `obj` is immediately assigned another value through GET_CONFIG_VALUE macro. Resolution : Don't initialize. Helped-by: oni-link <knil.ino@gmail.com>
* | | Fix warnings: terminal.c: get_config_string(): Dead init: RI.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dead initialization @ 1109. Diagnostic : Real issue. Rationale : `obj` is immediately assigned another value through GET_CONFIG_VALUE macro. Resolution : Don't initialize. Helped-by: oni-link <knil.ino@gmail.com>
* | | Fix warnings: terminal.c: redraw(): Np dereference: RI.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 1053. Diagnostic : Real issue. Rationale : Branch "Exiting focused terminal" can actually be executed when term is NULL. Resolution : Guard branch with term check.
* | | Fix warnings: eval.c: f_termopen(): Use-after-free: MI.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Use-after-free @ 15081. Diagnostic : Multithreading issue. Rationale : `get_dict_callback` can return NULL on two different cases: 1) when the dict doesn't contain the given key; this case is not considered an error. 2) when the key exists but there's some problem with its value; this is considered an error. Then, code calling `get_dict_callback` in `common_job_callbacks`, as well as code calling `common_job_callbacks`, uses `did_emsg` to distinguish between error/non-error cases. Suggested error path presumes an error condition within `common_job_callbacks`, with `did_emsg` being true, but then being false just after returning to calling code in `f_termopen`. That, clearly, could only happen if another thread run in between those points. Resolution : Refactor `get_dict_callback` and `common_job_callbacks`, so that they clearly distinguish between error/non-error situations, without recurring to globals.
* | | Fix warnings: eval.c: f_jobstart(): Np dereference: FP.Eliseo Martínez2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 10812. Diagnostic : False positive. Rationale : `args->lv_first` can't be NULL, as we have just stated above that that there's at least one item. Resolution : Assert.
* | | legacy tests: migrate test38 #2367Murali Suresh2015-04-07
| | |
* | | tests: Migrate legacy tests 44 and 99 #2366David Bürgin2015-04-07
| | | | | | | | | | | | Consolidates the old test44 and test99 tests into one busted spec.
generated by cgit (git 2.34.1) at 2025-07-21 09:10:32 +0000