| Commit message (Collapse) | Author | Age |
... | |
|\ \ \
| | | |
| | | | |
Fix clang analysis warnings. (5)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem : Dereference of null pointer @ 179.
Diagnostic : False positive.
Rationale : Error occurs if `s` paramater is null, which should not
happen.
Resolution : Declare parameter as non null.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem : Dereference of null pointer @ 3247.
Diagnostic : Multithreading issue.
Rationale : Problem only occurs if global `ctrl_x_mode` is modified
while calling function is executing.
Solution : Use local copy instead of global.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
test40.in, test60.in, test_breakindent.in, test_listlbr.in, test_listlbr_utf8.in:
To avoid <tab> triggering wildmode, set wildchar to 'cp' default.
test_breakindent.ok:
Modify expectations to avoid cpoptions+=n
|
|/ / /
| | |
| | |
| | |
| | | |
test69.in:
because 'nocp' sets 'esckeys', add an extra <esc> to avoid 'tm' pause.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Shell command output was broken in @8a5a8db, which refactored nvim to no longer
switch to cooked mode(linefeeds are processed differently).
Fix the problem by refactoring write_output to accept to extra arguments that
control the flushing behavior and where data will be written to: buffer or
directly to the screen.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Closes https://github.com/neovim/docs/pull/26
Also added stubs for 'cryptmethod' and 'key', and placeholders for
explanation regarding removal of crypto functionality.
|
|\ \ \
| |_|/
|/| | |
Fix memory leak detected in #1510.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
ASAN detected this heap-use-after-free.
A job started by channel_from_job() could terminate and result in a call
to free_channel(), while channel_send_call() was still active/pending
and accessing Channel elements.
Original patch by @tarruda.
|
| | |
| | |
| | |
| | |
| | | |
LSAN/ASAN detected, on an error code path, that not all elements of a
struct ChannelCallFrame were freed.
|
| | | |
|
|/ / |
|
| |
| |
| |
| |
| | |
This reverts commit 53bccaaf99cc9564fe5096d37e9aacfd19e3d2d4, reversing
changes made to 4049002f6b1f6db33a497c1f8473ab15233395d5.
|
|\ \
| | |
| | | |
Don't use env vars for configuration time options
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
- Clang's Address Sanitizer options may cause problems when running
the unit tests, restrict the use of these options to the nvim target.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- As a general rule of thumb one shouldn't use environment variables
for setting configuration options for CMake. The reason for this is
we don't know when CMake will be executed and re-evaluate that variable.
- e.g. If we run cmake a first time with a var set, and then run make on
a second session (with no var) and cmake is called because a
dependency changed, the option would be disabled
- This commit removes the use of environment vars from
src/nvim/CMakeLists.txt entirely
- Removed SKIP_UNITTEST since it could only be used to remove a target
at configuration time (and the target was optional anyway)
- Turned SANITIZE into an option, clang-asan.sh now passes cmake
-DSANITIZE=ON
- Removed SKIP_EXEC since it was disabling a target at configuration time
(not being used)
|
|\ \ \
| |_|/
|/| | |
Refactor declaration generation error detection
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Call compiler from CMake instead of lua script to generate a
preprocessor file - allows for better/early error detection if
the compiler fails
- Preprocessor files are saved along with the headers as .i files
- Accept preprocessor lines with trailing chars after # as is
the case in Clang/Windows
- The fourth argument to gendeclarations.lua is now the path to
the proprocessor output file
|
| | |
| | |
| | |
| | |
| | | |
Fix previous changes to be compatible with agreed changes to upstream
vim (https://code.google.com/p/vim/issues/detail?id=287).
|
| |/
|/| |
|
| |
| |
| |
| | |
This is now always defined.
|
| | |
|
| |
| |
| |
| |
| |
| | |
Nvim now relies much less on setting terminal mode to cooked mode, remove most
calls to settmode, except for those that happen on startup or when suspending.
Eventually even those will be handled by the UI layer.
|
| | |
|
|\ \
| | |
| | | |
Fix SegFault when entering Command(q:) or Search(q/) History
|
| | | |
|
|/ / |
|
|\ \
| |/
|/| |
constify and func-attribute memory.c and strings.c
|
| | |
|
| |
| |
| |
| | |
Fix MB_COPY_cHAR() to accept const pointers.
|
|\ \
| | |
| | | |
Remove MiNT, NeXTSTEP, OS/2 References
|
| | |
| | |
| | |
| | |
| | | |
Paul Slootman was removed from the top of os_unix.c as OS/2 is no longer
supported, but is still credited in runtime/doc/intro.txt.
|
| | | |
|
| | |
| | |
| | |
| | | |
Remove trailing whitespace as well.
|
|\ \ \
| | | |
| | | | |
Fix newline substitution causing abort
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem : Command `s/\n//` is being translated into a call to do_join
with a count of 1. But do_join asserts its precondition count
>= 2, which is causing the program to abort.
Note : This in fact revealed bigger problems: generated join command
line count, as well as reported substitutions/lines were
wrong in several cases, since patch 7.4.232.
See:
[patch] http://markmail.org/message/vo7ruair5raccawp
[issue] https://code.google.com/p/vim/issues/detail?id=287
Solution : - Don't generate join command for single-line-range case.
- Make generated join command include:
* lines in range + 1, when range doesn't end at last line.
* lines in range, when range ends at last line.
- Make reported substitutions/lines always be
number-of-lines-joined - 1.
|
|\ \ \
| |/ /
|/| | |
Non-unix-specific os_unix function.
|
| | |
| | |
| | |
| | |
| | | |
Move general initialization functions to early_init, which simplifies
test/unit/helpers.lua, which requires all these functions.
|
| | |
| | |
| | |
| | |
| | | |
Use the portable uv_kill.
Don't bother with archaic BeOS.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Switch from POSIX's write() to fwrite(stdout,...) and disable buffering
since vim buffers output explicitly and flushes when needed, like when a
key is typed.
|
|\ \ \
| | | |
| | | | |
Fix some more -Wconversion warnings.
|
| | | |
| | | |
| | | |
| | | | |
Warnings were fixed in #1488.
|
| | | | |
|
| | | | |
|
| | | | |
|
| |/ / |
|
|\ \ \
| | | |
| | | | |
eval: Fix coverity false positive.
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
** CID 74786: Resource leak (RESOURCE_LEAK)
/src/nvim/eval.c: 10614 in f_jobsend()
/src/nvim/eval.c: 10616 in f_jobsend()
save_tv_as_string() should return NULL and input_len <= 0 for an empty
string or error. Callers should check that input != NULL instead of
input_len > 0 and assert(input == NULL) when the length must be checked.
|