aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* 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
| * | 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.
generated by cgit (git 2.34.1) at 2026-02-26 19:14:52 +0000