aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | doc/provider: cleanupJustin M. Keyes2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Move design/impl discussion to develop.txt - Move clipboard notes to provider.txt
| * | | | | | doc: cleanupJustin M. Keyes2016-06-15
| | | | | | |
| * | | | | | doc/terminal: cleanupJustin M. Keyes2016-06-15
| | | | | | |
| * | | | | | doc/clipboard: cleanupJustin M. Keyes2016-06-15
| | | | | | |
| * | | | | | doc: Remove gui_w32.txtJustin M. Keyes2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's no longer relevant, except for 2 sections which are tracked as issues #4927 and #4928.
| * | | | | | doc: remove "nvim-" qualfier from tagsJustin M. Keyes2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is almost never necessary to qualify documentation as "Nvim specific". The main exception to this is vim_diff.txt.
| * | | | | | doc: msgpack_rpc: De-emphasize the transport protocol.Justin M. Keyes2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tighten our jargon, avoid mentioning "msgpack" everywhere we mention the RPC API. Prefer "RPC API" when speaking about the remote API. Though it's conceivable that we may one day support some protocol other than msgpack, that isn't relevant to most of our discussion of the API, including this document. The file name msgpack_rpc.txt is preserved to avoid totally breaking URLs.
| * | | | | | doc: Move spell design discussion.Justin M. Keyes2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is worth preserving, but let it live in spell.txt so we can dedicate develop.txt to nvim-specific discussion.
* | | | | | | Merge pull request #4841 from jamessan/vim-7.4.1126Justin M. Keyes2016-06-16
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | vim-patch:7.4.1126
| * | | | | | vim-patch:7.4.1126James McCoy2016-06-15
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Can only get the directory of the current window. Solution: Add window and tab arguments to getcwd() and haslocaldir(). (Thinca, Hirohito Higashi) https://github.com/vim/vim/commit/c970330676eaae7ba7cd05cfa46df5a413853ef9
* | | | | | Merge pull request #4924 from jamessan/test-cleanupJames McCoy2016-06-15
|\ \ \ \ \ \ | | | | | | | | | | | | | | Functional test cleanup
| * | | | | | test: functional: Remove unnecessary use of clipboard registerJames McCoy2016-06-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | menu_spec.lua yanks to the clipboard, but never pastes from it. This can leave a child xsel process waiting around for something to paste the content, causing the test process to hang. Since the test isn't explicitly trying to exercise the clipboard, simply use the default register.
| * | | | | | test: functional: Remove test_bkc_* files after testingJames McCoy2016-06-15
|/ / / / / /
* | | | | | Add new functionality to the `=` marker in the STLTJ DeVries2016-06-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new functionality is explained in the documentation. Also, many tests have been added to the buffer_spec.lua file
* | | | | | runtime: Add script for Python diagnostic information (#4885)Tommy Allen2016-06-13
| | | | | |
* | | | | | Merge #3745 from cacplate/ops_WconversionJustin M. Keyes2016-06-13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Enable -Wconversion in ops.c
| * | | | | | remove some asserts and lintCharles Joachim2016-05-30
| | | | | | |
| * | | | | | ops.c: enable -Wconversion warningCharles Joachim2016-05-30
| | | | | | |
* | | | | | | vim-patch:7.4.1352 (#4914)Michael Ennen2016-06-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The test script lists all functions before executing them. Solution: Only list the function currently being executed. https://github.com/vim/vim/commit/93bf558caef2d507ef6baf56eaf6025b63da1e34
* | | | | | | tui/input.c: Handle Ctrl-Z (suspend) for TERM=linux (#3100) (#4911)Mateusz Czapliński2016-06-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix #3100. On virtual consoles (Alt-F1, etc.), the Ctrl-Z combination was lost.
* | | | | | | 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
| | | | | | | | | | |