aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | Merge pull request #5592 from jamessan/vim-7.4.1591James McCoy2016-11-11
|\ \ \ \ | |/ / / |/| | | vim-patch:7.4.1591
| * | | vim-patch:7.4.1591James McCoy2016-11-11
|/ / / | | | | | | | | | | | | | | | | | | Problem: The quickfix title is truncated. Solution: Save the command before it is truncated. (Anton Lindqvist) https://github.com/vim/vim/commit/5584df65a0ca2315d1eebc13c54a448bee4d0758
* | | Merge #5561 'inccommand'Justin M. Keyes2016-11-09
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Initial work by: Robin Elrharbi-Fleury (Robinhola) Audrey Rayé (Adrey06) Philémon Hullot (DesbyP) Aymeric Collange (aym7) Clément Guyomard (Clement0) Major revisions by: KillTheMule Björn Linse <bjorn.linse@gmail.com> Justin M. Keyes <justinkz@gmail.com>
| * | | 'inccommand': Preserve curbuf->b_u_newhead.Justin M. Keyes2016-11-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add tests for undotree(). Helped-by: Björn Linse <bjorn.linse@gmail.com> When "curhead" points to a valid head, the value of "newhead" is meaningless (and really should be set to null). The undo state for a buffer is _logically_ the enum: enum UndoState { CurrentHead(head), NewHead(head), EmptyTree } nvim _represents_ this as: whenever `curbuf->b_u_curhead` is nonnull it should be used as the current head, and `curbuf->b_u_newhead` is ignored. If the there is a current head, then this will be redoed on the next redo, and its parent will be undone on next undo. Only if `b_u_curhead` is NULL, `b_u_newhead` will be used as the head to undo (and it is not possible to redo). Also both can be NULL, to indicate an empty undotree. (To be fair, this only strictly true when calling undo.c from the outside, in some places _within_ a function in undo.c both values might be meaningful) Apparently `undotree()` breaks this non-abstraction, this _cosmetic_ issue can easily be fixed by `ex_substitute` also saving and restoring `b_u_newhead`, but is doesn't reflect any error really how `u_undo_and_forget` manipulates the _actual_ state of the undo tree.
| * | | perf: do_sub(): avoid work, avoid screen updatesJustin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | | | | | - Don't fill matched_lines if not showing a preview (!eap->is_live). - Encapsulate: Move cursor placement logic to show_sub().
| * | | 'inccommand': preserve 'modified'Justin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | | | | | During the live preview, the buffer-local 'modified' flag should not be changed.
| * | | 'inccommand': Do not trigger during scripts, feedkeys().Justin M. Keyes2016-11-08
| | | |
| * | | 'inccommand': test: scripts/feedkeys() should not trigger previewKillTheMule2016-11-08
| | | |
| * | | 'inccommand': rename 'incsubstitute'Justin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'inccommand' allows us to expand the feature to other commands, such as: :cdo :cfdo :global Also rename "IncSubstitute" highlight group to "Substitute".
| * | | 'inccommand': format line numbers as "|123| "Justin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | This matches what Quickfix traditionally does.
| * | | 'inccommand': set buffer name to [Preview]Justin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | | | | | | | | | [inc_sub] is less obvious for users. Also, in the future we may want to generalize the idea of a "preview buffer", or "incremental commands" besides :substitute.
| * | | 'inccommand': disable 'cursorline', 'spell' in previewJustin M. Keyes2016-11-08
| | | |
| * | | 'inccommand': preserve b:changedtickJustin M. Keyes2016-11-08
| | | |
| * | | lintJustin M. Keyes2016-11-08
| | | |
| * | | 'inccommand': reworkJustin M. Keyes2016-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Eliminate/isolate static/global variables - Remove special-case parameter from buflist_new() - Remove special-case ECMD_RESERVED_BUFNR - To determine when u_undo_and_forget() should be done, check b_changedtick instead of a heuristic. - use mb_string2cells() instead of strlen() to measure the :sub patterns - call ml_close() before buf_clear_file(). Avoids leaks caught by ASan. Original patch by: Robin Elrharbi-Fleury (Robinhola) Audrey Rayé (Adrey06) Philémon Hullot (DesbyP) Aymeric Collange (aym7) Clément Guyomard (Clement0)
| * | | LintedKillTheMule2016-10-30
| | | |
| * | | Tests for incsubstitution featureKillTheMule2016-10-30
| | | |
| * | | Incsubsitution featureKillTheMule2016-10-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally implemented by * Clement0 * DesbyP * aym7 * Adrey06 * Robinhola in #4811. Major reworkings and bug fixes by * bfredl Most tests suggested by ZyX-l, suggestions for improvements by oni-link.
* | | | CheckHealth: ignore non-existing markdownCodeBlock group (#5570)Marco Hinz2016-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | People using [1] would experience an error, because their markdown syntax doesn't define the markdownCodeBlock group. [1]: https://github.com/plasticboy/vim-markdown Closes #5569
* | | | CheckHealth: timeout system() calls (#5565)Tommy Allen2016-11-08
| | | |
* | | | Merge pull request #2905 from bfredl/utf8Björn Linse2016-11-05
|\ \ \ \ | | | | | | | | | | Only allow encoding=utf-8 and simplify multibyte code
| * | | | encoding: update documentationBjörn Linse2016-11-05
| | | | |
| * | | | encoding: update testsBjörn Linse2016-11-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change shada reencoding tests to check for correct handling of UTF-8 and binary strings. Delete enc=latin1 json tests.
| * | | | encoding: delete non-UTF-8 implementations of multibyte functionsBjörn Linse2016-11-05
| | | | | | | | | | | | | | | | | | | | Deleted documentation was duplicated at specific utf_ implementation
| * | | | encoding: cleanup mbyte.c given fixed encoding=utf-8Björn Linse2016-11-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Eliminate mb_init(): Set "enc_utf" and "has_mbyte" early. Eliminate "enc_unicode" and "enc_latin1like". init_chartab() and screenalloc() are already invoked elsewhere in the initialization process. The EncodingChanged autocmd cannot be triggered. At initialization, there is no spellfiles to reload
| * | | | encoding: only allow encoding=utf-8Björn Linse2016-11-05
|/ / / /
* | | | Health: introduce help links (#5557)Marco Hinz2016-11-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can now use help links like in normal help pages. The bars around them will be concealed as well. * Health: link string "SUCCESS" to ModeMsg So far we linked "SUCCESS" to the Function highlight group. The newly introduced healthHelp group links to Idenfifier as it does for links in normal help pages. Now the problem is that the Function group links again to Identifier as well, so both, "SUCCESS" and help links, would use the same colors. * Health: don't use Markdown code blocks Every line indented by more than 4 spaces would lose its highlighting otherwise. * Health: add helper health#help_to_link()
* | | | Merge pull request #5524 from brcolow/vim-7.4.1967James McCoy2016-11-02
|\ \ \ \ | | | | | | | | | | | | | | | vim-patch:7.4.1967
| * | | | vim-patch:7.4.1967Michael Ennen2016-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Falling back from NFA to old regexp engine does not work properly. (fritzophrenic) Solution: Do not restore nfa_match. (Christian Brabandt, closes vim/vim#867) https://github.com/vim/vim/commit/6747fabc7348bf5f41ccfe851e2be3e900ec8ee0 Helped-by: jamessan
* | | | | Merge pull request #5550 from jamessan/big-endian-fixesJames McCoy2016-11-02
|\ \ \ \ \ | | | | | | | | | | | | Making nvim big-endian friendly
| * | | | | lintJames McCoy2016-11-02
| | | | | |
| * | | | | nvim_win_get_number: Use int as rv to fix BE issuesJames McCoy2016-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rv is int64_t but its address is being passed into win_get_tabwin as if it were an int. This breaks on big-endian systems, since win_get_tabwin will store the data to the "wrong" half of the int64_t, thus returning invalid data out of nvim_win_get_number.
| * | | | | object_to_vim: Fix buffer/window/tabpage conversion on BE systemsJames McCoy2016-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since data.integer is a different (larger) integer type than data.{buffer,window,tabpage}, we cannot abuse the union by using data.integer to access the value for all 4 types. Instead, remove the {buffer,window,tabpage} members and always use the integer member. In order to accomodate this, perform distinct validation and coercion between the Integer type and Buffer/Window/Tabpage types in object_to_vim, msgpack_rpc helpers, and gendispatch.lua.
| * | | | | Use int as the standard type for boolean options.James McCoy2016-11-02
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All options are accessed by passing char_u pointers around, casting the pointer to the right pointer type for the specific option, and then dereferencing that pointer. This dance works fine on little-endian systems when some bool options are int types (as in Vim) and some are bool types (as would make more sense), but on big-endian systems *(int *)varp when varp is pointing to a bool will read random memory. Therefore, all boolean options must remain a consistent type and int is currently the easiest to choose.
* | | | | Merge #5542 from mhinz/health/clipboardJustin M. Keyes2016-11-02
|\ \ \ \ \ | | | | | | | | | | | | CheckHealth: add clipboard check
| * | | | | provider/clipboard.vim: never show a warning on sourcingMarco Hinz2016-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Never throw an error when provider/clipboard.vim is sourced for the first time. Save the error instead and expose it via `provider#clipboard#Error()`, mimicking provider/python.vim.
| * | | | | health/provider.vim: add clipboard checkMarco Hinz2016-11-02
| | | | | |
| * | | | | provider/clipboard.vim: refactorMarco Hinz2016-11-02
|/ / / / /
* | | | | tui/suspend_event(): set STDIN to "blocking"Justin M. Keyes2016-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set STDIN to blocking on "suspend". This was missed in #2598. See 8a782f1699e2a59a3f3e91f6d7c35a3312b82b41. Also on "continue"/SIGCONT explicitly set STDIN back to NON-blocking. We don't explicitly do that on *startup* (libuv does it internally), but on "continue" it seems wise to restore libuv's expectations. Closes #5459
* | | | | doc: Remove features from vim_diff which have been merged upstream (#5554)James McCoy2016-10-31
| | | | | | | | | | | | | | | | | | | | - TabNew/TabClosed: Introduced in 7.4.2075 and 7.4.2077 - hl-EndOfBuffer: Introduced in 7.4.2213
* | | | | Merge pull request #5527 from brcolow/vim-7.4.1730James McCoy2016-10-31
|\ \ \ \ \ | |_|_|/ / |/| | | | vim-patch:7.4.[1730,1734,1741,1742,1779,1782]
| * | | | vim-patch:7.4.1782Michael Ennen2016-10-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: strcharpart() does not work properly with some multi-byte characters. Solution: Use mb_cptr2len() instead of mb_char2len(). (Hirohito Higashi) https://github.com/vim/vim/commit/fca66003053f8c0da5161d1fe4b75b3a389934b5
| * | | | vim-patch:7.4.1779Michael Ennen2016-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Using negative index in strcharpart(). (Yegappan Lakshmanan) Solution: Assume single byte when using a negative iindex. https://github.com/vim/vim/commit/73dfe917ba6357413aaf98a021c91add5ac6e9bc
| * | | | vim-patch:7.4.1742Michael Ennen2016-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: strgetchar() does not work correctly. Solution: use mb_cptr2len(). Add a test. (Naruhiko Nishino) https://github.com/vim/vim/commit/5d18e0eca59ffbba22c7f7c91c9f99d672095728
| * | | | vim-patch:7.4.1741Michael Ennen2016-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Not testing utf-8 characters. Solution: Move the right asserts to the test_expr_utf8 test. https://github.com/vim/vim/commit/b22bd46b9681d73d095f2eadff8163d3a6cf416b
| * | | | vim-patch:7.4.1734Michael Ennen2016-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test fails when not using utf-8. Solution: Split test in regularand utf-8 part. https://github.com/vim/vim/commit/0f518a8f4d4be4cac10389680f6bd5e3781f94b0
| * | | | vim-patch:7.4.1730Michael Ennen2016-10-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: It is not easy to get a character out of a string. Solution: Add strgetchar() and strcharpart(). https://github.com/vim/vim/commit/58de0e2dcc1f2d251b74892a06d71a14973f3187
* | | | | Merge #5546 from justinmk/global-ctrlcJustin M. Keyes2016-10-30
|\ \ \ \ \ | |_|_|/ / |/| | | | ex_global: Catch CTRL-C even if it is mapped.
| * | | | lintJustin M. Keyes2016-10-30
| | | | |
| * | | | ex_global: Catch CTRL-C even if it is mapped.Justin M. Keyes2016-10-30
|/ / / / | | | | | | | | | | | | | | | | Problem: If CTRL-C is mapped, it does not interrupt :global output. Solution: clear `mapped_ctrl_c` during :global.
generated by cgit (git 2.34.1) at 2025-07-18 21:07:58 +0000