| Commit message (Collapse) | Author | Age |
... | |
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
By "simple pointer" I mean a pointer that can be freed with a call to `free`
without leaking any member pointer.
This macro does exactly what `ga_clear_strings` does.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
Used to free garrays of `salitem_T` and `fromto_T` in spell.c, and
garray `wcmd_T` in ex_docmd.c.
Helped-by: Jiaqi Li
|
| |/
|/| |
|
| |
| |
| |
| |
| | |
Also get rid of some platform references made irrelevant by the removal
of their respective platform specific GUI code.
|
| |
| |
| |
| |
| |
| |
| | |
Even when this was finally removed 6 months ago in b2b920f, it had
already been disabled for a while. Due to this, just remove all remnants
of the option as opposed to putting a placeholder like what was done for
'shortname'and 'cryptmethod'.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
The remote_ui module is an implementation of the UI layer, and it attaches UI
instances that redirect redraw notifications to connected clients.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Instead of using classic cterm color numbers and attributes, treat "abstract_ui"
as a GUI: Pass rgb color numbers and gui attributes when the "highlight_set" UI
method is called. The terminal UI will have to translate RGB color information
to an appropriate color number, and the "term"/"cterm" :highlight keys will
eventually be deprecated.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This is how Nvim behaves when the "abstract_ui" termcap is activated:
- No data is written/read to stdout/stdin by default.
- Instead of sending data to stdout, ui_write will parse the termcap codes
and invoke dispatch functions in the ui.c module.
- The dispatch functions will forward the calls to all attached UI
instances(each UI instance is an implementation of the UI layer and is
registered with ui_attach).
- Like with the "builtin_gui" termcap, "abstract_ui" does not contain any key
sequences. Instead, vim key strings(<cr>, <esc>, etc) are parsed directly by
input_enqueue and the translated strings are pushed to the input buffer.
With this new input model, its not possible to send mouse events yet. Thats
because mouse sequence parsing happens in term.c/check_termcodes which must
return early when "abstract_ui" is activated.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
This allows to configure which of '*' and '+' should be used for
the unnamed clipboard, and is consistent with vim.
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
strings: Remove NONNUL_ALL from NULL-taking functions.
|
| | |
| | |
| | |
| | |
| | | |
vim_strup() is only used in a couple places, with no NULLs possible, so
keep the attribute and remove the NULL check.
|
|\ \ \
| | | |
| | | | |
Api: Improve Nul handling
|
| |/ / |
|
|\ \ \
| | | |
| | | | |
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
|