aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | main: Flush file in place of closing it, also do error reportingZyX2017-07-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apparently on travis OS X systems it crashes when cleaning up streams with stdout closed: (lldb) bt all * thread #1: tid = 0x0000, 0x00007fff8703df06 libsystem_kernel.dylib`__pthread_kill + 10, stop reason = signal SIGSTOP * frame #0: 0x00007fff8703df06 libsystem_kernel.dylib`__pthread_kill + 10 frame #1: 0x00007fff93a764ec libsystem_pthread.dylib`pthread_kill + 90 frame #2: 0x00007fff97c056df libsystem_c.dylib`abort + 129 frame #3: 0x00007fff97bccdd8 libsystem_c.dylib`__assert_rtn + 321 frame #4: 0x0000000107a4e106 nvim`uv__close(fd=<unavailable>) + 102 at core.c:521 frame #5: 0x0000000107a5307d nvim`uv__loop_close(loop=0x00007fff5847c018) + 77 at loop.c:118 frame #6: 0x0000000107a4d149 nvim`uv_loop_close(loop=0x00007fff5847c018) + 57 at uv-common.c:626 frame #7: 0x000000010783e5bc nvim`stream_set_blocking(fd=0, blocking=true) + 204 at stream.c:34 frame #8: 0x000000010795d66b nvim`mch_exit(r=0) + 91 at os_unix.c:147 frame #9: 0x00000001078d5663 nvim`command_line_scan(parmp=0x00007fff5847c760) + 1779 at main.c:787 frame #10: 0x00000001078d4393 nvim`main(argc=2, argv=0x00007fff5847c898) + 163 at main.c:249 frame #11: 0x00007fff8cdd65ad libdyld.dylib`start + 1 frame #12: 0x00007fff8cdd65ad libdyld.dylib`start + 1
| * | terminal: Silence -V666 error: value not correspond with string lengthZyX2017-07-04
| | | | | | | | | | | | | | | Looks like calling this function below with 4-character first strings made PVS think that OPT_LOCAL (it is equal to 4) is a string length.
| * | syntax: Fix V763: parameter always rewritten before being usedZyX2017-07-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the result of malloc error handling elimination: push_current_state() used to (not) return OK depending on whether growing garray failed or not and this return was checked, if errorred out push_next_match() will simply return its argument unchanged. Now when allocations are supposed to either always succeed or crash Neovim this check was returned, push_current_state() was stripped of its return value and moved out of if() condition, resulting in V763.
| * | spell: Silence V512: buffer underflowZyX2017-07-04
| | |
| * | spell: Fix V728: excessive checkZyX2017-07-04
| | |
| * | regexp: Silence V595: potential null dereferenceZyX2017-07-04
| | | | | | | | | | | | The code uses 2-iteration loop antipattern: retval is NULL on first iteration, not NULL on second, yet this is still a false positive.
| * | normal: Fix V728: excessive checkZyX2017-07-04
| | |
| * | main: Use msgpack_file_write in place of “fbuffer”ZyX2017-07-04
| | | | | | | | | | | | Apparently the latter is not a part of the public C API.
| * | os: Add OS_STD*_FILENO constantsZyX2017-07-04
| | |
| * | os/fileio: Add ability to use os/fileio.c for file descriptorsZyX2017-07-04
| | | | | | | | | | | | Code imported from #6299
| * | os/fileio: Add msgpack_file_write functionZyX2017-07-04
| | |
| * | main: Fix V522: potential NULL dereferenceZyX2017-07-04
| | | | | | | | | | | | | | | It is useless to use sbuffer here and print that to stdout, just using “fbuffer” instead.
| * | ex_getln: Silent V519: value is assigned twice successivelyZyX2017-07-04
| | | | | | | | | This is usual “passing data via global” false positive.
| * | socket: Silence V641: buf size is not multiple of what it is cast toZyX2017-07-04
| | |
| * | charset: Fix V728: excessive checkZyX2017-07-04
| | |
* | | Merge #6958 from ZyX-I/fix-6957Justin M. Keyes2017-07-04
|\ \ \
| * | | shada: Fix linter errorZyX2017-07-04
| | | |
| * | | shada: Make sure that code does not attempt to read too long itemsZyX2017-07-04
| |/ / | | | | | | Fixes #6957
* | | Merge pull request #6961 from ZyX-I/pvscheck-ccNikolai Aleksandrovich Pavlov2017-07-04
|\ \ \ | | | | | | | | pvscheck: Add --environment-cc switch
| * | | pvscheck: Add --environment-cc switchZyX2017-07-04
|/ / / | | | | | | | | | | | | | | | To be used to make bot-ci able to use clang-4.0 without hacks. [ci skip]
* | | Merge #6955 'Fix invalid :echo output'Justin M. Keyes2017-07-04
|\ \ \ | |/ / |/| |
| * | functests: Ensure different SIDs on successive source() callsZyX2017-07-04
| | |
| * | functests: Add some more :echo tests which also check for regressionZyX2017-07-04
| | | | | | | | | | | | Fixes #6954
| * | functests: Make ex_cmds/echo actually use :echoZyX2017-07-04
| | |
| * | message: Fix `:echo "\x80"` printing `~@<80>`ZyX2017-07-04
| | |
| * | functests: Copy eval/string_spec.lua to ex_cmds/echo_spec.luaZyX2017-07-04
|/ /
* | Merge #6947 from ZyX-I/consistent-get_keymapJustin M. Keyes2017-07-03
|\ \
| * | unittests: Fix allocation ordering for tv_dict_add_str()ZyX2017-07-02
| | |
| * | functests: Test how spaces appear in get_keymap outputZyX2017-07-02
| | |
| * | getchar: Fix linter errorZyX2017-07-02
| | |
| * | functests: Use more extensive testingZyX2017-07-02
| | | | | | | | | | | | Fixes #6937
| * | message: Add support for replacing `<` to str2specialZyX2017-07-02
| | |
| * | functests: Add tests for new behaviourZyX2017-07-02
| | | | | | | | | Apparently it is not working yet.
| * | functests: Remove local_copy functionZyX2017-07-02
| | |
| * | eval: Make nvim_get_keymap output more robustZyX2017-07-02
| | |
| * | eval: Use tv_dict_add_allocated_str() for mapblock_fill_dictZyX2017-07-02
| | |
| * | eval/typval: Add tv_dict_add_allocated_str() functionZyX2017-07-02
| | |
| * | *: Adjust usages of modified functionsZyX2017-07-02
| | |
| * | message: Refactor str2specialbufZyX2017-07-02
| | | | | | | | | | | | Does not alter its usages.
| * | message: Refactor str2special_save and str2specialZyX2017-07-02
| | | | | | | | | | | | Does not alter their usages as well.
| * | mbyte: Refactor mb_unescapeZyX2017-07-02
| | | | | | | | | | | | Does not alter its usages.
* | | dict_get_value(): name the missing key (#6952)Matthieu Coudron2017-07-03
| | |
* | | Merge pull request #6946 from jamessan/fix-expand_env_esc-testJames McCoy2017-07-02
|\ \ \ | |/ / |/| | test: expand_env_esc: Pass correct buffer size for outlen and assertion
| * | test: expand_env_esc: Pass correct buffer size for outlen and assertionJames McCoy2017-07-02
|/ / | | | | | | | | | | | | | | Running this test with a mocked passwd file whose $HOME was set to /home/jamessan/src/debian.org/pkg-vim/deb-packages/neovim/neovim-0.2.0/debian/fakehome caused the test to fail, since the expanded result was >= 99 bytes. The test should be reflecting the actual size of the buffer, instead of some arbitrary other number, anwyay.
* | ci/quickbuild: XXX: disable server_requests test (#6851)Justin M. Keyes2017-07-02
| | | | | | | | | | | | | | | | Temporarily disable this test which hangs quickbuild. From #6905: The hang occurs when calling nvim_set_current_line. References #6594 5a151555c8dce70bbf235e7f6d5bd1ced5e7c46c
* | test: handle single-char hostname (#6939)KunMing Xie2017-07-01
| |
* | Merge pull request #6936 from jamessan/unicode-10James McCoy2017-06-30
|\ \ | | | | | | Update to Unicode 10
| * | scripts: Tell curl to follow redirectsJames McCoy2017-06-29
| | |
| * | Update unicode filesJames McCoy2017-06-29
| | |
| * | Update emoji-data URL for Unicode 10James McCoy2017-06-29
|/ / | | | | | | vim-patch:8.0.0652