aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | | test: Use clipboard fixture instead of user's clipboard (#4903)James McCoy2016-06-11
| |_|/ / / / |/| | | | |
* | | | | | Merge #4839 from ZyX-I/luaviml'/lua'/encode_vim_to_object'/functestsJustin M. Keyes2016-06-11
|\ \ \ \ \ \ | | | | | | | | | | | | | | Check sanitizer results right after the test
| * | | | | | functests: Do not use setup/teardown where before_/after_each is neededZyX2016-06-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When skipping these test blocks they may error out: Error → test/functional/shell/viml_system_spec.lua @ 154 system() with output containing NULs setup ./test/functional/helpers.lua:75: attempt to index upvalue 'session' (a nil value) stack traceback: ./test/functional/helpers.lua:75: in function 'request' ./test/functional/helpers.lua:166: in function 'nvim_feed' ./test/functional/helpers.lua:195: in function 'feed' test/functional/shell/viml_system_spec.lua:14: in function <test/functional/shell/viml_system_spec.lua:13> Error → test/functional/shell/viml_system_spec.lua @ 155 system() with output containing NULs teardown ./test/functional/helpers.lua:75: attempt to index upvalue 'session' (a nil value) stack traceback: ./test/functional/helpers.lua:75: in function 'eval' test/functional/shell/viml_system_spec.lua:21: in function <test/functional/shell/viml_system_spec.lua:20>
| * | | | | | functests: Check logs in lua codeZyX2016-06-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is otherwise impossible to determine which test failed sanitizer/valgrind check. test/functional/helpers.lua module return was changed so that tests which do not provide after_each function to get new check will automatically fail.
* | | | | | | Merge #4908 from ZyX-I/clint-checks-2Justin M. Keyes2016-06-11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add more clint checks
| * | | | | | | clint: Add rules that forbids spaces after a cast operatorZyX2016-06-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://neovim.io/develop/style-guide.xml#Horizontal_Whitespace Note that this errors out for e.g. if (has_mbyte) mb_copy_char((const char_u **)(&f), &t); \ (found in macros.h:151) or #define ECMD_SET_HELP 0x02 /* set b_help flag of (new) buffer before (found in ex_cmds.h:11) (note `(new) buffer`). I left this as-is because these pieces of code are already caught by another rule (braces near if and `/*`-style comments). Other false positives I found were: 1. `FUNC_ATTR_NONNULL_ARG(1) FUNC_ATTR_WARN_UNUSED_RESULT`: rejected by requiring type name to start with `[a-zA-Z_]` (this makes `1` not be incorrectly recognized as a type). 2. `#define FOO(var) (var)`: rejected by creating `cast_line`. 3. `(expr) * (expr)`: rejected by constructing regexp so that unary operators require no spaces after them. 4. `int f(void) FUNC_ATTR_PURE`: rejected by requiring line to start with a space. 5. `"." STR(NVIM_VERSION_PATCH) NVIM_VERSION_PRERELEASE`: not rejected, such thing should be rare and it would be easy to get rid of the false positive by e.g. breaking line. Struct literals like `(MyStruct) { 4, 2 }` are not checked: 1. I am not sure whether they are under this rule at all (this is not a cast). 2. It would be hard to get rid of false positives (like the example with `if` above, but now for *valid* `if() {}`s).
| * | | | | | | func_attr: Fix other new linter errorsZyX2016-06-11
| | | | | | | |
| * | | | | | | *: Fix new linter errorsZyX2016-06-11
| | | | | | | |
| * | | | | | | clint: Reject indented preprocessor directivesZyX2016-06-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is also not allowed by the style guide: https://neovim.io/develop/style-guide.xml#Preprocessor_Directives.
| * | | | | | | *: Also fix the adjacent errorsZyX2016-06-11
| | | | | | | |
| * | | | | | | *: Fix errors from new linter checksZyX2016-06-11
| | | | | | | |
| * | | | | | | clint: Do not allow aligning line continuation charactersZyX2016-06-11
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rejected by https://neovim.io/develop/style-guide.xml#Horizontal_Whitespace. Note: what to do with “empty” lines is not described in the style guide, neither it is checked.
* / / / / / / Remove some unnecessary function attributes (#4909)oni-link2016-06-11
|/ / / / / / | | | | | | | | | | | | | | | | | | This removes attribute FUNC_ATTR_NONNULL_ALL for functions without a pointer parameter.
* | | | | | Merge #4813 'runtime: clipboard: start daemons in /'.Justin M. Keyes2016-06-10
|\ \ \ \ \ \
| * | | | | | runtime: clipboard: start daemons in /Aleksa Sarai2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids the issue of nvim started daemons causing mountpoints to be unmountable. This is currently the only place in runtime/ where this calling convention occurred. Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
| * | | | | | doc: update docs to reflect new cwd optionAleksa Sarai2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The termopen() docs don't need to be updated because they link to jobstart(). Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
| * | | | | | test: add tests for cwd handlingAleksa Sarai2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add both a test for cwd=/ and cwd=/tmp/nvim.XXXXX, to make sure that we don't have regressions in cwd handling. Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
| * | | | | | eval: allow setting cwd in {jobstart,termopen}()Aleksa Sarai2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Processes in vim are always started in the current directory, which causes issues when the process is a daemon and the current directory is a mountpoint. Fix this by adding an option to set the cwd of the new process with jobstart(). In addition, fix termopen() so that it actually uses the cwd option from the dict (it couldn't previously set the cwd value due to dead code). Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
* | | | | | | Merge pull request #4875 from shotat/vim-7.4.1130James McCoy2016-06-09
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:7.4.1130
| * | | | | | | vim-patch:7.4.1130shotat2016-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Memory leak in :vimgrep. Solution: Call FreeWild(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/61ff4dd6a4d47bd32383fe28087be2b37dec53f4 mark 1130 as NA
* | | | | | | | Merge pull request #4802 from brcolow/vim-7.4.1090James McCoy2016-06-09
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | vim-patch:7.4.{1090,1094}
| * | | | | | | | vim-patch:7.4.1094Michael Ennen2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test for :hardcopy fails on MS-Windows. Solution: Check for the +postscript feature. https://github.com/vim/vim/commit/ccb80989f2779c8441f7f15d160fb2141bd1676d
| * | | | | | | | vim-patch:7.4.1090Michael Ennen2016-05-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No tests for :hardcopy and related options. Solution: Add test_hardcopy. https://github.com/vim/vim/commit/b5690794cf081773628fa0f1f2b948fd129d5b39
* | | | | | | | | Merge pull request #4788 from brcolow/vim-7.4.1051James McCoy2016-06-09
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | vim-patch:7.4.{1051,1068}
| * | | | | | | | | vim-patch:7.4.1068Michael Ennen2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Wrong way to check for unletting internal variables. Solution: Use a better way. (Olaf Dabrunz) https://github.com/vim/vim/commit/71bcfdf30109c3d6e40d143adcaf33964b18a70b
| * | | | | | | | | vim-patch:7.4.1051Michael Ennen2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Segfault when unletting "count". Solution: Check for readonly and locked first. (Dominique Pelle) Add a test. https://github.com/vim/vim/commit/af8af8bfac5792fa64efbc524032d568cc7754f7
* | | | | | | | | | Merge pull request #4738 from brcolow/vim-7.4.1223James McCoy2016-06-09
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | vim-patch:7.4.1223
| * | | | | | | | | | vim-patch:7.4.1223Michael Ennen2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when setting v:errors to a number. Solution: Free the typval without assuming its type. (Yasuhiro Matsumoto) https://github.com/vim/vim/commit/a542c680a8b42cb766e64d4ee7374ef4dacb7832
* | | | | | | | | | | Merge pull request #4758 from jbradaric/vim-7.4.1394James McCoy2016-06-09
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | vim-patch:7.4.1394,7.4.1397,7.4.1464,7.4.1468
| * | | | | | | | | | eval.c: Fix linter errors.Jurica Bradaric2016-06-09
| | | | | | | | | | |
| * | | | | | | | | | vim-patch:7.4.1468Jurica Bradaric2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Sort test doesn't test with "1" argument. Solution: Also test ignore-case sorting. (Yasuhiro Matsumoto) https://github.com/vim/vim/commit/51d1d536802b5d8232d47e56f165ba8a009529b5
| * | | | | | | | | | vim-patch:7.4.1464Jurica Bradaric2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: When the argument of sort() is zero or empty it fails. Solution: Make zero work as documented. (suggested by Yasuhiro Matsumoto) https://github.com/vim/vim/commit/5131c144feb046c5e2b72e6c172159d80ce06b3c
| * | | | | | | | | | vim-patch:7.4.1397Jurica Bradaric2016-06-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Sort test fails on MS-Windows. Solution: Correct the compare function. https://github.com/vim/vim/commit/0bb6108eb4e1ecaed437bc507f514f5da7816d9e
| * | | | | | | | | | vim-patch:7.4.1394Jurica Bradaric2016-06-09
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Can't sort inside a sort function. Solution: Use a struct to store the sort parameters. (Jacob Niehus) https://github.com/vim/vim/commit/0b962473ddc7cee3cb45253dea273573bcca9bf9
* | | | | | | | | | Fix style according to linterTJ DeVries2016-06-09
| | | | | | | | | |
* | | | | | | | | | vim-patch:7.4.1153Michael Ennen2016-06-09
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Autocommands triggered by quickfix cannot always get the current title value. Solution: Call qf_fill_buffer() later. (Christian Brabandt) https://github.com/vim/vim/commit/6920c72d4d62c8dc5596e9f392e38204f561d7af Helped by @mhinz
* | | | | | | | | tests: Migrate legacy test 34. (#2849)Lucas Hoffmann2016-06-08
| | | | | | | | |
* | | | | | | | | test: Fix path to valgrind suppressions (#4892)James McCoy2016-06-08
| | | | | | | | |
* | | | | | | | | Merge pull request #4804 from brcolow/vim-7.4.1150Justin M. Keyes2016-06-08
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | vim-patch:7.4.{1150,1151}
| * | | | | | | | | vim-patch:7.4.1151Michael Ennen2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Missing change to eval.c Solution: Also change feedkeys(). https://github.com/vim/vim/commit/5f8a14b9dea094b8bbab94cfc1e8da8e633fbc01
| * | | | | | | | | vim-patch:7.4.1150Michael Ennen2016-06-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: 'langmap' applies to the first character typed in Select mode. (David Watson) Solution: Check for SELECTMODE. (Christian Brabandt, closes #572) Add the 'x' flag to feedkeys(). https://github.com/vim/vim/commit/25281634cda03ce302aaf9f906a9520b5f81f91e
* | | | | | | | | | Merge #4890 from jamessan/vim-38a5563Justin M. Keyes2016-06-07
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | vim-patch:38a5563,e0fa374,decb14d
| * | | | | | | | | | vim-patch:decb14dJames McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update channel.txt https://github.com/vim/vim/commit/decb14d68c3e3736566466aed2190f1d1cab587a NA since it's related to Channels
| * | | | | | | | | | vim-patch:e0fa374James McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/e0fa3742ead676a3074a10edadbc955e1a89153d Ignore changes to * doc/channel.txt: Channel related docs * doc/netbeans.txt, doc/os_dos.txt, doc/todo.txt: Not relevant to Neovim * doc/tags: Generated at build time
| * | | | | | | | | | vim-patch:38a5563James McCoy2016-06-07
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update runtime files. https://github.com/vim/vim/commit/38a55639d603823efcf2d2fdf542dbffdeb60b75 Ignore changes to * doc/channel.txt, doc/eval.txt, doc/various.txt: Channel related docs * doc/tags: Generated at build time * doc/todo.txt, doc/vi_diff.txt: Not relevant to neovim
* | | | | | | | | | Merge #4888 from jamessan/vim-13d5aeeJustin M. Keyes2016-06-07
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | vim-patch:13d5aee,705ada1,298b440,5e9b2fa,7c764f7,681baaf,cbebd48
| * | | | | | | | | | vim-patch:cbebd48James McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files. https://github.com/vim/vim/commit/cbebd4879cc78e670d79b2c57dc33d7b911c962a Ignore changes to * doc/channel.txt, doc/eval.txt: Channel related docs * doc/tags: Generated at build time * doc/todo.txt: Not relevant to Neovim * ftplugin/man.vim: Change already present in autoload/man.vim
| * | | | | | | | | | vim-patch:681baafJames McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update runtime files. https://github.com/vim/vim/commit/681baaf4a4c81418693dcafb81421a8614832e91 Ignore changes to * doc/channel.txt, doc/eval.txt, doc/usr_41.txt: Channel related docs * doc/tags: Generated at build time * doc/todo.txt: Not relevant to Neovim
| * | | | | | | | | | vim-patch:7c764f7James McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the python script executable. https://github.com/vim/vim/commit/7c764f7bbf71a7a49baae07641efd2ead69e4d08
| * | | | | | | | | | vim-patch:5e9b2faJames McCoy2016-06-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated runtime files and translations. https://github.com/vim/vim/commit/5e9b2fa9bb0e6061cf18457c173cd141a5dc9c92 Ignore changes to * doc/tags: generated at build time * doc/develop.txt, doc/todo.txt, doc/netbeans.txt, doc/vim-ja.UTF-8.1, doc/xxd-ja.UTF-8.1, lang/menu_*: Not applicable to Neovim * doc/editing.txt: Crypt related * doc/change.txt, doc/insert.txt, doc/various.txt: Removal of ex_extra tags, which already happened in Neovim * doc/vim-ja.UTF-8.1, doc/xxd-ja.UTF-8.1
generated by cgit (git 2.34.1) at 2025-05-21 12:20:53 +0000