| Commit message (Collapse) | Author | Age |
... | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As reported by clang-scan, `length_modifier` is never read in any code path
following this branch. It is safe to remove.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
`has_mbyte` is deprecated (globals.h), so `outputlen` is always assigned
within the if statement. Therefore, the previous initialization is
unnecessary.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
`enc_dbcs` and `enc_utf8` are deprecated (globals.h), so the second branch is
always taken.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As stated in globals.h, mbyte flags are deprecated, and code using it can be
refractored to remove dead code. Since has_mbyte is defined to true, this
refractoring correct.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
`[RO]` is appended to the status line and `len` is increased with the length of
this string (4). However, the string is marked for translation and may
thus well be larger (or smaller) than 4. Therefore, we check the length at
runtime. The resulting len is never actually used, and thus could be removed.
However, by keeping this line, the body of this if-statement is kept consistent
with surrounding code, and future changes can not forget to add this line when
additional strings are added to p.
|
|\ \ \ \
| | | | |
| | | | |
| | | | | |
vim-patch:7.4.2046
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Problem: qf_init_ext() is too big.
Solution: Move code to qf_parse_line() (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/e87e6dddc2b2a99572ec0db0833c052214c4fbd3
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Problem: Compiler warning for initializing a struct.
Solution: Initialize in another way. (Anton Lindqvist)
https://github.com/vim/vim/commit/bfafb4c4a01db3f8c508716daf689e0dfe92b649
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Problem: The qf_init_ext() function is too big.
Solution: Refactor it. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/e0d3797664c59afc9705808f86a7cf00fd6d874d
|
| | | | | |
|
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Problem: Cannot get all information about a quickfix list.
Solution: Add an optional argument to get/set loc/qf list(). (Yegappan
Lakshmanan)
https://github.com/vim/vim/commit/d823fa910cca43fec3c31c030ee908a14c272640
|
|\ \ \ \
| | | | |
| | | | | |
Better b:changedtick support
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
Avoids loosing b:changedtick value at `:bdelete`.
|
| | | | |
| | | | |
| | | | | |
Avoids use-after-free crashes when compiling with -DEXITFREE.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
Fixes #6147
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Reasonings:
1. It is not used for anything, but scope dictionaries currenly. So there is no
need to generalize and split it into dict_set_var (which will contain some
scope-dictionary-specific checks) and dict_set_value (which will work for any
dictionary).
2. Check for key size is no longer valid for non-scope dictionaries: you *can*
use empty keys there. In scope dictionaries also, but you actually are not
supposed to store there anything, but variables.
Note that actually one may still do
let b:[''] = 1
and “bypass” check for variable name. It won’t change what `echo b:` will show,
but it may affect code which iterates over scope dictionary keys and sets them
to something (if there is such code).
|
| | | | |
| | | | |
| | | | | |
Was replaced by the previous patch. Also fixes legacy test 055.
|
| | | | |
| | | | |
| | | | | |
Port of vim-patch:8.0.0343
|
| | | | |
| | | | |
| | | | | |
Port of vim-patch:8.0.0345
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- They are no longer responble for using gettext.
- They now receive string length and use %.* format specifier in messages.
- And one less global: one of the error messages is never repeated.
|
| | | | |
| | | | |
| | | | | |
Ref vim/vim#1497
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
If I am not mistaking, this commit should not change any functionality.
|
| | | | |
| | | | |
| | | | | |
It still may be unlocked by `:unlock b:.var`.
|
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Refactor writefile() and create more tests for it
|
| | | | | |
| | | | | |
| | | | | | |
This way success/failure return from this function is more precise.
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
1. When calling writefile(list, fname, []) do not show error message twice.
2. Do not allow file name to be overwritten for writefile([1], 2).
3. Do not show “Can’t open file with an empty name” error after error like
“using Float as a String” when type of the second argument is not correct.
4. Do not give multiple error messages and still continue for code like
`writefile(["test", [], [], [], "tset"])`.
Note that to fix 4. ideally I need tv_check_str_or_nr which is currently present
in two PRs: #6114 and #5119. I would want to avoid copying this function into
a yet another PR.
Ref vim/vim#1476.
|
| | | | | |
| | | | | |
| | | | | | |
Previously it could attempt to write trailing newline before returning.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
According to the documentation fsync() may fail with EROFS or EINVAL if “file
descriptor is bound to a special file which does not support synchronization”
(e.g. /dev/stderr). This condition is completely valid in this case since main
point of `file_fsync()` is dumping buffered input.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
terminal: 'modifiable'; 'scrollback'; follow output only if cursor is on last line
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes #2257
Closes #2636
References #2683
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes #2637
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Partial step towards #2637. Will crash if *all* lines are deleted.
Closes #2607
References #5431
|
|/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Problem: CTRL-Z and mouse click use CTRL-O unnecessary.
Solution: Remove stuffing CTRL-O. (James McCoy, closes vim/vim#1453)
https://github.com/vim/vim/commit/74a47162a07fddb532f4bead212f6c80ef474ae7
|