aboutsummaryrefslogtreecommitdiff
path: root/src/nvim
Commit message (Collapse)AuthorAge
...
* Replace vim_isspace() with ascii_isspace() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace vim_isxdigit() with to ascii_isxdigit() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace VIM_ISDIGIT() and vim_isdigit() with ascii_isdigit() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace vim_iswhite with ascii_iswhite() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* rpcstart: check argvars, /then/ alloc argv #2462Scott Prager2015-04-23
| | | | fixes #2458
* Merge #2491: Fix xfree of static value from vim_getenv() refactor.Eliseo Martínez2015-04-23
|\ | | | | | | Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
| * Fix xfree of static value in expand_shellcmd()Mark Bainter2015-04-23
| | | | | | | | | | | | | | | | The refactoring of vim_getenv() to remove the mustfree arg included reworking calling functions. expand_shellcmd was also using that to track its usage of the variable within the function, resulting in #2487. This change addresses that scenario and cleans up some of the function for style.
* | TAB0 might not be defined in termios.h #2483Rui Abreu Ferreira2015-04-23
| | | | | | | | | | - OpenBSD termios.h does not have TAB0, skip its use if it is not defined
* | Fix warnings: ex_getln.c: init_history(): NP dereference: FP.Eliseo Martínez2015-04-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 4199. Diagnostic : False positive. Rationale : Suggested error path implies that newlen == 0 and newlen > hislen, which implies hislen < 0, which is impossible (hislen is always between 0 and 10000). Resolution : Refactor code. Asserting oldlen >= 0 should have sufficed, but for some reason analyzer doesn't follow implications in that case. Asserting newlen != 0 || newlen <= hislen works. But the chance has been taken to do extensive refactoring of this function, as it was difficult to understand as it was. As a result of refactoring, assert is not needed anymore, as we don't call clear_hist_entry() in refactored version. Refactor : - Rework algorithm: * Drop guard for OOM case, which can't happen now. * Drop empty/growing/shrinking cases. Simplify to always doing the same. * Perform circular array reordering in all cases (before, it only did when shrinking). * Work in batches through memcpy/memset, instead of one entry at a time, as it did before. - Inline variable declarations. - Replace `ssize_t` by `int`. - Introduce oldlen as entry value of hislen. - Add a lot of comments. Helped-by: Scott Prager <splinterofchaos@gmail.com> Helped-by: oni-link <knil.ino@gmail.com>
* | Fix warnings: ops.c: op_yank_reg(): Dead assignment: HI.Eliseo Martínez2015-04-23
|/ | | | | | | Problem : Dead assignment @ 2480. Diagnostic : Harmless issue. Rationale : It's true `reg` is not used any more after that point. Resolution : Delete line.
* Add if_python3 supportShougo Matsushita2015-04-21
| | | | | Reviewed-by: Michael Reed <m.reed@mykolab.com>, Daniel Hahler <github@thequod.de> Helped-by: Daniel Hahler <github@thequod.de>
* Merge #2418: Remove char_u (4)Eliseo Martínez2015-04-19
|\ | | | | | | | | | | Reviewed-by: Scott Prager <splinterofchaos@gmail.com> Reviewed-by: Michael Reed <m.reed@mykolab.com> Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
| * Refactor after_pathsep() to use const qualified argsMark Bainter2015-04-18
| |
| * Refactor get_env() to respect const qualifierMark Bainter2015-04-18
| | | | | | | | | | | | Without the casts*, the compiler rightly warns about the os_getenv losing the qualifier. This refactor adds a variable to manage this properly, and renames the original variables to increase clarity.
| * Remove char_u: get_mess_lang()Mark Bainter2015-04-18
| |
| * Remove char_u: set_helplang_default()Mark Bainter2015-04-18
| |
| * Remove char_u: vim_setenv()Mark Bainter2015-04-18
| |
| * Remove char_u: vim_getenv()Mark Bainter2015-04-17
| |
| * Remove char_u: after_pathstep()Mark Bainter2015-04-17
| | | | | | | | See: #459
* | Merge pull request #2443 from jszakmeister/fix-more-release-warningsJohn Szakmeister2015-04-18
|\ \ | | | | | | Fix a few uninitialized variable warnings.
| * | Fix an uninitialized variable warning for call_start.Eliseo Martínez2015-04-18
| | |
| * | Fix a couple uninitialized variable warnings in the release build.John Szakmeister2015-04-18
| |/ | | | | | | | | | | Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com> Reviewed-by: Marco Hinz <mh.codebro@gmail.com> Reviewed-by: Björn Linse <bjorn.linse@gmail.com>
* | clipboard: cleanup `valid_yank_reg` and use it for `:redir`Björn Linse2015-04-17
| | | | | | | | | | | | | | This fixes missing clipboard support for `:redir` Helped-By: Michael Reed <m.reed@mykolab.com> Helped-By: Scott Prager <splinterofchaos@gmail.com>
* | clipboard: simplify handling of of put in visual mode.Björn Linse2015-04-17
| | | | | | | | | | When clipboard=unnamed and put over visual selection, reduces number of provider calls from 6 to 2. Also add test.
* | ops.c: eliminate unneccesary static variable `y_append`Björn Linse2015-04-17
| |
* | ops.c: eliminate static variable `y_current`Björn Linse2015-04-17
|/ | | | | | | This variable isn't stateful, and should be passed around instead. Helped-By: Scott Prager <splinterofchaos@gmail.com> Helped-By: Michael Reed <m.reed@mykolab.com>
* Eval: do not join a list if it's empty anyway #2441Marco Hinz2015-04-16
| | | | | | | | | | This spares some work and also prevents list_join() from calling ga_init() with a growsize of 0 which would lead to the nvimlog being littered with: [warning @ ga_set_growsize:64] 17675 - trying to set an invalid ga_growsize: 0 Also in Vim 7.4.702 https://github.com/vim/vim/commit/5216f767d4070d0085de6fa1391e6f2991c1baa5
* Merge pull request #2353 from jszakmeister/fix-unused-var-warnings-in-releaseJohn Szakmeister2015-04-16
|\ | | | | Fix several unused variable warnings in the release build.
| * Fix several unused variable warnings in the release build.John Szakmeister2015-04-16
| | | | | | | | | | Reviewed-by: Justin M. Keyes <justinkz@gmail.com> Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
* | Merge pull request #2331 from splinterofchaos/serverlistenScott Prager2015-04-14
|\ \ | | | | | | vimL: serverlisten({addr}), list(), and stop({addr})
| * | eval: learn serverstart, list, and close().Scott Prager2015-04-14
| | | | | | | | | | | | | | | | | | Implement functions for spawning, destroying, and listing active servers, and add server_address_list() to msgpack_rpc/server.c for the serverlist() vimL function.
| * | msgpack_rpc/server.c: Use a garray over a kmap.Scott Prager2015-04-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Testing of server_start() and server_stop() found that after adding a server at address addr, pmap_has(addr) would always return true, but pmap_get(addr) would always return NULL. Since a client is only expected to have a small number of servers, an array may be more efficient than a hash map, anyway. Discussion: https://github.com/neovim/neovim/pull/1302#issuecomment-88487148
| * | Implement os_unsetenv()Rui Abreu Ferreira2015-04-14
| | | | | | | | | | | | | | | | | | | | | - In UNIX systems where unsetenv() is available, it is used. Otherwise the variables are set with the empty string. - New check HAVE_UNSETENV for unsetenv() - Added unit test to env_spec.lua
* | | channel: recognized nvim-style errorsScott Prager2015-04-13
| | |
* | | msgpack: Allow notifications to execute commands.Scott Prager2015-04-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Consider: `let vim = rpcstart('nvim', ['--embed'])` Allows `rpcnotify(vim, ...)` to work like an asynchronous `rpcrequest(nvim, ...)`. Helped-by: Michael Reed <m.reed@mykolab.com> Helped-by: Justin M. Keyes <>
* | | 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
generated by cgit (git 2.34.1) at 2026-02-28 09:49:39 +0000