aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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)
* 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
* Merge pull request #9507 from bfredl/noscrollclearBjörn Linse2019-01-15
|\ | | | | screen: don't unconditionally clear messages on window scroll
| * screen: don't unconditionally clear messages on window scrollBjörn Linse2019-01-15
|/ | | | | | | | | | | | | | | In vim, scrolling a window might mess up the cmdline. To keep it simple, cmdline was always cleared for any window scroll. In nvim, where safe scrolling is implemented in the TUI layer, this problem doesn't exist. Clearing the message on scrolling, when we not do it e.g when switching tabs is a bit weird, as the former is a much smaller context change. A vim patch introduced the possibility to avoid the cmdlline clear for redraws caused by async events. This case will now trivially be covered, as the redraw is always avoided. vim-patch:8.0.0592: if a job writes to a buffer screen is not updated
* :recover : Fix crash on non-existent *.swp #9504raffitz2019-01-15
| | | | | | | Reverts d2944e6a298e. mf_open() _can_ fail if the file does not exist. closes #9503 closes #9504
* Merge pull request #9505 from bfredl/inspectBjörn Linse2019-01-14
|\ | | | | lua: expose full interface of vim.inspect and add test
| * lua: expose full interface of vim.inspect and add testBjörn Linse2019-01-14
|/ | | | | | Implement lazy loading for vim.submodule, this would be over-engineering for inspect only, but we expect to use this solution also for more and larger modules.
* Merge #9463 'Lua stdlib'Justin M. Keyes2019-01-14
|\
| * lua/stdlib: Load runtime modules on-demandJustin M. Keyes2019-01-14
| | | | | | | | | | | | | | | | | | | | Instead of eager-loading during plugin/* sourcing, define runtime modules such as `vim.inspect` as lazy builtins. Otherwise non-builtin Lua modules such as `vim.inspect` would not be available during startup (init.vim, `-c`, `--cmd`, …). ref #6580 ref #8677
| * lua/stdlib: vim.inspect, string functionsKillTheMule2019-01-14
|/ | | | | ref #6580 ref #8677
* Merge #9395 from pqzx/api-set-vvarJustin M. Keyes2019-01-14
|\
| * test/API: nvim_set_vvar() #9395Justin M. Keyes2019-01-14
| |
| * API: nvim_set_vvar(): set v: variables #9395jin cong2019-01-14
| | | | | | | | closes #9349
| * dict_set_var: check value before checking its containerJustin M. Keyes2019-01-14
| | | | | | | | | | | | | | - When setting a fixed/locked/readonly var, it is more relevant to report on the key, not its container dict. If its container dict (v:) is readonly, that does not mean the key itself is readonly. - Allow modifying a "fixed" var. "fixed" only prevents deletion.
* | pvscheck.sh: Fix download URL #9500Justin M. Keyes2019-01-14
|/ | | | | - old URL redirects to https://www.viva64.com/en/pvs-studio-download/ - page now contains tgz files for macOS and Linux; the pattern must match only the Linux URL
* Merge #9471 from justinmk/pvs-warningsJustin M. Keyes2019-01-13
|\
| * clang/"null pointer dereference": close_bufferJustin M. Keyes2019-01-13
| | | | | | | | False positive: win_valid_any_tab() already checks `win != NULL`.
| * clang/"Dead assignment": screen.cJustin M. Keyes2019-01-13
| | | | | | | | | | Vim 8.1 source has equivalent structure (this isn't a case of accidental regression), but it depends on FEAT_MBYTE.
| * clang/"Dead assignment"Justin M. Keyes2019-01-13
| |
| * clang/"null passed to nonnull arg": shada.cJustin M. Keyes2019-01-13
| |
generated by cgit (git 2.34.1) at 2025-06-25 04:21:49 +0000