aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* hardcopy.c: fix dead assignmentSander Bosma2017-02-20
| | | | | | `has_mbyte` is deprecated (globals.h), so `outputlen` is always assigned within the if statement. Therefore, the previous initialization is unnecessary.
* message.c: fix dead assignment by removing dead codeSander Bosma2017-02-20
| | | | | `enc_dbcs` and `enc_utf8` are deprecated (globals.h), so the second branch is always taken.
* misc1.c: remove dead initializationSander Bosma2017-02-20
|
* screen.c: remove dead codeSander Bosma2017-02-20
| | | | | | 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.
* screen.c: account for translated string lengthJente Hidskes2017-02-20
| | | | | | | | | | `[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.
* strings.h: Include <stdarg.h> for vim_vsnprintf's use of va_listJames McCoy2017-02-20
| | | | This fully resolves #6141.
* string.c: Include <stdarg.h> for va_list type and va_* macrosJames McCoy2017-02-19
| | | | Closes #6141
* globals.h: Avoid expression in array definition.Justin M. Keyes2017-02-19
| | | | See https://github.com/neovim/neovim/commit/aa56b24ee6553b4417f2c2defdde5be302a868cd#commitcomment-20949000
* Merge #6137 from justinmk/cmdline-ctrl-rJustin M. Keyes2017-02-19
|\ | | | | cmdline: CTRL-R: Omit trailing ^M character
| * cmdline: CTRL-R: Omit trailing <CR>.Justin M. Keyes2017-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The "technically correct" interpretation is to execute the first line that is seen (and this is what happens on middle-click paste in Vim). ^M is only intended to "defuse" the newline, so the user can review it. The parent commit changed the behavior to insert <Space> between lines, but that's a higher-risk change: it is arguably possible that some user *wants* the literal ^M chars when e.g. assigning to a register: :let @a='<C-R>b' To avoid that risk, keep the old behavior and only omit the last ^M. This makes `yy:<C-R>0` nicer at no cost.
| * cmdline: CTRL-R: <Space> instead of CR between lines.Justin M. Keyes2017-02-18
| | | | | | | | | | | | | | | | | | ^M isn't any more "correct" than space: the "technically correct" interpretation is to execute the first line that is seen (and this is what happens on middle-click paste in Vim). ^M is only intended to defuse the newline, so that the user can review the command. We can do that with a space instead, and then the command can be executed without having to fix it up first.
* | eval.c: has("unnamedplus"). (#6136)Kurt Bonatz2017-02-18
| | | | | | | | | | Return 1 for UNIX with a functioning clipboard provider. Closes #6103
* | options: Remove 'esckeys' (#6138)Justin M. Keyes2017-02-18
|/ | | This was never supported and it does not make sense for Nvim.
* doc: README.mdJustin M. Keyes2017-02-17
|
* doc: README.mdJustin M. Keyes2017-02-17
|
* Merge #6110 'refactor: Move vim_*printf to strings.c'.Justin M. Keyes2017-02-17
|\
| * strings: Fix “unexpected format specifier: %lp” ASAN errorZyX2017-02-15
| |
| * message,strings: Move vim_*printf functions to strings.cZyX2017-02-14
| | | | | | | | Allows eval/typval.h to #include message.h.
* | doc/provider: python virtualenvs #6135timeyyy2017-02-17
| | | | | | | | | | | | Closes #1887 Helped-by: Tommy Allen <tommy@esdf.io>
* | Merge #6114 'Partial string handling refactoring'.Justin M. Keyes2017-02-17
|\ \
| * | *: Fix linter errorsZyX2017-02-15
| | |
| * | *: Partial string handling refactoringZyX2017-02-15
| | | | | | | | | | | | | | | | | | | | | | | | Main points: - Replace `char_u` with `char` in some cases. - Remove `str[len] = NUL` hack in some cases when `str` may be considered `const`.
* | | Merge #6105 from justinmk/win32-bindepsJustin M. Keyes2017-02-16
|\ \ \ | | | | | | | | Windows: ship with common tools
| * | | win/CI: Cache dependencies.Justin M. Keyes2017-02-16
| | | |
| * | | win/package: Copy externals without analyzing.Justin M. Keyes2017-02-16
| | | | | | | | | | | | | | | | | | | | | | | | These are just blobs that we jammed into the package. find_program() and WindowsDllCopy.cmake do not make sense here, they search include paths and try to determine DLL dependencies (GetPrerequisites).
| * | | win/package: nvim-qt GUIJustin M. Keyes2017-02-16
| | | | | | | | | | | | | | | | Closes #6126
| * | | win/package: provide common toolsJustin M. Keyes2017-02-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | References #1507 Closes #1811 curl.exe curl_7_52_1_openssl_nghttp2_x86.7z from https://winampplugins.co.uk/curl/ curl 7.52.1 (x86_64-pc-win32) libcurl/7.52.1 OpenSSL/1.0.2k zlib/1.2.8 nghttp2/1.19.0 Protocols: dict file ftp ftps gopher http https imap imaps ldap pop3 pop3s rtsp smb smbs smtp smtps telnet tftp Features: AsynchDNS IPv6 Largefile NTLM SSL libz HTTP2 HTTPS-proxy tidy.exe http://tidybatchfiles.info HTML Tidy for Windows tidy version 5.3.14 date 2017.01.09 compiled for Windows win32 x86. 7za.exe http://www.7-zip.org 7-Zip 16.04 (2016-10-04) cat.exe http://unxutils.sourceforge.net from gVim: diff.exe GNU diffutils version 2.7 xxd.exe V1.10 27oct98 by Juergen Weigert (Win32) ye olde hacked-up tee.exe
| * | | win/build: Download winptyRui Abreu Ferreira2017-02-16
| | | | | | | | | | | | | | | | | | | | Winpty has x86/x64 binary builds, download them when building Neovim.
| * | | win/build: Fix libuv recipe for Windows/NMake X86_64Rui Abreu Ferreira2017-02-16
| | | | | | | | | | | | | | | | | | | | The 64bit check for the libuv recipe worked for the VS generator but not for NMake.
| * | | win/build: detect architectureRui Abreu Ferreira2017-02-16
|/ / /
* | | tui.c: Handle missing "key_dc" terminfo entry (#6128)svaante2017-02-16
| | | | | | | | | Closes #6025
* | | ex_docmd.c: Allow unescaped spaces in :edit filename (#6119)Kurt Bonatz2017-02-15
| | | | | | | | | | | | | | | | | | This makes :edit consistent on all platforms. Also affects :argedit, et al. Wild (tab) completion doesn't work, though. Closes #6010
* | | unittest: Filter out standard defines so that they do not spam stderr (#6113)Nikolai Aleksandrovich Pavlov2017-02-15
|/ /
* | Merge #6108 from ZyX-I/pr-5678Justin M. Keyes2017-02-14
|\ \ | | | | | | Better core dumps checking
| * | ci: Make sure core* is the last component of pathZyX2017-02-14
| | |
| * | ui_detach: Do not redraw during teardown/exit.Justin M. Keyes2017-02-14
| | |
| * | ci: Better core dump checkingZyX2017-02-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Do not exclude any directories from `find` search, remove dumps before tests instead. - Install `apport` on travis so that linux tests should produce core dumps (based on information from travis-ci/travis-ci#3754, not sure whether it still applies). - Check cores in lua so that one has an idea which test is failing exactly. Do this only 10% of time on linux because traversing the file system is slow. Unit tests are still not touched, though it is what `app` argument in `check_cores` is for. TODO? consider using `find`, it may be faster. Consider retiring `os.execute`, dealing with escaping is bad.
| * | ci: Try checking for core dumpsZyX2017-02-14
| | | | | | | | | | | | | | | Note: can’t use `dbg_cmd` string because I need arguments with spaces (i.e. `bt all` and `thread apply all bt full`).
| * | ci: Extract prepare_build() from build_nvim()ZyX2017-02-14
| |/
* | Merge pull request #6118 from lonerover/vim-patchJames McCoy2017-02-14
|\ \ | | | | | | scripts/vim-patch.sh: don't ignore runtime/doc/tagsrch.txt
| * | vim-patch.sh: don't ignore runtime/doc/tagsrch.txtlonerover2017-02-14
| | |
* | | api/helpers: Remove NULL dereference (#6109)Nikolai Aleksandrovich Pavlov2017-02-14
| | | | | | | | | | | | Even though all used compilers can swallow this code, it is still undefined behaviour.
* | | providers: Disable if `g:loaded_*` exists.Justin M. Keyes2017-02-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | The exists('g:loaded_foo') convention is rather common, and may be relied upon in some cases. It's also very unlikely that a user or plugin has any reason to set g:loaded_foo to zero, so the principle of least surprise can be brushed aside here. https://github.com/neovim/neovim/issues/6107#issuecomment-279532143
* | | health.vim: Do not check intentionally disabled providers.Justin M. Keyes2017-02-13
| |/ |/| | | | | Closes #6107
* | ci: Do not use `oldtest` target on Travis. (#6101)Justin M. Keyes2017-02-13
| | | | | | | | | | | | | | | | | | We must invoke src/nvim/testdir/Makefile directly. Explained in 3d1084f264ed08ed75d038510e2e53359d8544eb: > Running tests from the top-level Makefile will use the third-party > dependencies from .deps instead of the ones from the Travis cache. If we could run `oldtest` with CMake, we would not need to do this. Need USES_TERMINAL feature (CMake 3.2+) for that.
* | Merge #6106 from justinmk/health.vimJustin M. Keyes2017-02-13
|\ \ | | | | | | health.vim: Reduce visual noise.
| * | health.vim: ruby provider adviceJustin M. Keyes2017-02-12
| | |
| * | health.vim: Reduce visual noise.Justin M. Keyes2017-02-12
|/ / | | | | | | | | | | - SUCCESS should be extra green, regardless of colorscheme. - Do not highlight INFO and SUGGESTIONS, they should not demand the user's attention.
* | coverity/13728: Null pointer dereference (#6100)Justin M. Keyes2017-02-12
| | | | | | coverity claims that `valid_tabpage(NULL)` can return true...
* | Merge #6084 from justinmk/fix-coverity-155968Justin M. Keyes2017-02-12
|\ \ | | | | | | coverity/155968: resource leak