aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* doc: Add one-liner job control exampleJustin M. Keyes2015-03-24
|
* doc: Cleanup for #2099Michael Reed2015-03-24
| | | | | Said PR changed the behavior of `undercurl` in the TUI, but didn't document it.
* Remove `--nofork` / `-f` remnantsMichael Reed2015-03-24
|
* Merge #2148 'misc1.c: split some environment functions'Justin M. Keyes2015-03-24
|\
| * refactor: add tests for env_expand_escbobtwinkles2015-03-24
| |
| * refactor: split some more functions from misc1.cbobtwinkles2015-03-24
|/ | | | | | | | | | | This commit pulls the some environment-variable handling functions out of misc1.c and in to os/env.c. Previously submited as #1231, this is the start of a patch series that does that work based on a more up-to-date master branch. Major tasks accomplished: - move functions and fix includes - fix clint/clang analysis warnings - correct documentation comments
* Merge PR #2146 'Improve functional test debuggability and efficiency'Thiago de Arruda2015-03-24
|\
| * test: Improve functional test debuggability and efficiencyThiago de Arruda2015-03-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Read TEST_TAG/TEST_FILTER env vars from cmake/RunTests.cmake. Setting these environment variables will pass --tags/--filter to busted, which can used to filter which tests are executed. - Remove calls to nvim msgpack-rpc API outside tests. This removes the requirement of having a static `clear` call in test/functional/helpers.lua - Use the new busted command-line option "--lazy" to ensure the setup/teardown hooks are only executed when a suite runs at least one test. Now its possible to run/debug a single test like this: ```sh TEST_FILTER='some test string' make test ``` Which will only run tests containing "some test string" in the title. Another option is: ```sh TEST_TAG=some-tag make test ``` After putting #some-tag into the test title. This also improves debugging experience because there will be no unnecessary gdbserver instances whe GDB=1 is passed.
| * test: Extract code to spawn nvim into the "spawn" helper functionThiago de Arruda2015-03-24
| | | | | | | | | | | | This is can be used for spawning nvim outside a test context. Also refactor screen.lua to use this function when loading the color map(It is better because the GDB/VALGRIND environment variables are ignored)
| * deps: Update busted and dependenciesThiago de Arruda2015-03-24
|/
* Merge PR #2237 'Job fixes to core and tests'Thiago de Arruda2015-03-24
|\
| * test: Fix tty-test programThiago de Arruda2015-03-24
| | | | | | | | | | The "tty ready" string must only be printed when the process is ready to receive signals, and this only happens when the event loop has started.
| * job: Fix memory erroroni-link2015-03-24
| | | | | | | | | | Fix pointer passed to the handles in the uv_close() calls when process_spawn() fails.
| * job: Fix process cleanup using SIGCHLD/waitpidThiago de Arruda2015-03-24
|/ | | | | Add a SIGCHLD handler for cleaning up pty processes passing the WNOHANG flag. It may also be used to cleanup processes spawned with uv_spawn.
* Merge pull request #2213 from jszakmeister/add-junit-supportJohn Szakmeister2015-03-24
|\ | | | | build: add support for running the tests in junit format
| * build: add support for running the tests in junit formatJohn Szakmeister2015-03-21
| | | | | | | | | | | | This requires a couple of extra modules that are not installed by default, and it requires capturing stdout of the tests--otherwise CMake output is intermixed with the XML output of busted.
* | Merge pull request #2214 from Pyrohh/version-cleanupJustin M. Keyes2015-03-23
|\ \ | | | | | | Version output cleanup
| * | version.c: Clean up --version / :version outputMichael Reed2015-03-22
| | | | | | | | | | | | | | | | | | | | | | | | Despite the +/- prefix, the majority of these features have been made non-optional at compile time, so their presence here is misleading. Also mention `:h vim-differences` to make it clear our that many features are non-optional.
| * | Remove *_BUILTIN_TCAPS & 'ttybuiltin'Michael Reed2015-03-22
| | | | | | | | | | | | | | | 'ttybuiltin' was removed and *_BUILTIN_TCAPS was made into dead code in PR #1820.
* | | Merge pull request #2212 from bfredl/concealfixBjörn Linse2015-03-22
|\ \ \ | |/ / |/| | fix missing conceal cchar when it is the first syntax group shown
| * | vim-patch: mark 7.4.673 as includedBjörn Linse2015-03-21
| | |
| * | syntax: let first syntax item seqnr be 1 instead of 0Björn Linse2015-03-21
| |/ | | | | | | | | in screen.c:win_line seqnr 0 represents the state: no current syntax item
* | Merge #2184: Fix coverity issues. (6)Eliseo Martínez2015-03-22
|\ \ | |/ |/| | | Reviewed-by: oni-link <knil.ino@gmail.com>
| * Passing-by: Add function attributes.Eliseo Martínez2015-03-22
| |
| * coverity/105985: Resource leak: RI.Eliseo Martínez2015-03-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Resource leak @ 94, 98, 102. Diagnostic : Real issue. Rationale : Coverity doesn't know that uv_pipe_open will save file descriptor to close them later. So, it signals file descriptors being leaked. This would then seem like a false positive we can fix by teaching coverity about uv_pipe_open through model file. But then we realize that the above is only true if uv_pipe_open succeeds. It it fails, then descriptors are really being leaked, which is why this is considered a real issue and not a false positive after all. Resolution : Add error handling to correctly close descriptors if uv_pipe_open fails at any point. Add model for uv_pipe_open so that Coverity knows it will save descriptors when no error. Helped-by: oni-link <knil.ino@gmail.com>
| * coverity/105982: Unckecked return value: RI.Eliseo Martínez2015-03-22
| | | | | | | | | | | | | | | | | | Problem : Unchecked return value from library @ 91. Diagnostic : Real issue. Rationale : fcntl can fail, which is not being checked. Resolution : Add corresponding error handling. Helped-by: oni-link <knil.ino@gmail.com>
| * coverity/105568: Free of array-typed value: FP.Eliseo Martínez2015-03-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Free of array-typed value @ 3628. Diagnostic : False positive. Rationale : expand_shell_cmd() is called with a mock value for file (*file = (char_u **)""). That means we want file to be filled with a new value. We can't use *file = NULL because that means we don't want file to be filled. Now, coverity incorrectly thinks that sentinel value is the one we are freeing up at some other later point, which is not the case. Resolution : Assert that, when we are freeing *file, its value is different than the sentinel one.
| * coverity/13777: String not null terminated: RI.Eliseo Martínez2015-03-22
|/ | | | | | | | | | | | | | Problem : String not null terminated @ 1543. Diagnostic : Real issue. Rationale : We are reading a struct block0, which contains some string fields, from a file, without checking for string fields to be correctly terminated. That could cause a buffer overrun if file has somehow been garbled. Resolution : Add string fields check for nul termination. Mark issue as intentional (there seems to be no way of teaching coverity about read_eintr being ok that way). Helped-by: oni-link <knil.ino@gmail.com>
* Update to libuv 1.4.2Michael Reed2015-03-21
|
* legacy test script: minor string handling fix #2181Gustaf Lindstedt2015-03-20
| | | | | | | | Add check to see if a string contains ], which can result in cases where wrapping a string in [[...]] breaks. Use [=[...]=] instead on those strings. Use [=[...]=] for insert() and expect().
* vim-patch:7.4.503 #2178Perry Hung2015-03-20
| | | | | | | | | | | | | | | | Problem: Cannot append a list of lines to a file. Solution: Add the append option to writefile(). (Yasuhiro Matsumoto) https://code.google.com/p/vim/source/detail?r=v7-4-503 -Ported old legacy test over to test/functional/legacy/writefile_spec.lua -Tests for mapping and signs from the original patch were removed since they have nothing to do this with feature Tested with: make oldtest, make test on OS X. Signed-off-by: Perry Hung <iperry@gmail.com>
* README: Do not call translator transpilerNikolai Aleksandrovich Pavlov2015-03-20
| | | Never liked this term and never saw it in any dictionaries.
* tui: Add support for true color terminalsThiago de Arruda2015-03-20
| | | | | | | | | This is enabled by setting the `$NVIM_TUI_ENABLE_TRUE_COLOR` environment variable, eg: ``` $ NVIM_TUI_ENABLE_TRUE_COLOR=1 nvim ```
* deps: Use released versions for unibilium, msgpack-c.Florian Walch2015-03-20
|
* Merge pull request #2144 from jszakmeister/fix-warning-in-releaseJohn Szakmeister2015-03-20
|\ | | | | Fix a couple warnings in the release build.
| * Fix unused variable in wstream.c for a release build.John Szakmeister2015-03-14
| |
| * Avoid an unused variable warning in the release build.John Szakmeister2015-03-13
| |
* | Remove BINARY_FILE_IO option #2179Perry Hung2015-03-18
| | | | | | | | | | | | | | | | | | The 'binary' mode flag is ignored on all POSIX conforming systems (man 3 fopen). For all the others, BINARY_FILE_IO needs to be set. Always set BINARY_FILE_IO. Signed-off-by: Perry Hung <iperry@gmail.com>
* | misc2.c: Move emsg* functions to message.c #2152Michael Reed2015-03-19
| | | | | | | | Clean up said functions and some outdated comments while we're at it.
* | Merge #1930 'hlsearch/incsearch screen tests + convert test 63'Justin M. Keyes2015-03-19
|\ \
| * | tests/ui: cleanup ignores in screen_basic_specBjörn Linse2015-03-19
| | |
| * | tests/legacy: convert test 63 (`:match` and `matchadd()` )Björn Linse2015-03-19
| | |
| * | tests/ui: test hlsearch and incsearchBjörn Linse2015-03-19
|/ /
* | tests: update legacy Makefile #2186Justin M. Keyes2015-03-19
| | | | | | | | Many tests were migrated, but entries were left in the old Makefile.
* | Merge PR #2182 'Improve ui/busy handling and early input reading'Thiago de Arruda2015-03-18
|\ \
| * | main: Simplify code that deals with early user inputThiago de Arruda2015-03-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | A read stream will be started before the first ex command is processed. This stream will be used to read early user input before handling control over to the UI module. Which stdio stream will be used depends on which types of file descriptors are connected, and whether the "-" argument was passed.
| * | ui: Refactor so that busy state won't be the defaultThiago de Arruda2015-03-18
|/ / | | | | | | | | | | | | | | | | | | | | Even though assuming nvim is busy most times is simpler, it has a problem: A lot of unnecessary busy_start/busy_stop notifications are sent to the UI. That's because in the majority of scenarios almost no time is spent between `event_poll` calls. This restores the normal behavior which is to call busy_start only when nvim is going to perform some task that can take a significant amount of time. Also improve the usage of buffering in the TUI when changing the cursor state.
* | tui.c: Send fewer cursor-hide commands to the terminal.oni-link2015-03-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since #2158 all connected UIs are informed about the busy state of nvim. This can be used to decide whether to hide or show the cursor (in one place). In the TUI, this is tui_flush(). To prevent cursor flashing, the terminal is always redrawn with an invisible cursor. After that the cursor is shown if necessary. In the current implementation a cursor-hide command will always be the first command in the next redraw, to prevent flashing. This is not necessary. Instead we start the TUI with a hidden cursor and only need to hide the cursor in the next redraw, if the cursor was shown in the last redraw. Otherwise the cursor is still hidden. So instead of sending every redraw the cursor-hide command, we only need to send the command while not busy(a state in nvim with low frequency).
* | Remove platform dependent HAVE_OPENDIR #2010Jonas Dourado2015-03-17
| |
* | contrib: fix YCM completion for header files #2151bobtwinkles2015-03-16
| | | | | | | | [ci skip]