aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | | | | | | eval/encode: Remove unneeded variable, add missing includeZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/encode: Free memory just in caseZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After string_convert() with .vc_fail=true these blocks should never be entered because they indicate invalid unicode.
* | | | | | | | eval/encode: Reduce length of encode_list_writeZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | Changes suggested by oni-link.
* | | | | | | | eval/encode: Fix invalid UTF-8 strings handling:ZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Do not allow reading past buffer end when creating error messages. 2. Fix surrogate pairs range, avoid magic constants.
* | | | | | | | eval/encode: Fix writing strings starting with NL to listZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error [found][1] by oni-link. [1]: https://github.com/neovim/neovim/pull/4131/files#r52239384
* | | | | | | | eval: Fix QuickBuild failuresZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compiler used by one VM in QuickBuild has found a number of false positives. Everything is fine on travis. List of failures: From [QuickBuild][1], build [7429][2]: 14:38:19,945 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c: In function ‘assert_bool’: 14:38:19,945 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c:7551:40: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] 14:38:20,058 WARN - cc1: all warnings being treated as errors . This is not making much sense (7551:40 is `!=` in `{SpecialVarValue} != ({bool}?{SpecialVarValue}:{SpecialVarValue})`), but this error is present. --- Also fail from [build][3] [4930][4]: 15:47:00,853 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval/encode.c: In function ‘encode_read_from_list’: 15:47:00,853 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval/encode.c:258:30: error: conversion to ‘char’ from ‘int’ may alter its value [-Werror=conversion] , pointing to `:` in `{char} = ({char} == {const} ? {const} : {char})` where `{const}` is character constant like `'\n'`. I have no idea where exactly it saw conversion, so simply casted everything to (char). --- [Build][5] error: 08:32:03,472 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c: In function ‘tv_equal’: 08:32:03,472 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c:5077:1: error: control reaches end of non-void function [-Werror=return-type] --- Build [4949][7]: 11:28:00,578 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c: In function ‘f_type’: 11:28:00,578 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c:16085:24: error: ‘n’ may be used uninitialized in this function [-Werror=uninitialized] 11:28:00,581 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c: In function ‘f_empty’: 11:28:00,581 WARN - /home/quickbuild/buildagent/workspace/root/neovim/pull-requests-automated/src/nvim/eval.c:8505:24: error: ‘n’ may be used uninitialized in this function [-Werror=uninitialized] [1]: http://neovim-qb.szakmeister.net/wicket/page?5-1.ILinkListener-content-buildTab-panel-masterStep-body-children-0-step-body-children-2-body-children-3-step-body-children-0-step-body-children-0-step-head-logLink [2]: http://neovim-qb.szakmeister.net/build/4929 [3]: http://neovim-qb.szakmeister.net/build/4930 [4]: http://neovim-qb.szakmeister.net/wicket/page?1-1.ILinkListener-content-buildTab-panel-masterStep-body-children-0-step-body-children-1-body-children-3-step-body-children-0-step-body-children-0-step-head-logLink [5]: http://neovim-qb.szakmeister.net/build/4948/step_status [7]: http://neovim-qb.szakmeister.net/build/4949
* | | | | | | | version: Record that vim-7.4.1271 is kinda of thereZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/encode: Fix non-utf-8 &encoding handling, add testsZyX2016-04-18
| | | | | | | |
* | | | | | | | *: Fix memory leaks found by clang sanitizerZyX2016-04-18
| | | | | | | |
* | | | | | | | shada: Make sure that NIL and EXT values can also be parsed backZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note: currently they are both *dumped*, but parsing them produces an error. This is inappropriate: variables should either be skipped with error message when dumping or should be read back properly. It also appears that I did not have test for “has wrong variable value type” error, so nothing got removed from errors_spec.
* | | | | | | | eval: Port parts of 7.4.1267 that are not already presentZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Make assert_true and assert_false accept v:true and v:falseZyX2016-04-18
| | | | | | | |
* | | | | | | | *: Fix gcc warningsZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Change dv_lock type to VarLockStatusZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Fix vim_str2nr invocationZyX2016-04-18
| | | | | | | |
* | | | | | | | *: Fix linter errorsZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Remove v:noneZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To get v:none back just rever this commit. This will not make json*() functions compatible with Vim though.
* | | | | | | | eval: Remove get_vim_var_tv functionZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Add support for special mapsZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Special dictionaries representing map are created when encountering duplicate key or when key is empty or contains NUL. Also checks that values are separated by a comma/colon properly.
* | | | | | | | eval/decode: Parse strings with NUL to special dictionariesZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/*: Fix some linter errorsZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Fix surrogate pairs processingZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Fail on control and invalid unicode charactersZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Make msgpackparse() function use new v: varsZyX2016-04-18
| | | | | | | |
* | | | | | | | eval/decode: Replace INIT_SPECIAL_DICT macros with inline functionZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Move some decoding functions to eval/decode.cZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Get rid of VV_LEN constantZyX2016-04-18
| | | | | | | |
* | | | | | | | vim: Move *MSG* macros to message.hZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | Note: OUT* macros were removed because they are no longer used anywhere.
* | | | | | | | eval: Move encode.c to eval/encode.cZyX2016-04-18
| | | | | | | |
* | | | | | | | cmake: Refactor code that creates directories and lists C filesZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Add jsondecode() functionZyX2016-04-18
| | | | | | | |
* | | | | | | | encode: Do not use default case for v_typeZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Add special variables v:false, v:null, v:noneZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Add special variable typeZyX2016-04-18
| | | | | | | |
* | | | | | | | encode: Add a hint for static analyzer that cur_tv is not NULLZyX2016-04-18
| | | | | | | |
* | | | | | | | eval: Add jsonencode() functionZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | Ref #3471
* | | | | | | | eval: Split out typval_T dumping functions to nvim/encode.cZyX2016-04-18
| |/ / / / / / |/| | | | | |
* | | | | | | Merge pull request #4565 from KillTheMule/vim-patch-1282Justin M. Keyes2016-04-17
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | vim-patch:7.4.{1000, 1001, 1030, 1282, 1284}
| * | | | | | Add new style test test_viml.vim to the buildKillTheMule2016-04-15
| | | | | | |
| * | | | | | Satisfy the linterKillTheMule2016-04-15
| | | | | | |
| * | | | | | Update version.c to 7.4.1284KillTheMule2016-04-15
| | | | | | |
| * | | | | | vim-patch:7.4.1284KillTheMule2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test 49 fails. Solution: Check for a different error message. https://github.com/vim/vim/commit/dc94a26a641914df5f1ba1ab47a5752e1137287e Applied cleanly except for version.c.
| * | | | | | vim-patch:7.4.1030KillTheMule2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: test49 is still slow. Solution: Move more tests from old to new style. https://github.com/vim/vim/commit/a2cce8630756769b2cefdc28c7290ae9262cddb1 Applied cleanly except for version.c.
| * | | | | | vim-patch:7.4.1001KillTheMule2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: test_viml isn't run. Solution: Include change in makefile. https://github.com/vim/vim/commit/f49e240c2def978247fa457aa105bb3024413f7d Applied cleanly except for version.c. Needs to be added to the build system.
| * | | | | | vim-patch:7.4.1000KillTheMule2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test 49 is slow and doesn't work on MS-Windows. Solution: Start moving parts of test 49 to test_viml. https://github.com/vim/vim/commit/c06624661a3aa6642304c06db9cebe553a4cab17 Applied cleanly except Makefile changes. File src/nvim/testdir/runtest.vim was configured to set nocp for all tests. This option is not in nvim anymore, so it was removed.
| * | | | | | vim-patch:7.4.1282KillTheMule2016-04-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when evaluating the pattern of ":catch" causes an error. (Dominique Pelle) Solution: Block error messages at this point. https://github.com/vim/vim/commit/768ce2435ae956041579ef2d26e3e9d3a2444e1e Applied manually. Could not reproduce the crash both using vim and nvim, therefore could not device a test. Should be merged anyways, since it's in vim?
* | | | | | | Merge pull request #4585 from dbarnett/vim-fc39ecJustin M. Keyes2016-04-17
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:fc39ec
| * | | | | | | vim-patch:fc39ecDavid Barnett2016-04-16
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update runtime files. https://github.com/vim/vim/commit/fc39ecf8ded5466d7e9cbde8db75517718b023d8
* | | | | | | eval.c: Fix heap corruption error when constructing sourcing_nameoni-link2016-04-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A wrong format specifier and not enough allocated memory for sourcing_name could lead to a heap corruption. Original patch by Rui Abreu Ferreira (@equalsraf) Fixes #4582
* | | | | | | Merge pull request #4574 from oni-link/fix.utfc_ptr2char_lenJustin M. Keyes2016-04-17
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | mbyte.c: Fix invalid memory access in utfc_ptr2char_len
generated by cgit (git 2.34.1) at 2025-07-13 19:05:15 +0000