aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | | | test: win: yes is unavailable on WindowsJan Edmund Lazo2018-01-06
| | | | | | | | |
| * | | | | | | | win: test: check non-shell system()Jan Edmund Lazo2018-01-06
| | | | | | | | |
| * | | | | | | | test: use unix fileformat to test NULs on systemlistJan Edmund Lazo2018-01-06
| | | | | | | | |
| * | | | | | | | win: enable more functional testsJan Edmund Lazo2018-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - plugin/shada_spec.lua: Use \r\n as Windows EOL for tests on BufWriteCmd, FileWriteCmd, FileAppendCmd. Alternative is 'set fileformat=unix'.
| * | | | | | | | win: enable tests in ex_terminal_specJan Edmund Lazo2018-01-06
| | | | | | | | |
| * | | | | | | | test: enable K_spec tests in WindowsJan Edmund Lazo2018-01-03
| | | | | | | | |
* | | | | | | | | version.c: update [ci skip] (#7780)Marvim the Paranoid Android2018-01-06
| | | | | | | | |
* | | | | | | | | third-party: revert to jemalloc 4.5.0Justin M. Keyes2018-01-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ref #7746 closes #7808 jemalloc-5.0.1 broke the Ubuntu Unstable PPA builds on arm64: https://launchpadlibrarian.net/351647411/buildlog_ubuntu-artful-arm64.neovim_0.2.0ubuntu1+git201712291800+3837+26~ubuntu17.10.1_BUILDING.txt.gz). make[5]: Entering directory '/<<BUILDDIR>>/neovim-0.2.0ubuntu1+git201712291800+3837+26~ubuntu17.10.1/build' Segmentation fault (core dumped) runtime/CMakeFiles/vimball-tags.dir/build.make:57: recipe for target 'runtime/CMakeFiles/vimball-tags' failed make[5]: *** [runtime/CMakeFiles/vimball-tags] Error 139 make[5]: Leaving directory '/<<BUILDDIR>>/neovim-0.2.0ubuntu1+git201712291800+3837+26~ubuntu17.10.1/build' CMakeFiles/Makefile2:7467: recipe for target 'runtime/CMakeFiles/vimball-tags.dir/all' failed jemalloc bug: https://github.com/jemalloc/jemalloc/issues/979
* | | | | | | | | tests: stderr output contains `cp` noiseJustin M. Keyes2018-01-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | closes #7811
* | | | | | | | | defaults: do :filetype stuff unless explicitly "off"Justin M. Keyes2018-01-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Until now, the default `:filetype ...` setup was skipped if the user config touched `:filetype` in any way (including implicitly via `:syntax on`). No one needs that, and it's very confusing. Instead, proceed with `:filetype ... on` unless the user explicitly called `:filetype ... off`. closes #7765
* | | | | | | | | Merge #7805 'eval.c: fix some scan-build warnings'Justin M. Keyes2018-01-03
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add null check when adding variable to dict.Paul Rigge2018-01-02
| | | | | | | | | |
| * | | | | | | | | Add null pointer assertions for do_unlet_var.Paul Rigge2018-01-02
| | | | | | | | | |
| * | | | | | | | | Refactor profiling check in call_user_func.Paul Rigge2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | do_profiling is a global variable, and as such the clang static analyzer has trouble making arguments about it. This commit does one comparison against do_profiling and puts the result in a local variable. This prevents errors from the value of do_profiling changing between comparisons.
| * | | | | | | | | Add assertions for watchersPaul Rigge2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The clang static analyzer convinced itself lp->ll_newkey could be NULL. This adds an assertion that checks this doesn't actually happen, as well as a parallel assertion for di->di_key.
| * | | | | | | | | Add another const to tv_copyPaul Rigge2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clang static analyzer had trouble with filter_map in eval.c because tv_copy could, in principle, change the v_type of argvars[0]. It saw a potential null pointer going somewhere it shouldn't as a result. The from argument in tv_copy should be const, which also cleans up the static analyzer's complaint.
| * | | | | | | | | Add assertion in set_var_lval for null pointer.Paul Rigge2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the lval is a index into a list, li should not be null.
* | | | | | | | | | vim-patch:8.0.0955: Test_existent_file() fails on some file systemsJustin M. Keyes2018-01-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test_existent_file() fails on some file systems. Solution: Run the test again with a sleep when the test fails without a sleep. (James McCoy, closes vim/vim#1984) https://github.com/vim/vim/commit/82de3c2c036bc89c2d9bdea236e0a7f1208a5571
* | | | | | | | | | Merge #7801 'tutor: adjust for 80 columns'Justin M. Keyes2018-01-03
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | tutor: install metadata files for tutor documentsFelipe Morales2018-01-01
| | | | | | | | | | |
| * | | | | | | | | | tutor: readjust tutor for 80 char wide terminalsFelipe Morales2018-01-01
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #7804 from jamessan/vim-8.0.0160James McCoy2018-01-03
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | | [RFC] vim-patch:8.0.0160
| * | | | | | | | | | lintJames McCoy2018-01-02
| | | | | | | | | | |
| * | | | | | | | | | vim-patch:8.0.0160: EMSG() is sometimes used where it should be IEMSG()James McCoy2018-01-02
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: EMSG() is sometimes used for internal errors. Solution: Change them to IEMSG(). (Dominique Pelle) And a few more. https://github.com/vim/vim/commit/de33011ec623fd562419dede6bf465b5b9881a20
* | | | | | | | | | Merge pull request #7803 from jamessan/remove-ci-special-casesJames McCoy2018-01-02
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | [RFC] Simplify Travis builds
| * | | | | | | | | | ci: asan_check: No-op unless performing ASAN buildJames McCoy2018-01-02
| | | | | | | | | | |
| * | | | | | | | | | travis: Reduce stages to flaky builds (gcov, tsan) and everything elseJames McCoy2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Separating the non-flaky builds (asan, normal builds, lint) into separate stages simply slowed down overall CI turnaround. Since none of the builds rely on the output of others, reducing the stages increases the opportunities for parallel builds.
| * | | | | | | | | | travis: Remove ubuntu-r-toolchain/test ppaJames McCoy2018-01-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use unversioned gcc/gcov commands rather than pulling in a separate repo.
| * | | | | | | | | | travis: Use Ubuntu's clang instead of llvm's repoJames McCoy2018-01-02
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The llvm repos commonly have access issues, so removing them will improve stability of the Travis builds. Filtering check_log's output through asan_symbolize also avoids the version dance every time a new clang version makes its way into Travis.
* | | | | | | | | | tui: support TERM=konsole-256colorJustin M. Keyes2018-01-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | TERM=konsole-256color is recognized by ncurses. TERM=konsole-xterm might be more clever, but should not be necessary (for Nvim at least), we already special-case Konsole in various places. We may need to clean up some areas that currently assume Konsole always "pretends xterm" (`TERM=xterm-256color`), though I didn't find any such cases. ref #6403 ref https://github.com/neovim/neovim/issues/6403#issuecomment-348713346
* | | | | | | | | | Merge #7800 'vim patches'Justin M. Keyes2018-01-01
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | vim-patch:8.0.0336: flags of :substitute not sufficiently testedJustin M. Keyes2018-01-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Flags of :substitute not sufficiently tested. Solution: Test up to two letter flag combinations. (James McCoy, closes vim/vim#1479) https://github.com/vim/vim/commit/8c50d50b6e19b755d7bad7b2724d14ead29364a7
| * | | | | | | | | | vim-patch: b:changedtick-related patchesJustin M. Keyes2018-01-01
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | vim-patch:8.0.0334 vim-patch:8.0.0335 vim-patch:8.0.0343 vim-patch:8.0.0345 Problem: Can't access b:changedtick from a dict reference. Solution: Make changedtick a member of the b: dict. (inspired by neovim vim/vim#6112) https://github.com/vim/vim/commit/79518e2ace5fce7b9c49060e462a6e935dba0a84 vim-patch:8.0.0343: b:changedtick can be unlocked Problem: b:changedtick can be unlocked, even though it has no effect. (Nikolai Pavlov) Solution: Add a check and error E940. (closes #1496) vim-patch:8.0.0345: islocked('d.changedtick') does not work Problem: islocked('d.changedtick') does not work. Solution: Make it work. vim-patch:8.0.0335: functions test fails Problem: Functions test fails. Solution: Use the right buffer number. https://github.com/vim/vim/commit/507647da3151f7ffccac1b217936240daa79849c
* / / / / / / / / / vim-patch:8.0.0339: illegal memory access with vi' (#7794)KunMing Xie2018-01-01
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Illegal memory access with vi' Solution: For quoted text objects bail out if the Visual area spans more than one line. https://github.com/vim/vim/commit/46522af72424c7fadfa7a4cbba3dd21b82d19131
* | | | | | | | | Merge pull request #7795 from jamessan/vim-8.0.0591James McCoy2017-12-31
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | [RFC] vim-patch:8.0.0591,8.0.0634,8.0.0641,8.0.0657
| * | | | | | | | doc: deprecate 'gdefault'James McCoy2017-12-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | | | vim-patch:8.0.0657: cannot get and set quickfix list itemsJames McCoy2017-12-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot get and set quickfix list items. Solution: Add the "items" argument to getqflist() and setqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/6a8958db259d4444da6e6956e54a6513c1af8860
| * | | | | | | | vim-patch:8.0.0641: cannot set a separate highlighting for the quickfix lineJames McCoy2017-12-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot set a separate highlighting for the current line in the quickfix window. Solution: Add QuickFixLine. (anishsane, closes vim/vim#1755) https://github.com/vim/vim/commit/2102035488e80ef6fd5038ed15d21672712ba0f6
| * | | | | | | | vim-patch:8.0.0634: cannot easily get to the last quickfix listJames McCoy2017-12-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot easily get to the last quickfix list. Solution: Add "$" as a value for the "nr" argument of getqflist() and setqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/875feea6ce223462d55543735143d747dcaf4287
| * | | | | | | | vim-patch:8.0.0591: changes to eval functionality not documentedJames McCoy2017-12-30
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Changes to eval functionality not documented. Solution: Include all the changes. https://github.com/vim/vim/commit/45d2cca1ea3f90fc70ad99d0c6812a9d8536303c
* | | | | | | | Merge pull request #7762 from ZyX-I/remove-some-listitemsJustin M. Keyes2017-12-31
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove some tv_list_item_…() functions
| * | | | | | | | eval: Fix linter errorsZyX2017-12-31
| | | | | | | | |
| * | | | | | | | eval,lua/converter: Fix problems spotted in reviewZyX2017-12-31
| | | | | | | | |
| * | | | | | | | eval: Refactor some potentially dangerous list appendsZyX2017-12-25
| | | | | | | | |
| * | | | | | | | eval: Remove magic numbers from find_some_match() type argumentZyX2017-12-25
| | | | | | | | |
| * | | | | | | | *: Fix linter errorsZyX2017-12-24
| | | | | | | | |
| * | | | | | | | eval: Replace some tv_list_item_remove() callsZyX2017-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is nothing wrong with them, just it is generally better to remove a range then to remove items individually.
| * | | | | | | | unittests: Do gc after reporting error, not beforeZyX2017-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reason: test may contain cleanup at the endwhich is needed for GC to work properly, but is not done if test fails. With collectgarbage() in former position it would crash when collecting garbage.
| * | | | | | | | unittests: Remove start of trace, not endZyX2017-12-24
| | | | | | | | |
| * | | | | | | | eval: Rename tv_list_remove_items() to tv_list_drop_items()ZyX2017-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | tv_list_remove_items() may cause confusion with tv_list_item_remove()