aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* *: Silence some false positivesZyX2017-04-16
|
* 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
| | | |
| * | | spellfile: Fix memory leakZyX2017-04-09
| | | |
| * | | coverity/13689: Check file header with memcmpZyX2017-04-09
| | | | | | | | | | | | | | | | Not that it is actually useful (would fail in any case), but should fix coverity report.
| * | | coverity/13688: Check for NUL bytes in salfromZyX2017-04-09
| | | |
| * | | coverity/13687: Do not allow NUL byte in region namesZyX2017-04-09
| | | |
| * | | spellfile: Use old errorZyX2017-04-09
| | | | | | | | | | | | This makes first test not actually show any change in behaviour.
| * | | coverity/13686: Do not allow NUL byte in precondition regexZyX2017-04-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this commit it emitted e_spell_trunc in the first case and treated file as completely valid on the second. While first is fine (both errors are actually valid, though old error is probably better), second results in incorrect regex used.
* | | | cmake: Do not forget to actually create a touch file for errors.tar.gzZyX2017-04-10
|/ / /
* | | Merge pull request #6485 from jamessan/vim-8.0.0377James McCoy2017-04-09
|\ \ \ | | | | | | | | vim-patch:8.0.0377,8.0.0378,8.0.0322,8.0.0376