| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
|
| |
Problem: :cfile does not abort like other quickfix commands.
Solution: Abort when desired. Add tests for aborting. (Yegappan Lakshmanan,
closes vim/vim#5121)
https://github.com/vim/vim/commit/6a0cc916bd3cd6c2fd88b2972c92ade225603229
|
|
|
|
|
|
| |
Problem: Searchit() has too many arguments.
Solution: Move optional arguments to a struct. Add the "wrapped" argument.
https://github.com/vim/vim/commit/92ea26b925a0835badb0af2d5887238a4198cabb
|
|
|
|
|
| |
Problem: "%v" in 'errorformat' does handle multi-byte characters.
Solution: Handle multi-byte characters. (Yegappan Lakshmanan, closes vim/vim#3700)
https://github.com/vim/vim/commit/c45eb770a5988734ff2c572e5e2ce307158c33c8
|
|
|
|
|
|
| |
Problem: Checking for empty quickfix stack is not consistent.
Solution: Use qf_stack_empty(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/019dfe6855e011c02427bb922aafeae0245372c9
|
|
|
|
|
|
| |
Problem: The ex_make() function is too long.
Solution: Split it into several functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/b434ae2a1fcbbd43244c6130451de7f14346e224
|
|
|
|
|
|
| |
Problem: copy_loclist() is too long.
Solution: Split in multiple functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/09037503ea5f957ad23121bc61e15e4bb1765edf
|
|
|
|
|
|
| |
Problem: The ex_copen() function is too long.
Solution: Refactor to split off two functions. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/476c0db00205590974395df717519407a7717270
|
|
|
|
|
|
|
| |
Problem: Cannot get the window id associated with the location list.
Solution: Add the "filewinid" argument to getloclist(). (Yegappan
Lakshmanan, closes vim/vim#3202)
https://github.com/vim/vim/commit/c9cc9c78f21caba7ecb5c90403df5e19a57aa96a
|
|
|
|
|
|
| |
Problem: Quickfix code uses cmdidx too often.
Solution: Add is_loclist_cmd(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/396659592fe039decc8c088694912067fe32a681
|
|
|
|
|
|
|
| |
Problem: qf_init_ext() is too long.
Solution: Split it into multiple functions. (Yegappan Lakshmanan,
closes vim/vim#2939)
https://github.com/vim/vim/commit/6053f2d29a979ffed1fe01b0a2f28e23750530e9
|
|
|
|
|
|
| |
Problem: The qf_add_entries() function is too long.
Solution: Split in two parts. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/6f6ef7c1951b080843f3da049d3f5d0679de7348
|
|
|
|
|
|
| |
Problem: Helpgrep with language doesn't work properly. (Takuya Fujiwara)
Solution: Check for the language earlier. (Hirohito Higashi)
https://github.com/vim/vim/commit/c631f2df624954184509df49479d52ad7fe5233b
|
|
|
|
|
|
| |
Problem: Directly checking 'buftype' value.
Solution: Add the bt_normal() function. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/91335e5a67aaa9937e65f1e779b9f3f10fd33ee4
|
| |
|
|
|
|
|
|
| |
Problem: efm_to_regpat() is too long.
Solution: Split off three functions. (Yegappan Lakshmanan, closes vim/vim#2924)
https://github.com/vim/vim/commit/6bff719f7e472e918c60aa336de03e799b806c4f
|
|
|
|
|
| |
Problem: ex_helpgrep() is too long.
Solution: Refactor the function. (Yegappan Lakshmanan, closes vim/vim#2766)
https://github.com/vim/vim/commit/2225ebb48644f3924311b8df02a1319ab7675d42
|
|
|
|
|
| |
Problem: The qf_jump_to_usable_window() function is too long.
Solution: Split it in parts. (Yegappan Lakshmanan, closes vim/vim#2891)
https://github.com/vim/vim/commit/7a2b0e55e9460493c4a949bda8be70950dbb8f85
|
|
|
|
|
|
| |
Problem: Cursor moves to wrong column after quickfix jump.
Solution: Set the curswant flag. (Andy Massimino, closes vim/vim#3331)
https://github.com/vim/vim/commit/2dfcef4c08a3371e2126504bea00b274f937a840
|
|
|
|
|
|
| |
Problem: No good check if restoring quickfix list worked.
Solution: Let qf_restore_list() return OK/FAIL. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/90f1e2b7bcf56112e1535b693acf131727179a6e
|
|
|
|
|
|
| |
Problem: Coverity complains about a negative array index.
Solution: When qf_id2nr() cannot find the list then don't set qf_curlist.
https://github.com/vim/vim/commit/38efd1d17a6e6aa2add71efdf2cde4a788e5f5e5
|
|
|
|
|
|
| |
Problem: No test for fixed quickfix issue.
Solution: Add a test. Clean up the code a bit. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/3f347e4716c44cf6458be407e712e3d708d82580
|
|
|
|
|
|
| |
Problem: duplicated quickfix code.
Solution: Move the code to a function.
https://github.com/vim/vim/commit/8d8a65e389cef318ae661ff0fe7b1b00fd7cb25f
|
|
|
|
|
|
| |
Problem: Quickfix: mixup of FALSE and FAIL, returning -1.
Solution: Use FAIL and INVALID_QFIDX. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/29ce409bfca52bb8a07e2975d06fd788458e9861
|
| |
|
|
|
|
|
|
|
|
| |
Problem: Highlighting in quickfix window could be better. (Axel Bender)
Solution: Use the qfSeparator highlight item. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/93a32e2ec4b5a391c5006ca09f196e6510c8c2e9
This adds `syn_name2attr` already (from previous patch 8.0.1123,
vim/vim@1b9645de3).
|
|
|
|
|
|
|
| |
Problem: Superfluous quickfix code, missing examples.
Solution: Remove unneeded code. Add a few examples. Add a bit more
testing. (Yegappan Lakshmanan, closes vim/vim#2916)
https://github.com/vim/vim/commit/78ddc06bdd5c59cffdbb61eed7dcb5dcc4a17f19
|
|
|
|
|
|
|
| |
Problem: qf_set_properties() is to long.
Solution: Refactor the function. Define INVALID_QFIDX. (Yegappan
Lakshmanan, closes vim/vim#2812)
https://github.com/vim/vim/commit/a2aa8a2b22de909619d7faa3ff5383a6224defc5
|
|
|
|
| |
ll_get_or_alloc_list() never fails because OOM is an abort condition.
|
|
|
|
|
|
| |
Problem: :cepxr no longer jumps to the first error.
Solution: Use the quickfix list identifier. (Yegappan Lakshmanan)
Fixes https://github.com/neovim/neovim/issues/10895
|
|
|
|
| |
Also revert possibly-wrong change in f_sign_getplaced() from
652be3cb0040.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
Problem: Calling message_filtered() a bit too often.
Solution: Only call message_filtered() when filtering is already false.
https://github.com/vim/vim/commit/a9defadb8f03ecd03f3297305d5482ba380774dc
|
|
|
|
|
|
| |
Problem: Using dict_add_nr_str() is clumsy.
Solution: Split into two functions. (Ozaki Kiichi, closes vim/vim#3154)
https://github.com/vim/vim/commit/e0be167a805fd547c25ec1ec97fd4c7f13046236
|
|
|
|
|
|
| |
Problem: :clist output can be very long.
Solution: Support filtering :clist entries. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/4cde86c2ef885e82fff3d925dee9fb5671c025cf
|
|
|
|
|
|
|
| |
Problem: Crash when running quickfix tests.
Solution: Do not alloc a new location list when checking for the reference
to be still valid.
https://github.com/vim/vim/commit/d6b01a2d3884ee0abe10aad161f584889a496e3f
|
|
|
|
|
|
| |
Problem: Crash when autocommands call setloclist(). (Dominique Pelle)
Solution: If the quickfix list changes then don't jump to the error.
https://github.com/vim/vim/commit/0366c0161e988e32420d2f37111a60129684905b
|
|
|
|
|
|
|
| |
Problem: Get "no write since last change" error in terminal window.
Solution: Use another message when closing a terminal window. Make ":quit!"
also end the job.
https://github.com/vim/vim/commit/f5be7cd01642fafc4b7d68894eb60cca60c7a405
|
|
|
|
|
|
| |
Problem: CTRL-W CR does not work properly in a quickfix window.
Solution: Split the window if needed. (Jason Franklin)
https://github.com/vim/vim/commit/0a08c63da17dfd93ac2885e3f3f8a083a9b3131c
|
|
|
|
|
|
| |
Problem: Leaking memory when autocommands make a quickfix list invalid.
Solution: Call FreeWild(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/ee5b94a8ecbaeafa11c7c7c8cf7bef765fbea912
|
|
|
|
|
|
|
| |
Problem: Crash when autocommands delete the current buffer. (Dominique
Pelle)
Solution: Check that autocommands don't change the buffer.
https://github.com/vim/vim/commit/600323b4ef51a58a8e800d8ca469383a3c911db7
|
|
|
|
|
|
| |
Problem: :vimgrep may add entries to the wrong quickfix list.
Solution: Use the list identifier. (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/e1bb879f49665bb828197135b80aaf72cc190073
|
|
|
|
|
|
|
| |
Problem: Accessing freed memory in :lfile.
Solution: Get the current window after executing autocommands. (Yegappan
Lakshmanan, closes vim/vim#2473)
https://github.com/vim/vim/commit/14a4deb064610c30a50f00d524dde9b3292aad59
|
|
|
|
|
|
|
| |
Problem: Using free memory using setloclist(). (Dominique Pelle)
Solution: Mark location list context as still in use when needed. (Yegappan
Lakshmanan, closes vim/vim#2462)
https://github.com/vim/vim/commit/12237448499aaeb8c4f2be7a1deda60c0f160627
|
|
|
|
| |
Accidentally removed in 34f9e72af9c9.
ref #6201
|
|
|
|
|
|
|
|
| |
Problem: Clearing a pointer takes two lines.
Solution: Add VIM_CLEAR() and replace vim_clear(). (Hirohito Higashi,
closes #2629)
vim-patch:8.0.1481
|
|\ |
|
|/
|
|
|
|
| |
Problem: Checks for type of stack is cryptic.
Solution: Define IS_QF_STACK() and IS_LL_STACK(). (Yegappan Lakshmanan)
https://github.com/vim/vim/commit/4d77c65a9e57e1d1a459c44411930c7a892835fd
|
|
|
|
|
|
| |
Problem: Coverity warning for using uninitialized variable.
Solution: Clear exarg_T.
https://github.com/vim/vim/commit/4ca41534b726c4116d2e430e877e34146b4d4831
|