aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * edit: Copy assert to before the warningZyX2017-04-16
| |
| * macros: Fix excessive checkZyX2017-04-16
| |
| * edit: Fix strange codeZyX2017-04-16
| | | | | | | | | | Based on the flow it looks like ptr could not be NULL here: if ptr_arg is NULL ptr is compl_leader, if compl_leader is NULL function exits. This also applies to Vim as far as I see.
| * digraph: Fix errors due to has_mbyte and friends being fixedZyX2017-04-16
| |
| * digraph: Ignore false positiveZyX2017-04-16
| | | | | | | | Reversed order is intentional, digraphs allow swapping characters.
| * diff: Silence -V519ZyX2017-04-16
| | | | | | | | Not exactly a false positive, but previous assignment is a part of the pattern “change global, run code which uses it, change global back”.
| * *: Silence some false positivesZyX2017-04-16
| |
* | Merge #6537 from justinmk/hlcursorJustin M. Keyes2017-04-18
|\ \ | | | | | | Cursor highlight after `:highlight clear|syntax reset`
| * | test: Cursor after `:hi clear|syntax reset`Justin M. Keyes2017-04-17
| | | | | | | | | | | | Also enable tests on Windows.
| * | highlight: default Cursor to guibg=fg, guifg=bgMatthieu Coudron2017-04-17
|/ / | | | | | | Closes #6508
* / fold: foldMoveRange(): fix :move bug #6534Matthew Malcomson2017-04-17
|/ | | | | | | | | | | | Closes #6540 In #6221 there was a mistake in calculating which folds need to be re-ordered. When there are no folds after those that have been adjusted, then `move_end` remains 0. This results in reverse_fold_order() swapping folds that have been adjusted with uninitialised folds in the remainder of the grow array. Add a check in foldMoveRange() to account for this case.
* Merge #6219 from jbradaric/vim-7.4.2170Justin M. Keyes2017-04-16
|\ | | | | vim-patch:7.4.{2170,2180,2240,2241,2242}
| * eval.c: Code style fixesJurica Bradaric2017-04-15
| |
| * test_timers.vim: Adjust timing to handle difference in implementationJames McCoy2017-04-15
| |
| * vim-patch:7.4.2359James McCoy2017-04-06
| | | | | | | | | | | | | | Problem: Memory leak in timer_start(). Solution: Check the right field to be NULL. https://github.com/vim/vim/commit/26fe0d56912e42c2b16a61b2480e19ba569aee98
| * vim-patch:7.4.2332James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | Problem: Crash when stop_timer() is called in a callback of a callback. Vim hangs when the timer callback uses too much time. Solution: Set tr_id to -1 when a timer is to be deleted. Don't keep calling callbacks forever. (Ozaki Kiichi) https://github.com/vim/vim/commit/75537a93e985ef32e6c267b06ce93629855dd983
| * vim-patch:7.4.2304James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | Problem: In a timer callback the timer itself can't be found or stopped. (Thinca) Solution: Do not remove the timer from the list, remember whether it was freed. https://github.com/vim/vim/commit/417ccd7138d4d230d328de8b0d3892dd82ff1bee
| * vim-patch:7.4.2281James McCoy2017-04-06
| | | | | | | | | | | | | | Problem: Timer test fails sometimes. Solution: Reduce minimum time by 1 msec. https://github.com/vim/vim/commit/0426bae2abede764d0dd366a28663d1c6e6ab0fe
| * Merge remote-tracking branch 'origin/master' into vim-7.4.2170James McCoy2017-04-05
| |\
| * | vim-patch:7.4.2242Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | Problem: Timer test sometimes fails. Solution: Increase the maximum time for callback timer test. https://github.com/vim/vim/commit/17f1347b867cbcc0ce380bf9a2466b4c31896f04
| * | vim-patch:7.4.2241Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | Problem: Timer test sometimes fails. Solution: Increase the maximum time for repeating timer. https://github.com/vim/vim/commit/973365dcc40a41e6b72ece56f15cebfee69b1329
| * | vim-patch:7.4.2240Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | Problem: Tests using the sleep time can be flaky. Solution: Use reltime() if available. (Partly by Shane Harper) https://github.com/vim/vim/commit/f267f8bdf777073e392ada5b31d837c7b6090eb4
| * | version.c: Mark 7.4.2171 and 7.4.2181 as NA.Jurica Bradaric2017-03-20
| | |
| * | vim-patch:7.4.2180Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | | | | Problem: There is no easy way to stop all timers. There is no way to temporary pause a timer. Solution: Add timer_stopall() and timer_pause(). https://github.com/vim/vim/commit/b73598e2f022a22fec512ea681c70d2775e8fd87
| * | vim-patch:7.4.2170Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | Problem: Cannot get information about timers. Solution: Add timer_info(). https://github.com/vim/vim/commit/8e97bd74b5377753597e3d98e7123d8985c7fffd
* | | Merge pull request #6493 from ZyX-I/pvs-scriptNikolai Aleksandrovich Pavlov2017-04-16
|\ \ \ | | | | | | | | [RFC] scripts: Create script which checks Neovim with PVS-studio
| * | | pvscheck: Add --recheck argumentZyX2017-04-10
| | | |
| * | | pvscheck: Do not use --depthZyX2017-04-10
| | | |
| * | | pvscheck: Do not trace helpZyX2017-04-10
| | | |
| * | | pvsscript: Use git clone and not git worktreeZyX2017-04-10
| | | |
| * | | pvscheck: Add helpZyX2017-04-10
| | | |
| * | | scripts: Create script which checks Neovim with PVS-studioZyX2017-04-10
| | | |
* | | | Merge #6528 from ZyX-I/revise-malloc-attrJustin M. Keyes2017-04-15
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Revise places where FUNC_ATTR_MALLOC is present Closes #6521
| * | | | os/shell: Remove FUNC_ATTR_MALLOC from shell_build_argvZyX2017-04-15
| | | | | | | | | | | | | | | Returns an array of allocated strings.
| * | | | ops: Remove FUNC_ATTR_MALLOC from copy_registerZyX2017-04-15
| | | | | | | | | | | | | | | Returned storage has a pointer to a newly allocated array.
| * | | | option: Remove FUNC_ATTR_MALLOC from get_winbuf_optionsZyX2017-04-15
| | | | | | | | | | | | | | | | | | | | Same as tv_dict_alloc() and additionally it saves some strings inside a dictionary.
| * | | | eval/typval: Remove FUNC_ATTR_MALLOC from tv_list_alloc_retZyX2017-04-15
| | | | | | | | | | | | | | | | | | | | Same as tv_list_alloc, but additionally ret_tv receives pointer to the newly allocated list.
| * | | | eval/typval: Remove FUNC_ATTR_MALLOC from tv_dict_allocZyX2017-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allocated dict points to previously allocated dict. Queue in allocated dict points to itself. Hashtab in allocated dict points to inside itself. Allocated dict is saved to gc_first_dict.
| * | | | eval/typval: Remove FUNC_ATTR_MALLOC from tv_dict_item_copyZyX2017-04-15
| | | | | | | | | | | | | | | Allocated storage may receive pointer to the list after tv_copy().
| * | | | eval/typval: Remove FUNC_ATTR_MALLOC from tv_list_allocZyX2017-04-15
| | | | | | | | | | | | | | | | | | | | Allocated list points to previously allocated list. Allocated list is saved to gc_first_list.
| * | | | os/fileio: Remove FUNC_ATTR_MALLOC for file_open_newZyX2017-04-15
|/ / / / | | | | | | | | fp contains pointer to rbuffer
* | | | test: make locale dependent oldtest more reliable (#6526)Björn Linse2017-04-15
| | | |
* | | | ops: fix i<c-r> with multi-byte text (#6524)Björn Linse2017-04-15
| | | |
* | | | test: check_cores(): Escape $TMPDIR path. (#6520)Justin M. Keyes2017-04-14
| | | | | | | | | | | | | | | | | | | | | | | | Lua string:match() considers hyphen to be a special char, we want the path to be a literal match. Also remove "./", etc. References #6483
* | | | perf: tv_clear(): Cache gettext() result. (#6519)Justin M. Keyes2017-04-14
| | | | | | | | | | | | Closes #6437
* | | | Merge #6497 from justinmk/win-quotJustin M. Keyes2017-04-12
|\ \ \ \ | | | | | | | | | | win: system('...'): special-case cmd.exe
| * | | | win: os_shell_is_cmdexe() + testsJustin M. Keyes2017-04-12
| | | | |
| * | | | win: default shellxescape, shellxquote to emptyRui Abreu Ferreira2017-04-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calling cmd.exe in Windows follows a very different pattern from Vim. The primary difference is that Vim does a nested call to cmd.exe, e.g. the following call in Vim system('echo a 2>&1') spawns the following processes "C:\Program Files (x86)\Vim\vim80\vimrun" -s C:\Windows\system32\cmd.exe /c (echo a 2^>^&1 ^>C:\Users\dummy\AppData\Local\Temp\VIoC169.tmp 2^>^&1) C:\Windows\system32\cmd.exe /c C:\Windows\system32\cmd.exe /c (echo a 2^>^&1 ^>C:\Users\dummy\AppData\Local\Temp\VIo3C6C.tmp 2^>^&1) C:\Windows\system32\cmd.exe /c (echo a 2>&1 >C:\Users\dummy\AppData\Local\Temp\VIo3C6C.tmp 2>&1) The escaping with ^ is needed because cmd.exe calls itself and needs to preserve the special metacharacters for the last call. However in nvim no nested call is made, system('') spawns a single cmd.exe process. Setting shellxescape to "" disables escaping with ^. The previous default for shellxquote=( wrapped any command in parenthesis, in Vim this is more meaningful due to the use of tempfiles to store the output and redirection (also see &shellquote). There is a slight benefit in having the default be empty because some expressions that run in console will not run within parens e.g. due to unbalanced double quotes system('echo "a b')
| * | | | win: libuv_process_spawn(): special-case cmd.exeRui Abreu Ferreira2017-04-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable CommandLineToArgvW-standard quoting for cmd.exe. libuv assumes spawned processes follow the convention expected by CommandLineToArgvW(). But cmd.exe is non-conformant, so for cmd.exe: - With system([]), the caller has full control (and responsibility) to quote arguments correctly. - With system(''), shell* options are used. libuv quoting is disabled if argv[0] is: - cmd.exe - cmd - $COMSPEC resolving to a path with filename cmd.exe Closes #6329 References #6387
| * | | | win/test: Enable more system() testsRui Abreu Ferreira2017-04-12
| | | | |