aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
| * | | | | | | | | MSVC: Avoid variadic macro bug in STATIC_ASSERTRui Abreu Ferreira2016-04-30
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MSVC does not handle __VA_ARGS__ as expected in STATIC_ASSERT, avoid its use to work around it since we don't need it. The underlying issue seems to be one of https://connect.microsoft.com/VisualStudio/Feedback/Details/1232378 https://connect.microsoft.com/VisualStudio/Feedback/Details/1099052 The bug only seems to manifest when using multiple variadic macros that call each other.
* / / / / / / / / normal: convert MCHAR etc operator and register types to enum MotionTypeBjörn Linse2016-05-01
|/ / / / / / / /
* | | | | | | | Merge pull request #4676 from ZyX-I/fix-hist_char2type-crashJustin M. Keyes2016-04-30
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | ex_getln: Do not crash with :append/:insert/:change
| * | | | | | | | ex_getln: Do not crash with :append/:insert/:changeZyX2016-04-29
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change effectively disables history for lines inserted using this method. Not a big problem since it does not work for them in Vim in first place. Also solves a bug(?): ex_window() run while in :append mode opens search history in Vim for some reason. Now it opens empty cmdline window.
* | | | | | | | Merge pull request #4679 from equalsraf/tb-windows-lcJustin M. Keyes2016-04-29
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Windows: without libintl use LC_CTYPE instead of LC_MESSAGES
| * | | | | | | | Windows: without libintl use LC_CTYPE instead of LC_MESSAGESRui Abreu Ferreira2016-04-29
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If libintl is not available, LC_MESSAGES is not defined. For now fallback to using LC_CTYPE. Neovim and Vim have diverged significantly in ex_cmds2.c concerning this logic. In other locations the fallback is actually LC_COLLATE, but in this case Vim calls get_mess_env() (which in turn falls back to LC_CTYPE). In Neovim get_mess_env() is only available with libint. This means we are not completely consistent with Vim when handling LC_ environment variables and do not build against libintl.
* | | | | | | | Merge pull request #4675 from barraponto/patch-1Justin M. Keyes2016-04-29
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Increase readability of unit conversion.
| * | | | | | | Increase readability of unit conversion.Capi Etheriel2016-04-29
| | | | | | | |
* | | | | | | | Add missing include fcntl.hRui Abreu Ferreira2016-04-29
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | In Windows, open() flags like O_RDONLY need fcntl.h.
* | | | | | | os_nodetype: impl with libuvJustin M. Keyes2016-04-29
| | | | | | |
* | | | | | | Remove old UNIX ifdef from buf_write()Rui Abreu Ferreira2016-04-28
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When backupcopy=auto buf_write assumes backupcopy=yes when the file is a hard/symbolic link. However this check was guarded by a UNIX ifdef. The check itself is portable and the guard can be removed. Added a couple tests to check the behaviour of bkc=auto and bkc=no with a symbolic link. Reported in #4525
* | | | | | option.c: include header for completeopt_was_setJustin M. Keyes2016-04-28
| | | | | |
* | | | | | Merge pull request #4661 from brcolow/vim-7.4.1401Justin M. Keyes2016-04-27
|\ \ \ \ \ \ | | | | | | | | | | | | | | vim-patch: 7.4.1401
| * | | | | | vim-patch: 7.4.1401Michael Ennen2016-04-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Having 'autochdir' set during startup and using diff mode doesn't work. (Axel Bender) Solution: Don't use 'autochdir' while still starting up. (Christian Brabandt) https://github.com/vim/vim/commit/6bd364e08461159ad3c153ffba4def5b896486a1
* | | | | | | Merge pull request #4652 from HiPhish/coverity-defectsJustin M. Keyes2016-04-27
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix coverity errors in `haslocaldir()` and `getcwd()`.
| * | | | | | | Fix coverity errors in haslocaldir() and getcwd.HiPhish2016-04-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Vim function `haslocaldir()` would crash if the users called it with the two arguments `-1, -1`. Now it returns `0` in that case. The coverity issue was complaining about a NULL dereference, but there can never be a case where the pointer `tp` is NULL and being dereferenced. An assertion has been put in place to satisfy coverity. Furthermore the functions themselves have been cleaned up. First of all the documentation comment for the different scopes has been extended and a macro for the minimum scope has been introduced. In both functions any time a scope is used as a range (e.g. in a loop) macros instead of actuals scopes are used, that makes the functions more robust if new scopes are added. Second, in the implementation of `getcwd()` there was a superfluous loop, it has been removed completely. I also changed all `goto end` to plaing `return` statements by moving the allocation of `cwd` down, that way there is no need for `goto` anymore.
* | | | | | | | Merge pull request #4625 from brcolow/vim-7.4.1075Justin M. Keyes2016-04-27
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | vim-patch: 7.4.1075
| * | | | | | | | vim-patch: 7.4.1075Michael Ennen2016-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when using an invalid command. Solution: Fix generating the error message. (Dominique Pelle) https://github.com/vim/vim/commit/05fe017c1ac0503b706dad695097572fde01ab0b
* | | | | | | | | Merge pull request #4655 from brcolow/vim-7.4.1035Justin M. Keyes2016-04-27
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | vim-patch: 7.4.1035
| * | | | | | | | | vim-patch: 7.4.1035Michael Ennen2016-04-26
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: An Ex range gets adjusted for folded lines even when the range is not using line numbers. Solution: Only adjust line numbers for folding. (Christian Brabandt) https://github.com/vim/vim/commit/a3306958dcb9aadff1e1e8521d908d86b10ac99a
* | | | | | | | | vim-patch:7.4.1296KillTheMule2016-04-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cursor changes column with up motion when the matchparen plugin saves and restores the cursor position. (Martin Kunev) Solution: Make sure curswant is updated before invoking the autocommand. https://github.com/vim/vim/commit/f068dcafcfe0c8018e5a559c50769ca1364bd9a5 Applied manually. Could reproduce the bug with vim/vim@4d8747c but not with vim/vim@2693ca2, so it must have appeared inbetween. For discussion, see https://groups.google.com/forum/#!msg/vim_dev/t2sdeFhkybs/WEtLJpCODQAJ http://vim.1045645.n5.nabble.com/Cursor-behaviour-change-td5726895.html Could not reproduce with current nvim master.
* | | | | | | | | Merge pull request #4622 from Shougo/vim-7.4.1753Justin M. Keyes2016-04-27
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | vim-patch:7.4.1753
| * | | | | | | | vim-patch:7.4.1753Shougo Matsushita2016-04-22
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: "noinsert" in 'completeopt' is sometimes ignored. Solution: Set the variables when the 'completeopt' was set. (Ozaki Kiichi) https://github.com/vim/vim/commit/c020042083b9c0a4e932b562c3bef97c76328e18
* | | | | | | | vim-patch:7.4.1006 #4605Michael Ennen2016-04-27
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The fix in patch 7.3.192 is not tested. Solution: Add a test, one for each regexp engine. (Elias Diem) https://github.com/vim/vim/commit/96c664af27ec9535f2c3cd9b889faad3e9460ad6 The patch was applied to 044_099_regexp_multibyte_magic_spec.lua as these two legacy tests (44 and 99) were merged together (and de-duplicated) in Neovim.
* | | | | | | Merge pull request #4649 from justinmk/vimpatchesJustin M. Keyes2016-04-27
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:7.4.1092
| * | | | | | | remove disable_char_avail_for_testing()Justin M. Keyes2016-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_cursor_func.vim hangs at the call to disable_char_avail_for_testing(). The test does not actually need this function (and it correctly fails if the fix from 7.4.1300 is reverted). Given that disable_char_avail_for_testing is a gigantic hack, if we can avoid it let's do so.
| * | | | | | | vim-patch:7.4.1118Justin M. Keyes2016-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Tests hang in 24 line terminal. Solution: Set the 'more' option off. https://github.com/vim/vim/commit/a99b90437af730dcafd9143c0942c87777a00d52
| * | | | | | | vim-patch:7.4.1092Justin M. Keyes2016-04-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: It is not simple to test for an exception and give a proper error message. Solution: Add assert_exception(). https://github.com/vim/vim/commit/a803c7f94070f94b831fdfd1984f288c8b825b5d
| * | | | | | | legacy test: MakefileJustin M. Keyes2016-04-25
| | | | | | | |
| * | | | | | | Satisfy the linter.KillTheMule2016-04-25
| | | | | | | |
| * | | | | | | vim-patch:7.4.1300KillTheMule2016-04-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot test CursorMovedI because there is typeahead. Solution: Add disable_char_avail_for_testing(). https://github.com/vim/vim/commit/2ab375e54ef4eac438d1aef8b99d9e71f2fa0c63 Most of it manually applied.
| * | | | | | | vim-patch:5a46a58KillTheMule2016-04-25
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing test file. https://github.com/vim/vim/commit/5a46a58eb6e50cb5204909cc2202e3400761263f
* | | | | | | Merge pull request #4630 from jamessan/vim-7.4.613Justin M. Keyes2016-04-26
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:7.4.613
| * | | | | | | regexp_nfa.c: Fix various linter errorsJames McCoy2016-04-25
| | | | | | | |
| * | | | | | | vim-patch:7.4.613James McCoy2016-04-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: The NFA engine does not implement the 'redrawtime' time limit. Solution: Implement the time limit. https://github.com/vim/vim/commit/70781ee4035b5fd5e3cbb3fe4c7646e19119f0a8
* | | | | | | | Merge pull request #4593 from ZyX-I/length-functionsJustin M. Keyes2016-04-26
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Make some function accept strings with length in place of just strings
| * | | | | | | *: Fix linter errorsZyX2016-04-18
| | | | | | | |
| * | | | | | | ex_getln: Make get_histtype return HIST_DEFAULT if neededZyX2016-04-18
| | | | | | | |
| * | | | | | | ex_getln: Make get_histtype accept length argumentZyX2016-04-18
| | | | | | | |
| * | | | | | | option: Allow zero-length optionsZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code that expected NUL-terminated strings allowed them and this behaviour is actually used.
| * | | | | | | option: Use findoption_len in do_setZyX2016-04-18
| | | | | | | |
| * | | | | | | option: Add find_key_option_len functionZyX2016-04-18
| | | | | | | |
| * | | | | | | option: Add findoption_len functionZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is like findoption(), but works with non-NUL-terminated strings.
| * | | | | | | keymap: Make replace_termcodes and friends accept length and cpo_flagsZyX2016-04-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reasons: - One does not have to do `s[len] = NUL` to work with these functions if they do not need to replace the whole string: thus `s` may be const. - One does not have to save/restore p_cpo to work with them.
* | | | | | | | Merge pull request #4647 from justinmk/vimpatchesJustin M. Keyes2016-04-26
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | version.c: update NA; vim-patch:7.4.998
| * | | | | | | | vim-patch:7.4.998Justin M. Keyes2016-04-25
| | | | | | | | |
| * | | | | | | | version.c: update NAKillTheMule2016-04-25
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7.4.797: - Modified function redraw_asap was removed in https://github.com/neovim/neovim/commit/e0e41b30c61922e099a067ac5c137e745699a1aa, together with its only caller check_termcode. - Grepping for the following regexps did not yield any result in nvim/src and its subdirectories: rows.*\*.*Columns (lines modified often), ScreenLinesC\[r\] (one of the sources of the bug), msg_scrolled followed by grepping for NORMAL (another line that was changed for the bug) - The out-of-bound access was on the array *screenlineC[MAX_MCO] (see the diff). I grepped for MAX_MCO an checked every array of that length for out-of-bounds access in its scope (I did not check for called functions, e.g. utfc_ptr2char(p, u8cc) where u8cc was of length MAX_MCO). I did not find any. - The code for drawing on the screen was moved to the TUI, which was newly written. 7.4.733: - test_listchars was converted to a lua test in https://github.com/neovim/neovim/pull/2492 - Since no file is sourced anymore, the behavior of the lua test should not depend on the setting of ff
* | | | | | | | Merge pull request #4631 from KillTheMule/vim-7.4.871Justin M. Keyes2016-04-25
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | vim-patch:7.4.871
| * | | | | | | Happy little, happy little, happy little linterKillTheMule2016-04-22
| | | | | | | |
| * | | | | | | vim-patch:7.4.871KillTheMule2016-04-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Vim leaks memory, when 'wildignore' filters out all matches. Solution: Free the files array when it becomes empty. https://github.com/vim/vim/commit/7b256fe7445b46929f660ea74e9090418f857696 The only nontrivial part of 7.4.871 missing (renamings of variables are in another commit; freeing *files after 0 matches was already there, just FAIL was not returned in that case)