aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | | | | | 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
| | | | | | | | | |
| * | | | | | | | | 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
| | | | | | | | |
* | | | | | | | | Merge pull request #6495 from bfredl/localefixBjörn Linse2017-04-11
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | run Turkish locale tests on travis and make the tests more reliable
| * | | | | | | | | ci: install Turkish locale and make locale tests more reliableBjörn Linse2017-04-11
|/ / / / / / / / /
* | | | | | | | | Merge #6490 from justinmk/testJustin M. Keyes2017-04-11
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | Closes #6487
| * | | | | | | | ci/appveyor: fix cache patternJustin M. Keyes2017-04-11
| | | | | | | | |
| * | | | | | | | test: helpers.execute() => helpers.feed_command()Justin M. Keyes2017-04-11
| | | | | | | | |
| * | | | | | | | win/test: Enable recover_spec.luaJustin M. Keyes2017-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; } }
| * | | | | | | | test: `:file {name}`Justin M. Keyes2017-04-11
| | | | | | | | |
| * | | | | | | | test/rmdir(): fallback to Nvim delete()Justin M. Keyes2017-04-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lua has too many pitfalls here: - os.execute() requires shell-escaping - os.execute() has breaking changes between Lua 5.1 and 5.2 - No native way in Lua to handle "readonly" etc. on Windows
| * | | | | | | | test/rmdir(): Remove `readonly` attr on Windows.Justin M. Keyes2017-04-10
|/ / / / / / / /
* | | | | | | | Merge #6489 from ZyX-I/own-output-handlerJustin M. Keyes2017-04-10
|\ \ \ \ \ \ \ \
| * | | | | | | | appveyor: Widen .deps cache dependencyZyX2017-04-10
| | | | | | | | |
| * | | | | | | | ci: Make appveyor use new output handler as wellZyX2017-04-10
| | | | | | | | |
| * | | | | | | | tests: Rename neovim output handler to nvimZyX2017-04-10
| | | | | | | | |
| * | | | | | | | third-party: Update busted versionZyX2017-04-10
| | | | | | | | |
| * | | | | | | | tests: Skip dumping elapsed times when not availableZyX2017-04-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Should provide some compatibility with old busted. And also removes duplicate parts from successString and skippedString, making them more like failureString and errorString which do not have times for technical reasons (busted for some reason did not yet compute duration before running the relevant handlers).
| * | | | | | | | tests: Print description also at the end of the suiteZyX2017-04-10
| | | | | | | | |
| * | | | | | | | tests: Use our own output handler on travisZyX2017-04-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reasoning: 1. gtest is better then something like utfTerminal, yet it is way too verbose. 2. gtest cannot be configured to show colors always. 3. Actually I am going to add a CMake target which will allow running tests (especially, functional tests) in parallel, but this is not going to work well with any of the default output handlers. Build in this case must be more or less silent, yet debuggable. New handler does not support this in this commit though.
* | | | | | | | | 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.
| * | | | | | | | | test: add tests for gu/gU behavior in Turkish localeBjörn Linse2017-04-10
| | | | | | | | | |
| * | | | | | | | | 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
| | | | | | | | |