aboutsummaryrefslogtreecommitdiff
path: root/src
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
| | | | | | | |
* | | | | | | | 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
* | | | | | | | | 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
| |_|/ / / / / / |/| | | | | | |
* | | | | | | | perf: tv_clear(): Cache gettext() result. (#6519)Justin M. Keyes2017-04-14
| | | | | | | | | | | | | | | | | | | | | | | | Closes #6437
* | | | | | | | 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: vim_strsave_shellescape: Handle 'shellslash'.Justin M. Keyes2017-04-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From Vim, misc2.c:vim_strsave_shellescape
* | | | | | | | win: defaults: 'shellredir', 'shellxquote', 'shellxescape'Justin M. Keyes2017-04-12
| | | | | | | |
* | | | | | | | refactor: pos_T macros to functions (#6496)Felipe Oliveira Carvalho2017-04-11
| | | | | | | |
* | | | | | | | win: mch_open_rw: specify S_IWRITE #6487erw72017-04-11
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Windows, `mch_open_rw` is not actually doing what it claims. This manifests as "E301: Oops, lost the swap file !!!" when filename is changed with :file {name}. Steps to reproduce (covered by test/functional/ex_cmds/file_spec.lua): nvim -u NONE :edit test :file test2 E301 Oops, lost the swap file!!! From libuv/src/win/fs.c: void fs__open(uv_fs_t* req) { ... attributes |= FILE_ATTRIBUTE_NORMAL; if (flags & _O_CREAT) { if (!((req->fs.info.mode & ~current_umask) & _S_IWRITE)) { attributes |= FILE_ATTRIBUTE_READONLY; } }
* | | | | | | Merge #6479 from bfredl/tolowerJustin M. Keyes2017-04-10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | remove vim_tolower/etc functions with broken locale-dependent behavior
| * | | | | | | lint: fix clint errors around mb_tolower callsBjörn Linse2017-04-10
| | | | | | | |
| * | | | | | | vim-patch:8.0.0552Björn Linse2017-04-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Toupper and tolower don't work properly for Turkish when 'casemap' is empty. (Bjorn Linse) Solution: Check the 'casemap' options when deciding how to upper/lower case. https://github.com/vim/vim/commit/3317d5ebbe8304da82b8088446060afcae0012af vim-patch:8.0.0553 Problem: Toupper/tolower test with Turkish locale fails on Mac. Solution: Skip the test on Mac. https://github.com/vim/vim/commit/9f4de1f5435b900e43e19766da1a5bed4686cf44 vim-patch:8.0.0554 Problem: Toupper and tolower don't work properly for Turkish when 'casemap' contains "keepascii". (Bjorn Linse) Solution: When 'casemap' contains "keepascii" use ASCII toupper/tolower. https://github.com/vim/vim/commit/1cc482069a3407132aeb43a55d6dc284153e79c7 vim-patch:8.0.0555 Problem: Toupper/tolower test fails on OSX without Darwin. Solution: Skip that part of the test also for OSX. (Kazunobu Kuriyama) https://github.com/vim/vim/commit/d2381a2cadb9ef359ad5efb916734c635b29bd13
| * | | | | | | vim-patch:8.0.0243Björn Linse2017-04-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When making a character lower case with tolower() changes the byte cound, it is not made lower case. Solution: Add strlow_save(). (Dominique Pelle, closes vim/vim#1406) https://github.com/vim/vim/commit/cc5b22b3bfdc0e9e835cf7871166badda31447bd Join almost identical strup_save and strlow_save functions to one Function.
| * | | | | | | mbyte: replace vim_tolower with mb_tolower handling locale correctlyBjörn Linse2017-04-10
| | | | | | | |
* | | | | | | | Merge #6488 from ZyX-I/coverity-fixesJustin M. Keyes2017-04-10
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | spellfile: Fix SAL sections readingZyX2017-04-09
| | | | | | | |
| * | | | | | | coverity/56795: Fix NULL dereference in :syn keyword non-printableZyX2017-04-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug was introduced 3 years earlier, in 13848aa: NULL keyword_copy was incorrectly treated as an indicator of OOM.
| * | | | | | | spellfile: Fix clint errorsZyX2017-04-09
| | | | | | | |