aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
| * | | | | | 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
|\ \ \ \ \ \
| * | | | | | 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
* | | | | | | | | | | 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
| | | | | | | | |
* | | | | | | | | 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
* | | | | | | | | | 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
* | | | | | | | | | 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
* | | | | | | | | Merge pull request #4879 from justinmk/test55Justin M. Keyes2016-06-05
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | tests: Migrate legacy test 55
| * | | | | | | | | test: Clean up after migration of legacy test 55.Lucas Hoffmann2016-05-23
| | |_|_|_|_|_|_|/ | |/| | | | | | |
* | | | | | | | | Merge pull request #2844 from lucc/test10Justin M. Keyes2016-06-05
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | tests: migrate legacy test 10
| * | | | | | | | | Remove old version of legacy test 10.Lucas Hoffmann2016-05-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These files where last changed in revision 7 (vim version 7.0001) in Vim's original mercurial repository but are not executed by the makefile.
| * | | | | | | | | tests: Migrate legacy test 10.Lucas Hoffmann2016-05-23
| |/ / / / / / / /
* | | | | | | | | Merge #4854Justin M. Keyes2016-06-05
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | log: Rename to $NVIM_LOG_FILEJustin M. Keyes2016-06-05
| | | | | | | | | |
| * | | | | | | | | log: Move log to $NVIM_LOG_FILE_PATH, falling back to the old locationZyX2016-05-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #3561
* | | | | | | | | | vim-patch:7.4.1056 (#4870)Shota2016-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Don't know why finding spell suggestions is slow. Solution: Add some code to gather profiling information. https://github.com/vim/vim/commit/ca1fe985175385c609f8e06672a1014729aba05c
* | | | | | | | | | pty_process: split into plat-specific files (#3976)Rui Abreu Ferreira2016-06-04
| |_|/ / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #4860 from jamessan/tab-win-precedenceJustin M. Keyes2016-06-04
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | tcd: Determine correct scope from user input
| * | | | | | | | tcd: Determine correct scope from user inputJames McCoy2016-06-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a user specifies both {window} and {tab}, `getcwd()`/`haslocaldir()` are using "tab" as the scope that should be reported. However, it should be using "window" as the scope, within the specified tab page.
| * | | | | | | | tcd: Use user-provided tab page for `getcwd()`/`haslocaldir()`James McCoy2016-06-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initial implementation for `:tcd` always used `curtab` to find the specified window. This would result in either inaccurate information or an unexpected error (e.g., when there are more windows in the user-specified tab page vs. the current tab page).
* | | | | | | | | tui/input.c: Add mouse release event (#4873)sach1t2016-06-04
| | | | | | | | |
* | | | | | | | | vim-patch:7.4.1108 (#4872)James McCoy2016-06-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Expanding "~" halfway a file name. Solution: Handle the file name as one name. (Marco Hinz) Add a test. Closes vim/vim#564. https://github.com/vim/vim/commit/58adb14739fa240ca6020cede9ab1f1cb07bd90a
* | | | | | | | | reorgJustin M. Keyes2016-06-03
| | | | | | | | |
* | | | | | | | | Make clint happy againJames McCoy2016-06-02
| | | | | | | | |