aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | | | | | | | refactor/single-include: cursor_shape.h #6442dedmass2017-04-08
| | | | | | | | |
* | | | | | | | | lintJames McCoy2017-04-07
| | | | | | | | |
* | | | | | | | | vim-patch:8.0.0550James McCoy2017-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Some etags format tags file use 0x01, breaking the parsing. Solution: Use 0x02 for TAG_SEP. (James McCoy, closes vim/vim#1614) https://github.com/vim/vim/commit/9585a1655ba0d34ea88574617112093a9bd4f2e9
* | | | | | | | | vim-patch:8.0.0499James McCoy2017-04-07
| |_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: taglist() does not prioritize tags for a buffer. Solution: Add an optional buffer argument. (Duncan McDougall, closes vim/vim#1194) https://github.com/vim/vim/commit/c6aafbaf3ea755e3ab4ee2e3045911126a08b038
* | | | | | | | win: os_get_hostname() #5416 (#6413)Justin M. Keyes2017-04-07
| | | | | | | |
* | | | | | | | eval: delimit string with NUL byte (#6467)Nicolas Hillegeer2017-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A recent refactor left cpy without a NUL terminator, simplify the code instead of patching over it. Instead of plain memcpy, it'd be better to employ harder to misuse string functions made for this purpose like xstrlcpy(), but path_tail() takes char_u arguments and returns them, leading to a lot of ugly casting. Fixes #6431.
* | | | | | | | completion: fix segfault with ignorecase+infercase (#6452)Justin M. Keyes2017-04-06
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Helped-by: Matthew Malcomson <hardenedapple@gmail.com> Closes #6451
* | | | | | | refactor/single-include: diff.h (#6443)Carlo Abelli2017-04-06
|/ / / / / /
* | | | | | options: remove 'guiheadroom'Justin M. Keyes2017-04-04
| | | | | |
* | | | | | 'guicursor': Disable by default for unknown terminals.Justin M. Keyes2017-04-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User can still set guicursor explicitly in init.vim. Closes #5990 Closes #6403
* | | | | | 'guicursor': enabled=false if 'guicursor' is emptyJustin M. Keyes2017-04-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #6429 Closes #6430
* | | | | | 'listchars': `Whitespace` highlight group #6367Yichao Zhou2017-04-03
| | | | | |
* | | | | | fileio: Fix most linter errorsZyX2017-04-03
| | | | | | | | | | | | | | | | | | One error is still kept: buf_write function is too large.
* | | | | | fileio: Save details about E212 errorZyX2017-04-03
| | | | | |
* | | | | | eval: Add s flag, use p_fs by default, error out on unknown flagZyX2017-04-03
| | | | | |
* | | | | | fileio: Refactor msg_add_fname to something which needs no commentsZyX2017-04-03
| | | | | |
* | | | | | fileio: Clean up IObuff-manipulation messZyX2017-04-03
| | | | | |
* | | | | | fixup!ZyX2017-04-03
| | | | | |
* | | | | | message: Do not use IObuff in emsgfZyX2017-04-03
| | | | | |
* | | | | | fileio: Refactor errmsg handlingZyX2017-04-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds os_strerror() result to a number of places. Also since I could not track where err\* variables are NULL and where they are not, using macros to make sure that all three variables are set at once. Removes #ifdef UNIX around the use of os_fsync, makes it use os_close in place of close in some places.
* | | | | | eval: Make writefile() able to disable fsync()ZyX2017-04-02
| | | | | |
* | | | | | Merge #6423 from justinmk/guicursorJustin M. Keyes2017-04-02
|\ \ \ \ \ \
| * | | | | | api/cursor_style_set: mode descriptionsJustin M. Keyes2017-04-01
| | | | | | |
| * | | | | | 'guicursor': Empty means "block cursor in all modes".Justin M. Keyes2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Also: update default 'guicursor' to match the documentation.
| * | | | | | tui: 'guicursor' colorMatthieu Coudron2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For now only supports valid hex colors (does not check for the validity the hex color) when termguicolors is set, otherwise it won't attempt to change the cursor color.
| * | | | | | tui: 'guicursor' shape #6044Matthieu Coudron2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #2583
* | | | | | | Merge #6409 from ZyX-I/separate-clintJustin M. Keyes2017-04-02
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | cmake: Add `clint` target to build Makefile
| * | | | | | cmake: Also depend on LINT_PRGZyX2017-03-31
| | | | | | |
| * | | | | | clint: Do not report zero errorsZyX2017-03-31
| | | | | | |
| * | | | | | cmake: Replace RunLint.cmake with code in src/nvim/CMakeLists.txtZyX2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also removes LINT_FILE environment variable, other then that functionality is kept. It is expected that developers needing partial linting will use `make lint`, touching interesting file before (if not done already by writing to them).
| * | | | | | cmake: Allow failing to download small suppress filesZyX2017-03-31
| | | | | | |
| * | | | | | cmake: Use file-specific supprressesZyX2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `make -C build clint` time is now make -j5 clint 95.29s user 1.86s system 409% cpu 23.751 total *without* downloading anything (much worse if something was not cached, still a bit better then top-level `make clint`). But since without neovim/bot-ci#95 it is downloading each file one-by-one total time with download (download also parallel!) is make -j5 -B clint 99.29s user 2.98s system 258% cpu 39.634 total Top-level makefile still gives make -j5 clint 59.33s user 0.28s system 95% cpu 1:02.41 total
| * | | | | | cmake: Add `clint` target to build MakefileZyX2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows linting only modified files and linting multiple files in parallel. In the current state is rather slow because errors.json is a 6 MiB file and needs to be reparsed each time. Results on my system (6-core): # In build dir, actually parallel make -j5 clint 241.24s user 8.39s system 334% cpu 1:14.74 total # In root, one process make -j5 clint 60.69s user 0.37s system 93% cpu 1:05.19 total In both cases download time included. That is not well for travis (though I would keep travis as-is because new variant will fail before checking all files), but already good enough for regular development: total times are nearly identical and this is the *full* build, further `make -C build clint` will check only modified files.
* | | | | | | tui: Only enable/disable mouse when there's something to do (#6411)James McCoy2017-04-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we get a mouse_on/mouse_off event, but the mouse is already in the corresponding state, there's no need to send the event up to the terminal. Closes #4394
* | | | | | | Merge #6422 from ZyX-I/fix-6420Justin M. Keyes2017-04-01
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | eval,fileio: Omit additional fsync() call
| * | | | | | | eval: Do not allocate FileDescriptorZyX2017-04-01
| | | | | | | |
| * | | | | | | eval,fileio: Omit additional fsync() callZyX2017-04-01
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Fixes #6420
* | | | | | | 'pastetoggle': support value >1 char (#6421)Matthew Malcomson2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we `set pastetoggle=abcde`, and manually type it, then `vgetorpeek()` sees part of the option before it has all been inserted into the typebuffer. To signify this it sets `keylen = KEYLEN_PART_KEY`, but the condition about whether to return the current key from `vgetorpeek()` only checks for `keylen = KEYLEN_PART_MAP`. Add a check for `KEYLEN_PART_KEY` to account for the `'pastetoggle'` option.
* | | | | | | vim-patch:7.4.2306lonerover2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Default value for 'langremap' is wrong. Solution: Set the right value. (Jürgen Krämer) Add a test. https://github.com/vim/vim/commit/da9ce2cde11ddd0e16cdfbab6d4ac4e8110218e1
* | | | | | | vim-patch:7.4.2236lonerover2017-04-01
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The 'langnoremap' option leads to double negatives. And it does not work for the last character of a mapping. Solution: Add 'langremap' with the opposite value. Keep 'langnoremap' for backwards compatibility. Make it work for the last character of a mapping. Make the test work. https://github.com/vim/vim/commit/920694c1b60fac8017b8909efcc24f189804a9bb
* | | | | | Merge pull request #6397 from jamessan/coverityJames McCoy2017-03-31
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix latest Coverity issues
| * | | | | | coverity/161216: Ensure buf is valid for lifetime of defstrJames McCoy2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Depending on the type of argument for input()/inputdialog()'s {text} argument, defstr may point to buf. Therefore it needs to be in scope for the lifetime of defstr. Also, use a different buffer for the handling of the 3rd argument to input()/inputdialog(). Although the buffer defstr points to is used immediately, it avoids potential mishaps if the code changes.
| * | | | | | coverity/16127: Verify lang is non-NULL before calling strlenJames McCoy2017-03-29
| | | | | | |
| * | | | | | ci: Update Coverity model for typval refactoringJames McCoy2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | coverity/161194: Restore check for 'keywordprg' being ":help"James McCoy2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 998d0ffc09d5c7358db62dc88c2e2b87622f60b5 removed the explicit check for ":help", relying instead on whether the user was in a help buffer. However, this breaks escaping the identifier for use in the lookup command. 2f54d6927cc02484b528a5e8b25b64c8d6580ddd tried to fix this by removing "!kp_ex" in "if (cmdchar == 'K' && !kp_ex)", but that causes shell escaping to be used instead of escaping for tag lookup.
| * | | | | | coverity/161195: Increase scope of exe_nameJames McCoy2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since exe_name is a stack allocated array, we need it to be in scope for the lifetime that vim_path points to it.
* | | | | | | Add handling for MSGPACK_OBJECT_FLOAT{32,64}James McCoy2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | msgpack-c previously only had MSGPACK_OBJECT_FLOAT, which was a 64-bit value. Now, 32-bit and 64-bit floats are supported as distinct types, but we'll simply continue to treat everything as 64-bit types.
* | | | | | | fold.c: more edge-cases when updating (#6207)Matthew Malcomson2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When foldUpdateIEMSRecurse() re-uses an existing fold, it misses the case where the existing fold spans from before startlnum to after firstlnum, the new fold does not span this range, and there is no "forced start" of a fold. We add a case for this in. Ensure that if there was no forced break in folds, we merge folds that now touch each other. Include testing for a tricky foldmethod=expr case that has never been a bug. This case works at the moment because of some effects that are not obvious when reading the code. A test for this could be useful to ensure a regression doesn't happen. vim-patch:8.0.0408
* | | | | | | vim-patch:7.4.2104 (#6332)Michael Ennen2017-03-31
| |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code duplication when unreferencing a function. Solution: De-duplicate. https://github.com/vim/vim/commit/97baee80f0906ee2f651ee1215ec033e84f866ad
* | | | | | api: {get,set}_option should {get,set} global value of local options (#6405)Jakob Schnitzer2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - nvim_get_option should return the global default of a local option. - nvim_set_option should set the global default of a local option.