aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* vim-patch:8.0.0413: menu test fails on MS-Windows #8173ckelsel2019-01-27
| | | | | | Problem: Menu test fails on MS-Windows using gvim. Solution: First delete the English menus. https://github.com/vim/vim/commit/a1c8ecfda90c0e0e519762ae0521d7f6e297c32e
* Merge #8194 from justinmk/fix-menuJustin M. Keyes2019-01-27
|\ | | | | fix ":menu Item.SubItem", fix menu_get("foo")
| * menu_get(): fix query behaviorJustin M. Keyes2019-01-27
| | | | | | | | | | | | - Return the menu properties, not only its children. - If the {path} param is given, return only the first node. The "next" nodes in the linked-list are irrelevant.
| * menu_get(): Do not include empty itemsJustin M. Keyes2019-01-27
| | | | | | | | | | Caused by a typo: `dict` instead of `dic`. Renamed variable to `d` to make it less similar.
| * fix ":menu Item.SubItem"Justin M. Keyes2019-01-27
|/ | | | | | | | | | | | | | :menu should print sub-menu contents. E.g. this should print the "File.Save" submenu: nvim -u NORC :source $VIMRUNTIME/menu.vim :menu File.Save Regressed in dc685387a3d6 Blocks #8173 menu_get() also was missing some results for some cases.
* Merge #9478 from justinmk/docJustin M. Keyes2019-01-26
|\
| * gen_api_vimdoc.py: Do not wrap on hyphens, long wordsJustin M. Keyes2019-01-26
| | | | | | | | | | | | - Any long symbol is intentional and should never be hardwrapped. - Vim help tags are often hyphenated, and hardwrapping on hyphens breaks the Vim help syntax parser.
| * doc [ci skip] #9478Justin M. Keyes2019-01-26
| | | | | | | | | | | | | | | | | | | | | | | | - Lua - developer guidelines - MAINTAIN.md - TUI: cleanup - TUI: mention Windows terminfo builtins - cleanup if_pyth, redirect python-bindeval tag Helped-by: Björn Linse <bjorn.linse@gmail.com> Helped-by: erw7 <erw7.github@gmail.com>
* | Merge #9552 from janlazo/vim-8.0.0411Justin M. Keyes2019-01-26
|\ \ | |/ |/|
| * vim-patch:8.0.0412: menu test fails on MS-WindowsJan Edmund Lazo2019-01-26
| | | | | | | | | | | | Problem: Menu test fails on MS-Windows. Solution: Use a menu entry with only ASCII characters. https://github.com/vim/vim/commit/5558d19432120696409c007c64d5ba52eed42670
| * vim-patch:8.0.0411: menu translations don't match when case is changed.Jan Edmund Lazo2019-01-26
|/ | | | | | Problem: We can't change the case in menu entries, it breaks translations. Solution: Ignore case when looking up a menu translation. https://github.com/vim/vim/commit/11dd8c1201033dd74e2ea665ba277425b4b965b0
* checkhealth: validate locale (#9548)Justin M. Keyes2019-01-26
| | | | | | Test case: LANG= LC_ALL= LC_CTYPE= tmux nvim -u NORC +'checkhealth nvim'
* Merge pull request #9520 from bfredl/errmsg2Björn Linse2019-01-23
|\ | | | | Lua: format multiline messages / emsgf_multiline()
| * tests/lua: test for multiline error messages in luaBjörn Linse2019-01-23
| |
| * cleanup: reduce some duplicate code, avoid function pointers for a conditionBjörn Linse2019-01-23
| | | | | | | | | | | | | | Add 'multiline' flag to history for correct :messages output Use larger buffer size for multiline messages. if this turns out to not be enough, we could do size calculation like api_set_error
| * ex_echo: reuse code from message.c to show arg to userAlexandre Dubray2019-01-22
| |
| * message.c: add msg_echo_attr functions, use it for lua error messagesAlexandre Dubray2019-01-22
| | | | | | | | | | The added function behaves like the non-echo function but display message in a echo-style way (i.e. tab and newline are preserved)
* | :terminal : Fix F1-F4 key codes (#9535)erw72019-01-23
| | | | | | fixes #9529
* | Merge pull request #9538 from beeender/patch-1Björn Linse2019-01-22
|\ \ | |/ |/| Fix api doc nvim_buf_lines_event example
| * Fix api doc nvim_buf_lines_event exampleChen Mulong2019-01-22
|/ | | | when send_buffer=true, the first nvim_buf_lines_event will come with lastline=-1 instead of 0.
* PVS/V1028 (rework): cast operands, not the result #9531Justin M. Keyes2019-01-21
| | | closes #9522
* CI/codecov: fix invalid yaml [ci skip]Justin M. Keyes2019-01-21
| | | | | | To validate: cat codecov.yml | curl --data-binary @- https://codecov.io/validate
* Merge #9532 from justinmk/fix-fpclassifyJustin M. Keyes2019-01-21
|\
| * build: Fix -Wconversion warnings for fpclassify et alJustin M. Keyes2019-01-21
| | | | | | | | | | | | | | | | | | | | | | | | closes #8274 The parent commit tries a different approach, but that fails on Apple Clang version: Apple LLVM version 10.0.0 (clang-1000.11.45.5) Target: x86_64-apple-darwin17.7.0 which somehow compiles the check_c_source_compiles() check, but then complains during later compilation that __fpclassify is not defined (regardless of "#include <math.h>").
| * build: Fix -Wconversion warnings for fpclassify et alJustin M. Keyes2019-01-20
|/ | | | | | | | | | | | closes #8274 - Instead of #undef and re-#define, define "xfoo" wrappers to avoid include-order sensitivity. - The warnings are bogus, caused by bad interaction between glibc and clang 6+. - https://bugs.llvm.org/show_bug.cgi?id=35268 - https://bugs.llvm.org/show_bug.cgi?id=39738 - https://bugzilla.redhat.com/show_bug.cgi?id=1472437
* CI/codecov: remove "only_pulls" directive [ci skip]Justin M. Keyes2019-01-20
| | | | | Since this was added, codecov has been adding comments to every PR, even though we have specified `comment: off`.
* Merge pull request #9526 from jamessan/remove-jemallocJames McCoy2019-01-20
|\ | | | | Remove support for using jemalloc instead of the system allocator
| * Remove support for using jemalloc instead of the system allocatorJames McCoy2019-01-19
| | | | | | | | | | | | There was never any investigation done to determine whether using jemalloc was actually a net benefit for nvim. It has been a portability limitation and adds another factor to consider when triaging issues.
* | Merge pull request #9429 from bfredl/grid_mouseBjörn Linse2019-01-20
|\ \ | | | | | | API/UI: add nvim_input_mouse() including multigrid mouse support
| * | ui: multigrid mouse supportBjörn Linse2019-01-20
| | |
* | | Merge #9525 'STRICT_ADD, STRICT_SUB: Log error'Justin M. Keyes2019-01-20
|\ \ \ | |_|/ |/| |
| * | STRICT_ADD, STRICT_SUB: Log error before abortJustin M. Keyes2019-01-20
| | |
| * | build: include auto/config.h explicitlyJustin M. Keyes2019-01-20
|/ / | | | | | | | | Otherwise the symbols defined in config/config.h.in may not be defined, depending on include-order.
* | pvscheck.sh: set --sourcetree-root [ci skip]Justin M. Keyes2019-01-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reverts previous experiment. PVS root is working correctly, one can observe this in the PVS-studio.err file, for example: /usr/local/clang-7.0.0/lib/clang/7.0.0/include/stddef.h:51:1: warning: V677 ... /usr/local/clang-7.0.0/lib/clang/7.0.0/include/stddef.h:132:1: warning: V677 ... ./src/nvim/fileio.c:1382:1: warning: V1026 ... ./src/nvim/fileio.c:1388:1: warning: V1026 ... The "./src/nvim/…" paths are correctly rooted, yet PVS somehow still thinks it should analyze "/usr/local/clang-7.0.0/…". See also: https://stackoverflow.com/q/44906903
* | pvscheck.sh: do not set --sourcetree-root [ci skip]Justin M. Keyes2019-01-19
|/ | | | | System headers should be ignored by PVS, but somehow aren't. See also: https://stackoverflow.com/q/44906903
* Merge #9518 from justinmk/pvs-warningsJustin M. Keyes2019-01-18
|\
| * pvscheck.sh: ignore stddef.hJustin M. Keyes2019-01-18
| | | | | | | | | | This system header should be ignored by PVS. See also: https://stackoverflow.com/q/44906903
| * PVS/V560: expression is always trueJustin M. Keyes2019-01-18
| | | | | | | | This macro serves as a sanity-check / documentation.
| * clang/"null pointer dereference": expand_wildcardsJustin M. Keyes2019-01-18
|/ | | | This invariant is explained in aa39fc56f68e.
* Merge #9508 from justinmk/pvs-warningsJustin M. Keyes2019-01-18
|\
| * PVS/V1032: pointer cast to a more strictly aligned typeJustin M. Keyes2019-01-18
| | | | | | | | | | | | | | | | | | | | | | Rework-of: ea7491586fcc Helped-by: Björn Linse <bjorn.linse@gmail.com> - The old (Vim) use of (char_u **)"" before ea7491586fcc is garbage, which hints that this value was never used. - The necessary condition is next to the NULL assigmnent, the pointer would only be started to be accessed, if the length assignment next to it is also changed.
| * PVS/V1032: pointer cast to a more strictly aligned typeJustin M. Keyes2019-01-18
| |
| * PVS/V1028: cast operands, not the resultJustin M. Keyes2019-01-18
| |
| * PVS/V501: diff.c: silence warningJustin M. Keyes2019-01-18
|/ | | | | | False positive: vim_fgets has side effects. ref 8586770e1fd8
* Merge #9292 from mhinz/xcode10Justin M. Keyes2019-01-17
|\
| * ci: switch to Xcode 10.1 / macOS 10.13Marco Hinz2019-01-17
| |
| * tests: load-adjust timer tests (functionaltest)Marco Hinz2019-01-17
| |
| * tests: load-adjust timer tests (oldtest)Marco Hinz2019-01-17
| |
| * tests: fix Test_help_tagjump()Marco Hinz2019-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Vim version of Test_help_tagjump() tests for `:help sm?le` here. That command got removed from Nvim, so the test was changed to check against `:help sp?it` instead. The new test already handled the case that on Win `:h split` would jump to the entry for split() and on all other systems to the entry for :split. Then this commit happened: bb3aa824b lua/stdlib: vim.inspect, string functions Since then `:h split` would jump to split() for macOS as well! I'm not sure why. Anyway, instead of adding another check for has('mac'), we change the test once more to be more akin to the original test. Instead of testing for :smile, which is exclusive to Vim, we check against :checkhealth, which is exclusive to Nvim.
* | test: Lua 5.2/5.3 compatMatěj Cepl2019-01-17
|/ | | | | close #9515 ref #9280