| Commit message (Collapse) | Author | Age |
|---|
| |
|
|
| |
[ci skip]
|
| |
|
|
|
|
| |
:fixdel was removed in 3baba1e7bc6698e6bc9f1d37fce88b30d6274bc9
Helped-by: Michael Reed <m.reed@mykolab.com>
|
| |
|
|
|
|
|
|
|
| |
- backticks in Vim help docs are supported by even the default
colorscheme; we should start using them for technical identifiers
instead of quotation marks.
- prefer judicious indentation to gratuitous bullets
Helped-by: Michael Reed <m.reed@mykolab.com>
|
| |
|
|
|
|
|
| |
Problem: Various small issues.
Solution: Fix those issues.
https://github.com/vim/vim/releases/tag/v7-4-553
|
| |
|
|
|
|
|
|
|
| |
Problem: curs_rows() function is always called with the second argument
false.
Solution: Remove the argument. (Christian Brabandt)
validate_botline_win() can then also be removed.
https://github.com/vim/vim/releases/tag/v7-4-550
|
| | |
|
| |\
| |
| |
| |
| |
| | |
vim_isxdigit, and vim_isspace
Reviewed-by: Michael Reed <m.reed@mykolab.com>
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |/ |
|
| |
|
|
| |
fixes #2458
|
| |\
| |
| |
| | |
Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
|
| | |
| |
| |
| |
| |
| |
| |
| | |
The refactoring of vim_getenv() to remove the mustfree arg
included reworking calling functions. expand_shellcmd was
also using that to track its usage of the variable within
the function, resulting in #2487. This change addresses
that scenario and cleans up some of the function for style.
|
| | |
| |
| |
| |
| | |
- OpenBSD termios.h does not have TAB0, skip
its use if it is not defined
|
| |\ \
| | |
| | | |
Refactor luarocks third-party recipe
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
- Replace repeated use of the luarocks command and build arguments
with variables ${LUAROCKS_BINARY} and ${LUAROCKS_BUILDARGS}
|
| | | |
| | |
| | |
| | |
| | |
| | | |
- When not running the unit tests, building the third-party busted,
nvim-client and other dependencies can be skipped by passing
-DUSE_BUNDLED_BUSTED=FALSE to the third-party project.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Reviewed-by: oni-link <knil.ino@gmail.com>
Reviewed-by: Scott Prager <splinterofchaos@gmail.com>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Problem : Dereference of null pointer @ 4199.
Diagnostic : False positive.
Rationale : Suggested error path implies that newlen == 0 and
newlen > hislen, which implies hislen < 0, which is
impossible (hislen is always between 0 and 10000).
Resolution : Refactor code.
Asserting oldlen >= 0 should have sufficed, but
for some reason analyzer doesn't follow implications in
that case.
Asserting newlen != 0 || newlen <= hislen works.
But the chance has been taken to do extensive
refactoring of this function, as it was difficult to
understand as it was.
As a result of refactoring, assert is not needed anymore,
as we don't call clear_hist_entry() in refactored version.
Refactor : - Rework algorithm:
* Drop guard for OOM case, which can't happen now.
* Drop empty/growing/shrinking cases. Simplify to always
doing the same.
* Perform circular array reordering in all cases
(before, it only did when shrinking).
* Work in batches through memcpy/memset, instead of one
entry at a time, as it did before.
- Inline variable declarations.
- Replace `ssize_t` by `int`.
- Introduce oldlen as entry value of hislen.
- Add a lot of comments.
Helped-by: Scott Prager <splinterofchaos@gmail.com>
Helped-by: oni-link <knil.ino@gmail.com>
|
| |/ /
| |
| |
| |
| |
| |
| | |
Problem : Dead assignment @ 2480.
Diagnostic : Harmless issue.
Rationale : It's true `reg` is not used any more after that point.
Resolution : Delete line.
|
| |\ \ |
|
| | | |
| | |
| | |
| | | |
Reviewed-by: Justin M. Keyes <justinkz@gmail.com>, Nikolay Pavlov <zyx.vim@gmail.com>
|
| | |/
| |
| |
| |
| | |
Reviewed-by: Michael Reed <m.reed@mykolab.com>, Daniel Hahler <github@thequod.de>
Helped-by: Daniel Hahler <github@thequod.de>
|
| |\ \
| |/
|/| |
[RFC] CONTRIBUTING: Mention the wiki's troubleshooting & FAQ articles
|
| |/
|
|
| |
[ci skip]
|
| | |
|
| |\
| |
| |
| |
| |
| | |
Reviewed-by: Scott Prager <splinterofchaos@gmail.com>
Reviewed-by: Michael Reed <m.reed@mykolab.com>
Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
Without the casts*, the compiler rightly warns about the os_getenv
losing the qualifier. This refactor adds a variable to manage this
properly, and renames the original variables to increase clarity.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
See: #459
|
| |\ \
| | |
| | | |
Fix a few uninitialized variable warnings.
|
| | | | |
|
| | |/
| |
| |
| |
| |
| | |
Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
Reviewed-by: Marco Hinz <mh.codebro@gmail.com>
Reviewed-by: Björn Linse <bjorn.linse@gmail.com>
|
| |\ \
| | |
| | | |
Cleanup implementation of registers and fix missing clipboard support in `:redir`
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This fixes missing clipboard support for `:redir`
Helped-By: Michael Reed <m.reed@mykolab.com>
Helped-By: Scott Prager <splinterofchaos@gmail.com>
|
| | | |
| | |
| | |
| | |
| | | |
When clipboard=unnamed and put over visual selection, reduces number of
provider calls from 6 to 2. Also add test.
|
| | | | |
|
| | |/
| |
| |
| |
| |
| |
| | |
This variable isn't stateful, and should be passed around instead.
Helped-By: Scott Prager <splinterofchaos@gmail.com>
Helped-By: Michael Reed <m.reed@mykolab.com>
|
| |\ \
| |/
|/| |
[RFC] build: Misc. small improvements
|
| | |
| |
| |
| |
| |
| |
| | |
No longer needed after
https://github.com/Olivine-Labs/busted/commit/aab5d19d0c615a33b865ebc771fdef80edb99356
This reverts commit d22f2f94047a5ed1e930ee46d0d2e8d6aacfa104.
|
| | |
| |
| |
| |
| | |
- Mention how to disable jemalloc during compilation
- Mention using additional CFLAGS during compilation
|
| | |
| |
| |
| | |
refs #2426
|
| |/ |
|
