aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* unittests: Add tests for tv_clear()ZyX2017-03-29
|
* unittests: Add test for tv_dict_set_keys_readonlyZyX2017-03-29
|
* unittests: Add tv_dict_copy testsZyX2017-03-29
|
* charset: Remove useless condition from vim_iswordc_tabZyX2017-03-29
|
* unittests: Add tests for tv_dict_extendZyX2017-03-29
|
* eval: Fix extend() behaviour with NULL lists and dictionariesZyX2017-03-29
| | | | | Ref #4615 Ref vim/vim#768
* unittests: Add tv_dict_clear testsZyX2017-03-29
|
* eval: Use tv_is_func in place of ==VAR_FUNC||==VAR_PARTIALZyX2017-03-29
| | | | Also fixes same error as in vim/vim#1557
* unittests: Add tv_dict_add* unit testsZyX2017-03-29
| | | | | | | | Also fixes incorrect location of `tv_dict_add` function and three bugs in other functions: 1. `tv_dict_add_list` may free list it does not own (vim/vim#1555). 2. `tv_dict_add_dict` may free dictionary it does not own (vim/vim#1555). 3. `tv_dict_add_dict` ignores `key_len` argument.
* unittests: Add tests for dictionary indexingZyX2017-03-29
|
* eval/typval: Refactor errors a bit: use emsgf alwaysZyX2017-03-29
|
* unittests: Enable tv_list_join tests backZyX2017-03-29
| | | | Unable to reproduce the problem on Mac OS X Sierra VPS, need to check whether it is reproducible on travis.
* unittests: Disable tv_list_join test on Mac OS onlyZyX2017-03-29
|
* mbyte: Include os_defs.h in mbyte.hZyX2017-03-29
|
* eval: Do not use S_LEN as snprintf argumentZyX2017-03-29
|
* eval/typval: Fix SEGV in test_alot.vim testZyX2017-03-29
|
* eval: Fix SEGV in test49ZyX2017-03-29
|
* unittests: Add tv_dict_item_{add,remove} testsZyX2017-03-29
|
* unittests: Add tv_dict_item_{alloc,free} testsZyX2017-03-29
|
* eval: Fix len argument to xstrlcatZyX2017-03-29
|
* ascii: Readd DEL_STR defineZyX2017-03-29
|
* syntax: Fix linter errorZyX2017-03-29
|
* eval/typval: Fix -Werror=return-typeZyX2017-03-29
|
* unittests: Add tv_dict_watcher_{add,remove} testsZyX2017-03-29
|
* unittests: Run tv_list_join tests in case it stopped failingZyX2017-03-29
|
* eval: Fix linter errorsZyX2017-03-29
|
* *: Make some more things const and with lengthZyX2017-03-29
|
* eval: Make sure that v:_null_dict does not crash dictwatcher*()ZyX2017-03-29
| | | Ref #4615
* eval: Make dictionary watchers work with empty keysZyX2017-03-29
| | | | | | | | Looks like dict_notifications_spec test used to depend on some state which should not be preserved. Changed all `setup()` calls to `before_each()` and added necessary state in addition to changes required to test empty keys. Note: unit tests for tv_dict_watcher* are still needed.
* eval: Move part of dictwatcher* functions to eval/typvalZyX2017-03-29
|
* functests: Fix buf_functions test on WindowsZyX2017-03-29
|
* *: Fix some Windows-specific warningsZyX2017-03-29
| | | | Also fixed an error in path_fnamecmp().
* eval/typval: More `const` qualifiers in `tv_dict*` function signaturesZyX2017-03-29
|
* unittests: Add tests for tv_list_idx_of_itemZyX2017-03-29
|
* unittests: Add tests for tv_list_find*() functionsZyX2017-03-29
| | | | | | | | Additional modifications: - More `const` qualifiers in tested functions. - `tv_list_find_str()` second argument is more in-line with other `tv_list_find*()` functions.
* eval/typval: Add tv_list_equal() tests, compare NULL lists equalZyX2017-03-29
|
* unittests: Test tv_list_join()ZyX2017-03-29
|
* unittests: Fix tests crashZyX2017-03-29
| | | | | | | | | | | Tests crash at some point without - `after_each(collectgarbage)` right before “typval.c list copy() copies list correctly and converts items” test. - Commenting out that test. - Adding `collectgarbage()` after the test (what actually this commit does). Adding `collectgarbage()` to top-level `after_each` block right after `restore_allocators` makes running this file crash even if it is run alone.
* eval/typval,tests: Fix extending list with itself, add testsZyX2017-03-29
| | | | | Adds unit test for tv_list_extend and regression test for extend() VimL function.
* unittests: Test tv_list_concat()ZyX2017-03-29
|
* eval/typval: Make tv_list_concat handle NULL lists correctlyZyX2017-03-29
| | | | Fixes some FIXMEs in eval/null_spec.lua.
* functests: Add null_spec.lua from #4615ZyX2017-03-29
| | | | | | For now it is full of FIXMEs and tests for incorrect behaviour. Sorted out to have FIXMEs in one place, commented crashing tests in other and correctly working tests in the third one.
* unittests: Test tv_list_copyZyX2017-03-29
| | | | | | | | | | | | | Also found some bugs: 1. var_item_copy() always fails to copy v:_null_list and v:_null_dict. Fixing this should mean fixing `deepcopy(v:_null_list)` which should’ve been, but was not listed in #4615. This also fixes `deepcopy(v:_null_dict)`. 2. var_item_copy() crashes when trying to copy NULL string with `conv != NULL`. 3. `conv` argument is ignored when copying list unless `deep` is true, but it was not reflected in documentation. 4. `tv_dict_item_alloc_len()` allocated more memory then needed. 5. typvalt2lua was not able to handle self-referencing containers.
* unittests: Add tests for tv_list_insert*()/…append*() functionsZyX2017-03-29
|
* unittests: Add tests for tv_list_insert()ZyX2017-03-29
|
* unittests: Add tests for list watchers and list alloc/free/unrefZyX2017-03-29
|
* unittests: Test tv_list_item_\* functionsZyX2017-03-29
| | | | To check that memory is free()d correctly.
* eval/typval: Add missing includes, also add a script to find themZyX2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Contains unfinished attempt to integrate IWYU (ref #549). To finish it different job should be done, specifically: - Instead of feeding IWYU with modified file a mirror source tree should be created with the help of CMake which will contain modified sources. This solves the problem with IWYU thinking that `*.generated.h` headers should be included in place of `*` headers. - Build IWYU as all other third-party utilities. - Make modified sources avoid problems with `nvim/func_attr.h` includes and various related tricks. Current script may only be used for manual checks like this: ./scripts/check-includes.py \ --generated-includes-dir build/include \ --generated-includes-dir build/src/nvim/auto \ --file src/nvim/eval/typval.c \ -- -Isrc -Ibuild/include -Ibuild/src/nvim/auto \ -DINCLUDE_GENERATED_DECLARATIONS (it is also somewhat fine with `--file src/nvim/eval/typval.h`). I have no idea why (I mean, why developer think that these lines are needed, why they are suggested is pretty obvious: because there is typedef which mentions them before structs are defined), but for typval.h it reports, among other things, that it should add lines struct dictvar_S; struct listitem_S; struct listvar_S; struct listwatch_S;
* eval: Move copy_tv to eval/typvalZyX2017-03-29
|
* message: Remove incorrect assertionZyX2017-03-29
| | | | | | It was there only to prove that *now* `len` argument is not used to forbid putting message into history (i.e. that Neovim behaviour did not change). After this commit it should be possible to use msg_puts_attr_len with len>=0 to put message into history should new code need this.