aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* rstream: Expose rstream_read_ptr and _available.Scott Prager2014-11-07
| | | | | Needed for accessing this information without direct access to the rstream_buffer.
* memory: implement xmemrchr()Scott Prager2014-11-07
|
* Merge pull request #1411 from xu-cheng/homebrew-formulaJohn Szakmeister2014-11-07
|\ | | | | Let homebrew handle external resources download
| * build: don't download and extract tarball if source is already presentXu Cheng2014-11-07
| | | | | | | | | | When building under Homebrew, we want to let Homebrew manage downloading and extracting the tarballs.
| * Homebrew: let Homebrew handle external resources downloadXu Cheng2014-11-07
| |
* | Merge PR #1424 'Only force-close stdout/stderr when the job exits'Thiago de Arruda2014-11-07
|\ \
| * | job: Only force-close stdout/stderr when the job exitsThiago de Arruda2014-11-07
|/ / | | | | | | | | stdout/stderr should only be closed after the job truly exits, or else we can lose data sent by it.
* | Merge pull request #1414 from xu-cheng/sha1_md5John Szakmeister2014-11-07
|\ \ | |/ |/| Prefer to SHA1 other than MD5 in third-party downloads
| * Prefer to SHA1 other than MD5 in third-party downloadsXu Cheng2014-11-07
|/ | | | If CMake version is less than 2.8.11, fallback to MD5
* Merge PR #1422 'Pass --show-possibly-lost=no to valgrind'Thiago de Arruda2014-11-07
|\
| * test: Pass --show-possibly-lost=no to valgrind in helpers.luaThiago de Arruda2014-11-07
|/ | | | | This command-line flag will suppress all warnings about interior pointers, which are used in hashtab.c.
* Merge PR #1421 'Improve test error handling and reporting'Thiago de Arruda2014-11-07
|\
| * test: Improve test environment setup and error handling/reportingThiago de Arruda2014-11-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During test setup, we used to call a vimscript function(BeforeEachTest) that attempted to restore Nvim to it's initial state as much as possible in order to provide a clean environment for running new tests. This approach has proven to be unreliable, as some tests leave state that can affect other tests, eventually causing failures that are difficult to debug. This commit changes the 'clear' function so it will restart Nvim every time it is called, which is a slower, but more reliable solution that will simplify spotting bugs in the future. Some other improvements/fixes were also performed: - Whenever an error is detected in a handler passed to "run()", the event loop will be stopped and the error will be propagated to the main thread. - Errors and the "cleanup()" function will always send a quit command to the current Nvim instance. This should prevent memory starvation when running tests under valgrind(where each Nvim instance can consume a lot of memory). - Fixed a wrong assertion in server_requests_spec.lua. Previously the failure was undetected in a notification handler. - Fixed some tests to expect fully clean registers. The deleted cleanup function used to put an empty string in every register, but that resulted in a extra line being added.
| * deps: Upgrade to latest version of lua-clientThiago de Arruda2014-11-06
|/
* Merge pull request #1340 from elmart/remove-long_uJustin M. Keyes2014-11-06
|\ | | | | Remove project-specific integer types: long_u. (2)
| * Review: Remove long_u: memfile: Add to clint.Eliseo Martínez2014-11-06
| |
| * Review: Remove long_u: memfile: Refactor: blocknr_T: long -> int64_t.Eliseo Martínez2014-11-06
| | | | | | | | | | Replace long with int64_t. Apart from being convenient, not using long is required by clint.
| * Review: Remove long_u: memfile: Refactor: int -> bool.Eliseo Martínez2014-11-06
| | | | | | | | Replace int with bool where appropriate.
| * Review: Remove long_u: memfile: Refactor: bhdr_T.bh_data: char_u* -> void*.Eliseo Martínez2014-11-06
| | | | | | | | | | This, in addition to being more correct, allows removing a lot of explicit casts.
| * Review: Remove long_u: memfile: Refactor: mf_hashtab_T.mht_fixed: Remove.Eliseo Martínez2014-11-06
| | | | | | | | Unused since 457bb2615154946d273d75e07f5d5a936f50ede0.
| * Review: Remove long_u: memfile: Refactor: long_u -> size_t.Eliseo Martínez2014-11-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - memfile_defs.h: * hashtab_T: mht_mask: long_u -> size_t. Masks are used to truncate keys to a value adequate for an index in the array of entries. Value of the mask plus one is the current size of the array. Both of those reasons imply the soundness of size_t for this type. * hashtab_T: mht_count: long_u -> size_t. - memfile.c: * total_mem_used: long_u -> size_t. * mf_hash_free_all: idx: long_u -> size_t. * mf_hash_add_item: idx: long_u -> size_t. * mf_hash_find: idx: long_u -> size_t. * mf_hash_grow: i: long_u -> size_t. * mf_hash_grow: j: long_u -> size_t.
| * Review: Remove long_u: memfile: Enable -Wconversion.Eliseo Martínez2014-11-06
| | | | | | | | | | | | | | | | | | | | | | | | | | - Add memfile.c to converted files list. - Fix conversion issues: * bhdr_T : bh_page_count : int -> unsigned. * bhdr_T : bh_flags : char -> unsigned. * mf_new() : page_count : int -> unsigned. * mf_get() : page_count : int -> unsigned. * mf_release() : page_count : int -> unsigned. * mf_alloc_bhdr() : page_count : int -> unsigned. * mf_trans_add() : page_count : int -> unsigned. * mf_put() : flags : int -> unsigned.
| * Review: Remove long_u: memfile: Cleanup: Fix naming inconsistencies.Eliseo Martínez2014-11-06
| | | | | | | | | | | | - Drop '_S' suffix for struct names. - Make struct names match corresponding type's name (just removing '_S' suffix). - Rename NR_TRANS type/struct (just ugly).
| * Review: Remove long_u: memfile: Cleanup: Others.Eliseo Martínez2014-11-06
| | | | | | | | | | | | | | | | | | | | | | | | memfile_defs.h: - Inline struct definitions in typedefs. - Move memfile_T definition to this file (weirdly, was in buffer_defs.h). memfile.c: - Use C99 style variable declarations. This is, move variable declarations as near to first-usage point as possible). - Modernize old-style function declarations. - Fix indent at some places (some multiline expressions and the like).
| * Review: Remove long_u: memfile: Cleanup: Comments.Eliseo Martínez2014-11-06
| | | | | | | | | | - Restyle comments (/// when appropiate, // otherwise). - Improve comments (add new comments, augment/clarify existing ones).
| * Remove long_u: eval: Replace with size_t.Eliseo Martínez2014-11-06
|/ | | | | Note: No effort is done on refactoring anything in eval module, as it will be eventually dropped.
* Merge pull request #1412 from fwalch/startup-helpJustin M. Keyes2014-11-06
|\ | | | | version: Refer to nvim help on startup screen.
| * version: Refer to nvim help on startup screen.Florian Walch2014-11-06
| |
* | Merge pull request #1370 from aktau/enable-iconvJustin M. Keyes2014-11-06
|\ \ | |/ |/| re-enable iconv
| * iconv: re-enableNicolas Hillegeer2014-11-06
| | | | | | | | | | This seems to have been disabled in the transition from vim to neovim, re-enable it.
* | Merge PR #1398 'Tests and msgpack rpc improvements'Thiago de Arruda2014-11-06
|\ \ | |/ |/|
| * channel: Delay notifications to avoid client race conditionsThiago de Arruda2014-11-06
| | | | | | | | | | | | | | | | | | | | It is currently possible for a client to send a response that doesn't match the current server->client request(at the top of the stack). This commit fixes that by delaying notifications to until the first `channel_send_call` invocation returns. Also remove the "call stack" size check, vim will already break if the call stack goes too deep.
| * test: Add more valgrind "possibly lost" suppressionsThiago de Arruda2014-11-06
| | | | | | | | These valgrind entries are false positives because vim uses interior pointers.
| * test: Simplify/fix options_spec.luaThiago de Arruda2014-11-06
| | | | | | | | | | | | | | | | | | | | The options_spec.lua suite has one purpose: Check if the :options commands will throw any exception(:options is implemented by $VIMRUNTIME/optwin.vim). For this it is best to use the `vim_command` API function since it will automatically catch exceptions and forward them via msgpack-rpc. Also, the option window seems to affect other tests, so call `restart` in the teardown hook.
| * test: Set VIMRUNTIME before running testsThiago de Arruda2014-11-05
|/ | | | This is required for testing scripts under the 'runtime' directory
* Merge pull request #1397 from fwalch/fix-test_fileJohn Szakmeister2014-11-05
|\ | | | | Restore TEST_FILE functionality for running individual tests.
| * CMake: Fix running individual (functional/unit) test.Florian Walch2014-11-05
|/
* Merge pull request #1395 from jszakmeister/fix-run-functional-testsJohn Szakmeister2014-11-05
|\ | | | | build: fix running of functional tests directly with CMake
| * build: fix running of functional tests directly with CMakeJohn Szakmeister2014-11-05
|/ | | | | | | If you aren't just building everything into build/, then the functional tests fail because they can't find the nvim executable. Let's pass in the location of the nvim executable, and set NVIM_PRG environment variable accordingly.
* Merge pull request #1392 from fwalch/fix-build-warningsJohn Szakmeister2014-11-05
|\ | | | | Fix build warnings.
| * CMake: Rename target "test" to "functionaltest".Florian Walch2014-11-05
| | | | | | | | | | | | | | | | Remove build warning: The target name "test" is reserved or not valid for certain CMake features, such as generator expressions, and may result in undefined behavior.
| * CMake: Modify _FORTIFY_SOURCE only for release builds.Florian Walch2014-11-05
| | | | | | | | | | | | Remove build warning for Debug builds: # warning _FORTIFY_SOURCE requires compiling with optimization (-O)
| * CMake: Pass build type CFLAGS to header generator.Florian Walch2014-11-05
|/
* Merge pull request #1390 from jszakmeister/turn-down-fortify-sourceJohn Szakmeister2014-11-05
|\ | | | | build: turn down the _FORTIFY_SOURCE level on gcc >= 4.0
| * build: turn down the _FORTIFY_SOURCE level on gcc >= 4.0John Szakmeister2014-11-05
|/
* Merge PR #1328 'Migrate legacy tests'Thiago de Arruda2014-11-04
|\
| * legacy tests: migrate test21Rainer Borene2014-11-04
| |
| * legacy tests: migrate test7Rainer Borene2014-11-04
| |
| * legacy tests: migrate test6Rainer Borene2014-11-04
| |
| * legacy tests: migrate test28Rainer Borene2014-11-04
| |