| Commit message (Collapse) | Author | Age |
... | |
| | |
|
| |
| |
| |
| |
| | |
False positive, see b6c1fae6a94d. Clang ignores the assert introduced in
that commit?
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There is various places where 'conceallevel' and 'concealcursor'
necessitates additional redraws. This tries to separate the different
cases and handle each accordingly:
- Share code with 'cursorline' for the common case: vertical move of
cursor within the same window (concealcursor not active)
- Improve the logic for managing 'concealcursor' and switching modes:
test for the case where the new mode behaves differently from the
last one.
- Clarify the special case for horizontal movement within a line when
'concealcursor' is active, now there is an if-statement only for this
and not hidden in larger check mostly for the first point.
- Keep the special case for moving between windows as is.
|
| |
| |
| |
| | |
vim-patch:8.1.0726: redrawing specifically for conceal feature
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Move `has_eval_provider()` check to `eval_call_provider()` to make sure that
every code path calls it first.
Previously we would, when pynvim was missing, get a nice error message for
`:python3 1`, but not for `:py3file blah`.
Fixes https://github.com/neovim/neovim/issues/9485
|
|\ \
| | |
| | | |
screen: make update_screen() the only entry point for redrawing
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
update_single_line() was only used for 'concealcursor'. But 'cursorline'
has very similiar characteristics (redraw both lines on move cursor
between lines) and works without its own special entry point to the
redraw subsystem.
Later on 'concealcursor' and 'cursorline' could share more logic, but for
now make the former use standard redrawWinline(). Make sure it is called
before update_screen(), so that it is immediately visible.
Get rid of update_prepare() and update_finish(), and all issues from
them and their callsites not being in sync with changes to
update_screen()
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
vim-patch:8.0.1336: cannot use imactivatefunc() unless compiled with +xim
vim-patch:8.0.1338: USE_IM_CONTROL is confusing and incomplete
vim-patch:8.1.0646: cannot build with Ruby 2.6.0
vim-patch:8.1.0656: trying to reconnect to X server may cause problems
vim-patch:8.1.0664: configure "fail-if-missing" does not apply to enable-gui
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Build failure without the +fold feature.
Solution: Add #ifdef.
https://github.com/vim/vim/commit/0e9deefb4fb4f99d0ab90b21ca38260be26bf5de
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: When 'rnu' is set folded lines are not displayed correctly.
(Vitaly Yashin)
Solution: When only redrawing line numbers do draw folded lines.
(closes vim/vim#3484)
https://github.com/vim/vim/commit/7701f308565fdc7b5096a6597d9c3b63de0bbcec
---
Explanation:
Before this patch, relative line numbers would update on a cursor
movement and overwrite fold highlighting in the line number columns.
Other operations can cause the fold highlighting to overwrite the line
number styles. Together, this causes the highlighting in the line number
columns to flicker back and forth while editing.
Test case: create `t.vim` with these contents:
set fdm=marker rnu foldcolumn=2
call setline(1, ["{{{1", "nline 1", "{{{1", "line 2"])
and then call `nvim -u NORC -S t.vim` and press `j`; observe that the fold
highlighting disappears.
|
|\ \ |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Custom operators can't act upon a forced motion. (Christian
Wellenbrock)
Solution: Add the forced motion to the mode() result. (Christian Brabandt,
closes vim/vim#3490)
https://github.com/vim/vim/commit/5976f8ff00efcb3e155a89346e44f2ad43d2405a
closes #8667
closes #9476
Christian Wellenbrock:
> For (most) built in text objects it's possible to force operation on
> them to be linewise, for example by using `dVab` (`:h o_V`,
> `motion_force`). When using custom text objects (defined as mappings
> by plugins for example), this doesn't currently work.
>
> Example:
>
> onoremap x viw
>
> Open a file with a few lines each containing some words. With the
> cursor on any word, try:
>
> 1. `dw` (builtin) deletes some characters
> 2. `dVw` (builtin) deletes linewise
> 3. `dx` (from mapping) deletes some characters
> 4. `dVx` (from mapping) deletes some characters, but should delete
> linewise
ref: https://github.com/wellle/targets.vim/issues/214
ref: https://gitter.im/neovim/neovim?at=5b379ff7f1664406610e7483
|
|\ \ \
| | | |
| | | | |
screen: remove ad-hoc code path for redrawing signs.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
using changed_lines_buf was technically incorrect, as the buffer wasn't
modififed.
|
| | | | |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
API: select items in popupmenu
|
| |/ / / |
|
|/ / /
| | |
| | | |
fixes #9461
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
There is no need to call update_screen() directly in an API function,
mode input processing invokes update_screen() as needed. And if the API
call is done in a context where redraw is disabled, then redraw is
disabled for a reason. A lot of API functions are of equal semantical
strength (nvim_call_function and nvim_execute_lua can also do whatever,
nvim_command is not special), this inconsistency has no purpose.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: Using "gt" sometimes does not redraw a tab. (Jason Franklin)
Solution: Always set must_redraw in redraw_all_later().
https://github.com/vim/vim/commit/04b4e1a42409f81d65ae4801ac867dc0bba3d7b8
ref https://github.com/vim/vim/commit/bf3250a8ad39797e3ccdac82d20c6f19533419e4#r31852304
> I fixed it in a more general way, in that if we don't find a window
> that doesn't have the redraw type set it, then it will not set
> must_redraw, even though that's clearly intended.
ref #9152
ref #9155
NA patches:
vim-patch:8.1.0698
vim-patch:8.1.0699
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Problem: When test_edit fails 'insertmode' may not be reset and the next
test may get stuck. (James McCoy)
Solution: Always reset 'insertmode' after executing a test. Avoid that an
InsertCharPre autocommand or a 'complete' function can change the
state. (closes vim/vim#3768)
https://github.com/vim/vim/commit/8ad16da7290190f55f88073d5586dfe133fddf45
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: get_buf_tv() is named inconsistently.
Solution: Rename it to tv_get_buf(). (Yegappan Lakshmanan, closes vim/vim#3759)
https://github.com/vim/vim/commit/f2d79fa92d0ed90732f52dd88da4ad66c2c5ce13
|
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Sentence text object in Visual mode is not tested.
Solution: Add a test. (Dominique Pelle, closes vim/vim#3758)
https://github.com/vim/vim/commit/6d3a1940be9d3f2b079f81baa9b19f839289b69e
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: Look-behind match may use the wrong line number. (Dominique Pelle)
Solution: Use the line number in regsave instead of the one in behind_pos,
we may be looking at the previous line. (closes vim/vim#3749)
https://github.com/vim/vim/commit/866f3558141aa68862aa2fedbb7747bf2365e838
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Spell highlighting does not always end. (Gary Johnson)
Solution: Also reset char_attr when spell errors are highlighted.
https://github.com/vim/vim/commit/637532b3c0ca41f0de7e90b6f3c0defe06369372
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Leaking memory when updating a single line.
Solution: Do not call start_search_hl() twice.
https://github.com/vim/vim/commit/6d5b4f566a2a50c1de7300336e9e4f5e761500a8
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem: When concealing is active and the screen is resized in the GUI it
is not immediately redrawn.
Solution: Use update_prepare() and update_finish() from
update_single_line().
https://github.com/vim/vim/commit/c10f0e7cb0f35eea489b038e56c87b818eee975b
|
| | |
| | |
| | |
| | |
| | |
| | | |
False positive: vim_fgets has side effects.
ref 8586770e1fd8
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
> V1026 The 'curwin->w_curswant' variable is incremented in the loop.
> Undefined behavior will occur in case of signed integer overflow.
|
|/ / |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
False positive. Documentation for grid_scroll says "`cols` is always
zero, reserved for future use".
|
| |
| |
| |
| |
| | |
False positive: `i` is intentionally, temporarily reassigned.
See a70fde1b4585 #9425
|
| |
| |
| |
| |
| |
| | |
Normally we consider OOM to be fatal, but the diff module has extra
functionality to handle OOM in case huge files are compared. Use
try_malloc instead of xmalloc in that case.
|
| | |
|
| |
| |
| |
| | |
False positive: vim_fgets has side effects.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Decide whether to highlight the visual-selected character under the
cursor, depending on 'guicursor' style:
- Highlight if cursor is blinking or non-block (vertical, horiz).
- Do NOT highlight if cursor is non-blinking block.
Traditionally Vim's visual selection does "reverse mode", which perhaps
conflicts with the non-blinking block cursor. But 'guicursor' defaults
to a vertical bar for selection=exclusive, and this confuses users who
expect to see the text highlighted.
closes #8983
|
| |
| |
| |
| | |
This check is meaningless, we assume the terminal supports reverse-mode.
|
| |
| |
| |
| |
| |
| | |
Problem: Arglist test fails on MS-windows.
Solution: Only use a file name with a double quote on Unix.
https://github.com/vim/vim/commit/3de8c2d1f027410db6a06f0fcd3355d96c8b8596
|
| |
| |
| |
| |
| |
| | |
Problem: :args \"foo works like :args without argument.
Solution: Fix check for empty argument. (closes vim/vim#3728)
https://github.com/vim/vim/commit/2ac372ccee1af6f9fa105bf2648d5e4efa554236
|