aboutsummaryrefslogtreecommitdiff
path: root/src/nvim
Commit message (Collapse)AuthorAge
...
| * | | | | | | coverity/16127: Verify lang is non-NULL before calling strlenJames McCoy2017-03-29
| | | | | | | |
| * | | | | | | coverity/161194: Restore check for 'keywordprg' being ":help"James McCoy2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 998d0ffc09d5c7358db62dc88c2e2b87622f60b5 removed the explicit check for ":help", relying instead on whether the user was in a help buffer. However, this breaks escaping the identifier for use in the lookup command. 2f54d6927cc02484b528a5e8b25b64c8d6580ddd tried to fix this by removing "!kp_ex" in "if (cmdchar == 'K' && !kp_ex)", but that causes shell escaping to be used instead of escaping for tag lookup.
| * | | | | | | coverity/161195: Increase scope of exe_nameJames McCoy2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since exe_name is a stack allocated array, we need it to be in scope for the lifetime that vim_path points to it.
* | | | | | | | Add handling for MSGPACK_OBJECT_FLOAT{32,64}James McCoy2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | msgpack-c previously only had MSGPACK_OBJECT_FLOAT, which was a 64-bit value. Now, 32-bit and 64-bit floats are supported as distinct types, but we'll simply continue to treat everything as 64-bit types.
* | | | | | | | fold.c: more edge-cases when updating (#6207)Matthew Malcomson2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When foldUpdateIEMSRecurse() re-uses an existing fold, it misses the case where the existing fold spans from before startlnum to after firstlnum, the new fold does not span this range, and there is no "forced start" of a fold. We add a case for this in. Ensure that if there was no forced break in folds, we merge folds that now touch each other. Include testing for a tricky foldmethod=expr case that has never been a bug. This case works at the moment because of some effects that are not obvious when reading the code. A test for this could be useful to ensure a regression doesn't happen. vim-patch:8.0.0408
* | | | | | | | vim-patch:7.4.2104 (#6332)Michael Ennen2017-03-31
| |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Code duplication when unreferencing a function. Solution: De-duplicate. https://github.com/vim/vim/commit/97baee80f0906ee2f651ee1215ec033e84f866ad
* | | | | | | api: {get,set}_option should {get,set} global value of local options (#6405)Jakob Schnitzer2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - nvim_get_option should return the global default of a local option. - nvim_set_option should set the global default of a local option.
* | | | | | | Remove PVS-Studio cruftJames McCoy2017-03-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | win: tempname(): Use $TMPDIR if defined.Justin M. Keyes2017-03-30
|/ / / / / /
* | | | | | refactor/single-include: buffer.h (#6396)relnod2017-03-30
| | | | | |
* | | | | | Merge #5119 from ZyX-I/split-evalJustin M. Keyes2017-03-29
|\ \ \ \ \ \
| * | | | | | eval/executor: Fix check-single-includesZyX2017-03-29
| | | | | | |
| * | | | | | *: Use const char * in set_one_cmd_contextZyX2017-03-29
| | | | | | | | | | | | | | | | | | | | | Also renames functions added in master and renamed here.
| * | | | | | eval/typval,api/buffer: Fix review commentsZyX2017-03-29
| | | | | | |
| * | | | | | eval/typval: Allow NULL dict as tv_dict_get_callback() argumentZyX2017-03-29
| | | | | | | | | | | | | | | | | | | | | Also removes NULL key input: tv_dict_find() does not allow this.
| * | | | | | eval: Fix -Werror=unitialized from QBZyX2017-03-29
| | | | | | |
| * | | | | | eval: Do not supply S_LEN to strncmpZyX2017-03-29
| | | | | | | | | | | | | | | | | | | | | It may be a macro as well.
| * | | | | | unittests: Add tv_get number testsZyX2017-03-29
| | | | | | |
| * | | | | | unittests: Test locks sectionZyX2017-03-29
| | | | | | |
| * | | | | | unittests: Add tests for tv_clear()ZyX2017-03-29
| | | | | | |
| * | | | | | charset: Remove useless condition from vim_iswordc_tabZyX2017-03-29
| | | | | | |
| * | | | | | eval: Fix extend() behaviour with NULL lists and dictionariesZyX2017-03-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ref #4615 Ref vim/vim#768
| * | | | | | 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
| | | | | | |
| * | | | | | 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
| | | | | | |
| * | | | | | 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
| | | | | | |
| * | | | | | 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
| | | | | | |
| * | | | | | *: 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
| | | | | | |
| * | | | | | 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.
| * | | | | | 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: 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;