diff options
| author | zeertzjq <zeertzjq@outlook.com> | 2024-03-21 07:50:12 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-21 07:50:12 +0800 |
| commit | c1c6c1ee12fa601194a410f78ecde11a9982a793 (patch) | |
| tree | b4d9bd7d18728b191debbb4f519ae8b29eeb39dd /src/nvim/api/vim.c | |
| parent | cfc9fcc91f780022e5e9397626ac6ab5cde91d62 (diff) | |
| download | rneovim-c1c6c1ee12fa601194a410f78ecde11a9982a793.tar.gz rneovim-c1c6c1ee12fa601194a410f78ecde11a9982a793.tar.bz2 rneovim-c1c6c1ee12fa601194a410f78ecde11a9982a793.zip | |
vim-patch:9.1.0190: complete_info() returns wrong order of items (#27955)
Problem: complete_info() returns wrong order of items
(after v9.0.2018)
Solution: Revert Patch v9.0.2018
(Girish Palya)
bug fix: complete_info() gives wrong results
1) complete_info() reverses list of items during <c-p>
2) 'selected' item index is wrong during <c-p>
3) number of items returnd can be wrong
Solution:
- Decouple 'cp_number' from 'selected' index since they need not be
correlated
- Do not iterate the list backwards
- Add targeted tests
Regression introduced by https://github.com/vim/vim/commit/69fb5afb3bc9da24c2fb0eafb0027ba9c6502fc2
Following are unnecessary commits to patch problems from above:
https://github.com/vim/vim/commit/fef66301665027f1801a18d796f74584666f41ef
https://github.com/vim/vim/commit/daef8c74375141974d61b85199b383017644978c
All the tests from above commits are retained though.
fixes: vim/vim#14204
closes: vim/vim#14241
https://github.com/vim/vim/commit/8950bf7f8b85c1287d4e696965d88091fcc60594
Remove EMPTY_IF_NULL() as it has been unnecessary since #12673.
Co-authored-by: Girish Palya <girishji@gmail.com>
Diffstat (limited to 'src/nvim/api/vim.c')
0 files changed, 0 insertions, 0 deletions