aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* CONTRIBUTING.md: more guidance on cosmetic changes #1346Kartik Agaram2014-10-30
|
* fs.c: add FUNC_ATTR_NONNULL_ALL to all functionsJustin M. Keyes2014-10-30
|
* Merge PR #1357 'Fixes to input/job modules'Thiago de Arruda2014-10-29
|\
| * travis: Make sure valgrind report is shown on test failuresThiago de Arruda2014-10-29
| |
| * api: Add vim_input function and mark vim_feedkeys as deferredThiago de Arruda2014-10-29
| | | | | | | | | | | | | | | | The `vim_feedkeys` must be deferred because it can potentially free the buffer passed to `os_inchar`(which in turns calls `vim_feedkeys` indirectly). The new `vim_input` function can be used to emulate user input(Since it does not mess with the typeahead, it is safe to execute without deferring).
| * input: Fix sizes of input/read buffersThiago de Arruda2014-10-29
| | | | | | | | | | Input buffer must be bigger than read buffer to ensure it always has space for converted data.
| * input/job: process ctrl+c and do conversion in the read callbackThiago de Arruda2014-10-29
|/ | | | | | | | | - Extract `process_interrupts` out of `convert_input` - Instead of waiting for os_breakcheck/os_inchar calls, call `convert_input` and `process_interrupts` directly from the read callback in input.c. - Remove the `settmode` calls from `job_wait`. Now that interrupts are processed in the event loop, there's no need to set the terminal to cooked which introduces other problems(ref 7.4.427)
* fix #1027: :wundo segfault in new, non-empty bufferJustin M. Keyes2014-10-26
|
* Merge pull request #1347 from splinterofchaos/433Justin M. Keyes2014-10-29
|\ | | | | vim-patch:7.4.433
| * version: Mark patch 430 NA.Scott Prager2014-10-25
| | | | | | | | Not relevant because we always compile with +conceal.
| * vim-patch:7.4.433Scott Prager2014-10-25
| | | | | | | | | | | | | | Problem: Test 75 fails on MS-Windows. Solution: Use ":normal" instead of feedkeys(). (Michael Soyka) https://code.google.com/p/vim/source/detail?r=v7-4-433
* | Merge #1342 "signs bugfix"Justin M. Keyes2014-10-28
|\ \
| * | fix 'sign unplace id'Kartik K. Agaram2014-10-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since the introduction of the FOR_ALL_BUFFERS macro, 'sign unplace id' without a buffer was only removing the sign from the first buffer rather than all buffers, as described in the documentation. :help sign-unplace -- modeline discussion: https://github.com/akkartik/neovim/commit/7863c247db#commitcomment-8342590
| * | api: Implement vim_command_output functionThiago de Arruda2014-10-28
|/ / | | | | | | | | This function can be used by API clients to execute a command and capture the output.
* | Merge PR #1350 'valgrind/jobs: Fix invalid reads.'Thiago de Arruda2014-10-28
|\ \
| * | test/job: Implement some basic jobs tests.Scott Prager2014-10-28
| | |
| * | test/shell: Add failure tests for `system()`.Scott Prager2014-10-28
| | |
| * | job_start: Do not close in/out/err on error.Scott Prager2014-10-28
| | | | | | | | | | | | | | | | | | The streams job_close_*() reference have not been initialized by the time we call uv_spawn() and libuv closes these pipes for us when spawn() fails.
| * | job/valgrind: Fix invalid reads/missing free.Scott Prager2014-10-28
|/ / | | | | | | | | | | | | | | The JobEvent structure may refer to a job after it has been freed. Apply @tarruda's patch to extract the job data before pushing the event. Also, fix the type, "data" -> "job", in on_job_exit() and free the job name in the last job event.
* | Merge PR #1354 'Fixes to channel.c, input.c and helpers.lua'Thiago de Arruda2014-10-28
|\ \ | |/ |/|
| * input: Fix conversion error in `convert_input()`Thiago de Arruda2014-10-28
| | | | | | | | | | | | | | The `rbuffer_consumed` was being passed a consumed count from another buffer, causing integer overflow in `rbuffer_relocate`. Fixes #1343
| * test: Small fixes and improvements to functional helpers.luaThiago de Arruda2014-10-28
| | | | | | | | | | | | | | | | - Move the cleanup function definition into `restart()` so restart can be selectively used as a hook - Improve error handling: Before this, errors while running the event loop would cause busted to get stuck. Now the error is properly raised by stopping the event loop first.
| * msgpack-rpc: Terminate server->client calls when the channel closesThiago de Arruda2014-10-28
| |
| * channel: fix `REQ` definition for msgpack-rpc loggingThiago de Arruda2014-10-28
|/
* Merge PR #1331 'Fixes to job and channel modules'Thiago de Arruda2014-10-23
|\
| * test: verify that msgpacks-rpc exceptions are workingThiago de Arruda2014-10-23
| |
| * msgpack-rpc: Improve logging of msgpack-rpc messagesThiago de Arruda2014-10-23
| | | | | | | | | | | | | | | | | | | | - Expose more logging control from the log.c module(get log stream and omit newlines) - Remove logging from the generated functions in msgpack-gen.lua - Refactor channel.c/helpers.c to log every msgpack-rpc payload using msgpack_object_print(a helper function from msgpack.h) - Remove the api_stringify function, it was only useful for logging msgpack-rpc which is now handled by msgpack_object_print.
| * job: Only decrease refcount after `settmode` in `job_wait`Thiago de Arruda2014-10-23
| | | | | | | | This is required to prevent the scenario explained by @akkartik in #1324
| * travis: Setup xvfb for running xclip testsThiago de Arruda2014-10-23
| |
| * job: Refactor to ensure that all callbacks will be invokedThiago de Arruda2014-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | It's possible that a child process won't close it's standard streams, even after it exits. This can be evidenced with the "xclip" program: :call system('xclip -i -selection clipboard', 'DATA') Before this commit, the above command wouldn't return, even though the xclip program had exited. That is because `xclip` wasn't closing it's stdout/stderr streams, which would block pending_refs from ever reaching 0. Now the job.c module was refactored to ensure all streams are closed when the uv_process_t handle is closed.
| * msgpack-rpc: Fix error handling and leak in call_request_handlerThiago de Arruda2014-10-23
| | | | | | | | | | Free the arguments array after sending the response and also avoid double-sending responses in cases of errors.
| * msgpack-rpc: Ensure stdio channels are properly closedThiago de Arruda2014-10-23
| | | | | | | | | | | | When stdio was closed, parse_msgpack was called with eof == true, with caused a free_channel call. To ensure the correct behavior for all types of channels, the close_channel must be called before free_channel.
| * runtime: Fix rpcstop call in python_setup.vimThiago de Arruda2014-10-23
|/
* Merge pull request #1332 from splinterofchaos/listbrJustin M. Keyes2014-10-23
|\ | | | | vim-patch:7.4.422
| * vim-patch:7.4.422Scott Prager2014-10-23
| | | | | | | | | | | | | | | | Problem: When using conceal with linebreak some text is not displayed correctly. (GrĂ¼ner Gimpel) Solution: Check for conceal mode when using linebreak. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-422
* | Merge pull request #1330 from splinterofchaos/na-patchesJustin M. Keyes2014-10-23
|\ \ | |/ |/| N/A patches
| * version: update patches to 488Scott Prager2014-10-23
| |
| * version: More NA patches.Scott Prager2014-10-23
| |
| * vim-patch:7.4.406Scott Prager2014-10-23
|/ | | | | | | Problem: Test 72 and 100 fail on MS-Windows. Solution: Set fileformat to unix in the tests. (Taro Muraoka) https://code.google.com/p/vim/source/detail?r=v7-4-406
* legacy tests: migrate test35. #1318Nate Sullivan2014-10-22
| | | | | Migrate vim's integration test 35 (increment/decrement commands) to lua/busted.
* Merge PR #1326 'Add verification of log macros'Thiago de Arruda2014-10-22
|\
| * debug: Improve debugging of msgpack-rpc requestsThiago de Arruda2014-10-22
| | | | | | | | | | | | - Add the api_stringify function to display API objects - Use api_stringify to display request arguments and return values in DLOG statements.
| * debug: Fix broken DLOG macro callsThiago de Arruda2014-10-22
| |
| * travis: Force verification of DLOG macrosThiago de Arruda2014-10-22
| | | | | | | | | | | | | | We use INFO as the default log level, which leaves code inside DLOG macros unverified(currently it has compilation errors). Decrease the log level on travis builds for automatic verification in PRs.
| * input: Fix ctrl+c handling in convert_inputThiago de Arruda2014-10-22
|/
* Merge PR #1316 'Refactor event deferral'Thiago de Arruda2014-10-22
|\
| * test: verify that v:shell_error is set by `system()`/`systemlist()`Thiago de Arruda2014-10-22
| |
| * input: Don't remove Ctrl+C from the input_bufferThiago de Arruda2014-10-21
| |
| * event: Remove direct calls to `uv_run` from job.c/shell.cThiago de Arruda2014-10-21
| |
| * travis: Remove `install_functional_test_deps` functionThiago de Arruda2014-10-21
| | | | | | | | | | It is no longer needed after the lua client was added as dependency (@69561ea922ae6789703c06cbc245929d7c625db9)