aboutsummaryrefslogtreecommitdiff
path: root/src/nvim/quickfix.c
Commit message (Collapse)AuthorAge
...
* vim-patch:8.0.1031: "text" argument for getqflist() is confusingJan Edmund Lazo2018-09-08
| | | | | | Problem: "text" argument for getqflist() is confusing. (Lcd47) Solution: Use "lines" instead. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/2c809b7c7d2bb5e4b7fd09c3d312cadecf0c1ff0
* vim-patch:8.0.1029: return value of getqflist() is inconsistentJan Edmund Lazo2018-09-08
| | | | | | Problem: Return value of getqflist() is inconsistent. (Lcd47) Solution: Always return an "items" entry. https://github.com/vim/vim/commit/da73253a0b908bad03ddcd625fe3fb32008efbf6
* vim-patch:8.0.1023: it is not easy to identify a quickfix listJan Edmund Lazo2018-09-08
| | | | | | Problem: It is not easy to identify a quickfix list. Solution: Add the "id" field. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/a539f4f1ae4a2b3a7dfce89cd3800214c9e990cf
* vim-patch:8.0.1006: quickfix list changes when parsing text with 'erroformat'Jan Edmund Lazo2018-09-07
| | | | | | | Problem: Cannot parse text with 'erroformat' without changing a quickfix list. Solution: Add the "text" argument to getqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/7adf06f4e25c795ba32ff0b2e8591330f6a41afb
* vim-patch:8.0.0922: quickfix list always added after current oneJan Edmund Lazo2018-09-07
| | | | | | | Problem: Quickfix list always added after current one. Solution: Make it possible to add a quickfix list after the last one. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/55b6926450d75788dada3ff44a35e328224df758
* vim-patch:8.0.0904: cannot set a location list from textJan Edmund Lazo2018-09-06
| | | | | | Problem: Cannot set a location list from text. Solution: Add the "text" argument to setqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/ae338338508ef42866204f90dca861ac555f4298
* lintJan Edmund Lazo2018-09-06
|
* window: refactor boolean variables in win_close()Jan Edmund Lazo2018-09-06
| | | | free_buf (param) and help_window (variable) are bool.
* vim-patch:8.0.0782: using freed memory in quickfix codeJan Edmund Lazo2018-09-06
| | | | | | Problem: Using freed memory in quickfix code. (Dominique Pelle) Solution: Handle a help window differently. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/d28cc3f55d4a5a980f6ac6fa682382822a223720
* vim-patch:8.0.0733: can only add entries to one list in the quickfix stackJan Edmund Lazo2018-09-05
| | | | | | | Problem: Can only add entries to one list in the quickfix stack. Solution: Move state variables from qf_list_T to qf_list_T. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/a7df8c70c85c793bc4d75abc625d36883ab029cc
* vim-patch:8.0.1413: accessing freed memory in :cbufferJan Edmund Lazo2018-08-15
| | | | | | Problem: Accessing freed memory in :cbuffer. Solution: Get quickfix list after executing autocmds. (closes vim/vim#2470) https://github.com/vim/vim/commit/aaf6e43b7a99cedb89d73ba749a46f7a0f16bbb6
* vim-patch:8.0.0735: no indication that the quickfix window/buffer changedJan Edmund Lazo2018-07-28
| | | | | | | | Problem: There is no way to notice that the quickfix window contents has changed. Solution: Increment b:changedtick when updating the quickfix window. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/a8788f4d0b991f466b607c2c5bc6fd600bc78a97
* vim-patch:8.0.0687: minor issues related to quickfixJan Edmund Lazo2018-07-28
| | | | | | | | | Problem: Minor issues related to quickfix. Solution: Set the proper return status for all cases in setqflist() and at test cases for this. Move the "adding" flag outside of FEAT_WINDOWS. Minor update to the setqflist() help text. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/86f100dc0922e83bead7bcd5fd2bb2abbf153f46
* vim-patch:8.0.1622: possible NULL pointer dereferenceJan Edmund Lazo2018-07-23
| | | | | | Problem: Possible NULL pointer dereferencey. (Coverity) Solution: Reverse the check for a NULL pointer. https://github.com/vim/vim/commit/6ed86ad170b60517eeddb54c2b22fdc888a22c0b
* vim-patch:8.0.0677: setting 'filetype' may switch buffersJan Edmund Lazo2018-06-26
| | | | | | | Problem: Setting 'filetype' internally may cause the current buffer and window to change unexpectedly. Solution: Set curbuf_lock. (closes vim/vim#1734) https://github.com/vim/vim/commit/1814183b865059679f6ee526ec23fc575e536e66
* vim-patch:8.0.0466: still macros that should be all-caps (#8510)Jan Edmund Lazo2018-06-10
| | | | | Problem: There are still a few macros that should be all-caps. Solution: Make a few more macros all-caps. https://github.com/vim/vim/commit/8820b48654b62472821d9b155fe03ab7ac13a05c
* Merge #8218 'Fix errors reported by PVS'Justin M. Keyes2018-04-27
|\ | | | | closes #4983
| * *: Fix clint errorsZyX2018-04-22
| |
| * quickfix: Silence PVS/V560: memory is referenced and may be changedZyX2018-04-22
| |
| * quickfix: Fix PVS/V547: allocator never returns NULL nowZyX2018-04-22
| |
| * quickfix: Fix PVS/V547,V560: no conversion with fixed &encodingZyX2018-04-22
| |
| * quickfix: Fix PVS/V560: remove duplicate conditionZyX2018-04-02
| | | | | | Condition was checked in surrounding if().
* | shada: Save numbered marksZyX2018-03-26
|/ | | | | | | Problems so far: - Marks in the current instance are not adjusted. - Duplicates are not removed (not that it works in Vim either now, not at 8.0.134 at least).
* refactor/rename: path_try_shorten_fname()Justin M. Keyes2018-03-24
|
* vim-patch:8.0.1403: using freed buffer in grep commandJustin M. Keyes2018-02-11
| | | | | | | Problem: Using freed buffer in grep command. (gy741, Dominique Pelle) Solution: Lock the dummy buffer to avoid autocommands wiping it out. https://github.com/vim/vim/commit/4fb921e388b9a042573ab06cce36e92874954197
* shell: use msg_outtrans_len_attr for :!cmdBjörn Linse2018-02-05
| | | | fixes #7830 and #7788
* *: Provide list length when allocating listsZyX2018-01-14
|
* vim-patch:8.0.0657: cannot get and set quickfix list itemsJames McCoy2017-12-31
| | | | | | | | Problem: Cannot get and set quickfix list items. Solution: Add the "items" argument to getqflist() and setqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/6a8958db259d4444da6e6956e54a6513c1af8860
* vim-patch:8.0.0634: cannot easily get to the last quickfix listJames McCoy2017-12-31
| | | | | | | | Problem: Cannot easily get to the last quickfix list. Solution: Add "$" as a value for the "nr" argument of getqflist() and setqflist(). (Yegappan Lakshmanan) https://github.com/vim/vim/commit/875feea6ce223462d55543735143d747dcaf4287
* lintJames McCoy2017-12-30
|
* vim-patch:8.0.0608: cannot manipulate other than the current quickfix listJames McCoy2017-12-30
| | | | | | | Problem: Cannot manipulate other than the current quickfix list. Solution: Pass the list index to quickfix functions. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/a3921f48c6b31a035c80fda49925dd3b42df0dec
* Merge branch 'master' into hide-container-implZyX2017-12-23
|\
| * vim-patch:8.0.0606: cannot set the context for a specified quickfix listJames McCoy2017-12-19
| | | | | | | | | | | | | | | | Problem: Cannot set the context for a specified quickfix list. Solution: Use the list index instead of the current list. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/6e62da3e14d32f76f60d5cc8b267059923842f17
| * vim-patch:8.0.0597: off-by-one error in size computationJames McCoy2017-12-19
| | | | | | | | | | | | | | Problem: Off-by-one error in buffer size computation. Solution: Use ">=" instead of ">". (Lemonboy, closes vim/vim#1694) https://github.com/vim/vim/commit/253f9128779f315ea670f9b4a17446b7b4c74927
| * vim-patch:8.0.0595: Coverity warning for not checking return valueJames McCoy2017-12-19
| | | | | | | | | | | | | | Problem: Coverity warning for not checking return value of dict_add(). Solution: Check the return value for FAIL. https://github.com/vim/vim/commit/beb9cb19c660484488a71a25eda46ab0fa579278
| * vim-patch:8.0.0590: cannot add a context to locationsJames McCoy2017-12-19
| | | | | | | | | | | | | | | | Problem: Cannot add a context to locations. Solution: Add the "context" entry in location entries. (Yegappan Lakshmanan, closes vim/vim#1012) https://github.com/vim/vim/commit/8f77c5a4ec756f3f866bd6b18feb6fca6f2a2e91
| * vim-patch:8.0.0580: cannot set the valid flag with setqflist()James McCoy2017-12-18
| | | | | | | | | | | | | | Problem: Cannot set the valid flag with setqflist(). Solution: Add the "valid" argument. (Yegappan Lakshmanan, closes vim/vim#1642) https://github.com/vim/vim/commit/f1d21c8cc83f40c815b6bf13cd2043152db533ee
| * vim-patch:8.0.0574: get only one quickfix list after :caddbufJames McCoy2017-12-18
| | | | | | | | | | | | | | Problem: Get only one quickfix list after :caddbuf. Solution: Reset qf_multiline. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/99895eac1cf71be43ece7e14b50e206e041fbe9f
| * vim-patch:8.0.0565: using freed memory in :caddbufJames McCoy2017-12-18
| | | | | | | | | | | | | | | | Problem: Using freed memory in :caddbuf after clearing quickfix list. (Dominique Pelle) Solution: Set qf_last to NULL. https://github.com/vim/vim/commit/31bdd13c335533c749993b57dcd980a87373139e
| * vim-patch:8.0.0584: memory leak when executing quickfix testsJames McCoy2017-12-18
| | | | | | | | | | | | | | Problem: Memory leak when executing quickfix tests. Solution: Free the list reference. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/d788f6fe89c77262c474de323f5dab6d1c814e27
| * vim-patch:8.0.0536: quickfix window not updated when freeing quickfix stackJames McCoy2017-12-18
| | | | | | | | | | | | | | Problem: Quickfix window not updated when freeing quickfix stack. Solution: Update the quickfix window. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/69f40be64555d50f603c6f22722cf762aaa6bbc1
| * vim-patch:8.0.0517: there is no way to remove quickfix listsJames McCoy2017-12-18
| | | | | | | | | | | | | | | | Problem: There is no way to remove quickfix lists (for testing). Solution: Add the 'f' action to setqflist(). Add tests. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/b6fa30ccc39cdb7f1d07b99fe2f4c6b61671dac2
| * vim-patch:8.0.0484: :lhelpgrep does not fail after a successful oneJames McCoy2017-12-17
| | | | | | | | | | | | | | | | | | Problem: Using :lhelpgrep with an argument that should fail does not produce an error if the previous :helpgrep worked. Solution: Use another way to detect that autocommands made the quickfix info invalid. (Yegappan Lakshmanan) https://github.com/vim/vim/commit/ee85df37634dfb0c40ae5de0b4f246aef460b392
| * lintJames McCoy2017-12-17
| |
| * vim-patch:8.0.0420: text garbled when the system encoding differs from ↵James McCoy2017-12-17
| | | | | | | | | | | | | | | | | | | | 'encoding' Problem: When running :make the output may be in the system encoding, different from 'encoding'. Solution: Add the 'makeencoding' option. (Ken Takata) https://github.com/vim/vim/commit/2c7292dc5bbf155fe2192d417363b8c085759cad
* | Merge branch 'master' into hide-container-implZyX2017-12-16
|\|
| * vim-patch:8.0.0074Michael Schupikov2017-12-15
| | | | | | | | | | | | | | | | | | | | Problem: Cannot make Vim fail on an internal error. Solution: Add IEMSG() and IEMSG2(). (Domenique Pelle) Avoid reporting an internal error without mentioning where. https://github.com/vim/vim/commit/95f096030ed1a8afea028f2ea295d6f6a70f466f Signed-off-by: Michael Schupikov <michael@schupikov.de>
* | quickfix: Fix :cexpr and :lexprZyX2017-12-10
| |
* | *: Hide list implementation in other files as wellZyX2017-12-10
|/
* io: retry fgets on EINTR (#7632)Matt Widmann2017-11-25
| | | | | | | | | | | | | | | | | | The calls to `fgets` in `src/nvim/if_cscope.c` (and elsewhere) can show communication errors to the user if a signal is delivered during its system calls. For plugins that proxy subprocess output into cscope requests, a `SIGCHLD` might *always* interfere with calls into `fgets`. To see this in a debugger, put a breakpoint on `cs_reading_emsg` and watch signals come in (with lldb, using `process handle --notify true --pass true`). Next, run a subcommand from neovim that calls through cscope when it returns. A tag picker plugin, like vim-picker and fzy, with `cscopetag` and `cscopetagorder=0` set, reproduced this reliably. The breakpoint will hit after a `SIGCHLD` is delivered, and `errno` will be set to 4, `EINTR`. The caller of `fgets` should retry when `NULL` is returned with `errno` set to `EINTR`.