| Commit message (Collapse) | Author | Age |
|
|
|
|
| |
Needed for accessing this information without direct access to the
rstream_buffer.
|
| |
|
|\
| |
| | |
Let homebrew handle external resources download
|
| |
| |
| |
| |
| | |
When building under Homebrew, we want to let Homebrew manage downloading and
extracting the tarballs.
|
| | |
|
|\ \ |
|
|/ /
| |
| |
| |
| | |
stdout/stderr should only be closed after the job truly exits, or else we can
lose data sent by it.
|
|\ \
| |/
|/| |
Prefer to SHA1 other than MD5 in third-party downloads
|
|/
|
|
| |
If CMake version is less than 2.8.11, fallback to MD5
|
|\ |
|
|/
|
|
|
| |
This command-line flag will suppress all warnings about interior pointers,
which are used in hashtab.c.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/ |
|
|\
| |
| | |
Remove project-specific integer types: long_u. (2)
|
| | |
|
| |
| |
| |
| |
| | |
Replace long with int64_t.
Apart from being convenient, not using long is required by clint.
|
| |
| |
| |
| | |
Replace int with bool where appropriate.
|
| |
| |
| |
| |
| | |
This, in addition to being more correct, allows removing a lot of
explicit casts.
|
| |
| |
| |
| | |
Unused since 457bb2615154946d273d75e07f5d5a936f50ede0.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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.
|
| |
| |
| |
| |
| |
| | |
- 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).
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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).
|
| |
| |
| |
| |
| | |
- Restyle comments (/// when appropiate, // otherwise).
- Improve comments (add new comments, augment/clarify existing ones).
|
|/
|
|
|
| |
Note: No effort is done on refactoring anything in eval module, as it
will be eventually dropped.
|
|\
| |
| | |
version: Refer to nvim help on startup screen.
|
| | |
|
|\ \
| |/
|/| |
re-enable iconv
|
| |
| |
| |
| |
| | |
This seems to have been disabled in the transition from vim to neovim,
re-enable it.
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| | |
These valgrind entries are false positives because vim uses interior pointers.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
| |
This is required for testing scripts under the 'runtime' directory
|
|\
| |
| | |
Restore TEST_FILE functionality for running individual tests.
|
|/ |
|
|\
| |
| | |
build: fix running of functional tests directly with CMake
|
|/
|
|
|
|
|
| |
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.
|
|\
| |
| | |
Fix build warnings.
|
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| | |
Remove build warning for Debug builds:
# warning _FORTIFY_SOURCE requires compiling with optimization (-O)
|
|/ |
|
|\
| |
| | |
build: turn down the _FORTIFY_SOURCE level on gcc >= 4.0
|
|/ |
|
|\ |
|
| | |
|
| | |
|
| | |
|
| | |
|