aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * 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
* | Merge pull request #2447 from equalsraf/tb-luarocks-refactorFlorian Walch2015-04-23
|\ \ | | | | | | Refactor luarocks third-party recipe
| * | Update busted version to 2.0.rc8-0Rui Abreu Ferreira2015-04-22
| | |
| * | Refactor luarocks third-party recipeRui Abreu Ferreira2015-04-22
| | | | | | | | | | | | | | | - Replace repeated use of the luarocks command and build arguments with variables ${LUAROCKS_BINARY} and ${LUAROCKS_BUILDARGS}
| * | Add option USE_BUNDLED_BUSTED to third-partyRui Abreu Ferreira2015-04-22
| | | | | | | | | | | | | | | | | | - When not running the unit tests, building the third-party busted, nvim-client and other dependencies can be skipped by passing -DUSE_BUNDLED_BUSTED=FALSE to the third-party project.
* | | Merge #2473: Fix clang analysis warnings. (10)Eliseo Martínez2015-04-23
|\ \ \ | |_|/ |/| | | | | | | | Reviewed-by: oni-link <knil.ino@gmail.com> Reviewed-by: Scott Prager <splinterofchaos@gmail.com>
| * | 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.
* | Merge PR #2208 'if_python3 support'Thiago de Arruda2015-04-23
|\ \
| * | Improve travis ci scriptShougo Matsushita2015-04-21
| | | | | | | | | | | | Reviewed-by: Justin M. Keyes <justinkz@gmail.com>, Nikolay Pavlov <zyx.vim@gmail.com>
| * | 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 pull request #2479 from Pyrohh/contributing-mdMichael Reed2015-04-22
|\ \ | |/ |/| [RFC] CONTRIBUTING: Mention the wiki's troubleshooting & FAQ articles
| * CONTRIBUTING: Mention the wiki's troubleshooting & FAQ articlesMichael Reed2015-04-21
|/ | | | [ci skip]
* Change neovim.org references to neovim.ioThiago de Arruda2015-04-19
|
* 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>
* | Merge pull request #2411 from bfredl/yankregsBjörn Linse2015-04-17
|\ \ | | | | | | Cleanup implementation of registers and fix missing clipboard support in `:redir`
| * | 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>
* | Merge pull request #2406 from Pyrohh/make-use-wildcardMichael Reed2015-04-17
|\ \ | |/ |/| [RFC] build: Misc. small improvements
| * Revert "third-party: temporarily drop back to an old version of the busted spec"Michael Reed2015-04-16
| | | | | | | | | | | | | | No longer needed after https://github.com/Olivine-Labs/busted/commit/aab5d19d0c615a33b865ebc771fdef80edb99356 This reverts commit d22f2f94047a5ed1e930ee46d0d2e8d6aacfa104.
| * contrib/local.mk.example: Misc. improvementsMichael Reed2015-04-16
| | | | | | | | | | - Mention how to disable jemalloc during compilation - Mention using additional CFLAGS during compilation
| * CMakeLists.txt: Remove unneeded globsMichael Reed2015-04-16
| | | | | | | | refs #2426
| * runtime/doc/Makefile: Use `wildcard`Michael Reed2015-04-16
|/
* 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
* Travis: Remove unused dependencies #2431grtlr2015-04-16
|
* 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>
* | Remove runtime/tools/ #2426Marco Hinz2015-04-14
| | | | | | | | | | | | This directory contains old (at least 10+ years) scripts that mainly filter output of other old programs to make them conformant with an 'errorformat' from 10+ years ago.
* | Merge pull request #2331 from splinterofchaos/serverlistenScott Prager2015-04-14
|\ \ | | | | | | vimL: serverlisten({addr}), list(), and stop({addr})
| * | server: add unit testsScott Prager2015-04-14
| | |
| * | 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
* | | third-party: temporarily drop back to an old version of the busted specJohn Szakmeister2015-04-14
| | | | | | | | | | | | | | | The new one contains a clone url that does not work correctly. Once that is fixed, we can go back to tracking master.
* | | third-party: remove stray argumentJohn Szakmeister2015-04-14
| | | | | | | | | | | | This was missed when dropping our custom output handler.
* | | runtime: Handle failures while loading individual remote pluginsThiago de Arruda2015-04-13
| | | | | | | | | | | | | | | The s:RegistrationCommands functions will now test the type of the "specs" rpc call. If it's not a list, then the plugin failed to load and is skipped
* | | travis: Don't use jemalloc for the gcc buildThiago de Arruda2015-04-13
| | | | | | | | | | | | It seems valgrind hangs if jemalloc is used.