| Age | Commit message (Collapse) | Author |
|---|
|
Problem: Looping over windows, buffers and tab pages is inconsistant.
Solution: Use FOR_ALL_ macros everywhere. (Yegappan Lakshmanan)
|
|
Problem: Line numbers in the error list are not always adjusted.
Solution: Set b_has_qf_entry properly. (Yegappan Lakshmanan)
|
|
Problem: Compiler warning for char/char_u conversion. (Tony Mechelynck)
Inefficient code.
Solution: Use more lines to fill with spaces. (Nikolai Pavlov) Add type cast.
|
|
Problem: Coverity warns for possible buffer overflow.
Solution: Use vim_strcat() instead of strcat().
|
|
Problem: qf_init_ext() is too big.
Solution: Move code to qf_parse_line() (Yegappan Lakshmanan)
|
|
Problem: When using ":vimgrep" may end up with duplicate buffers.
Solution: When adding an error list entry pass the buffer number if possible.
|
|
Problem: There is no way to get a list of the error lists.
Solution: Add ":chistory" and ":lhistory".
|
|
Problem: Compiler warning for initializing a struct.
Solution: Initialize in another way. (Anton Lindqvist)
|
|
Problem: The qf_init_ext() function is too big.
Solution: Refactor it. (Yegappan Lakshmanan)
|
|
Problem: More buf_valid() calls can be optimized.
Solution: Use bufref_valid() instead.
|
|
Problem: buflist_findname_stat() may find a dummy buffer.
Solution: Set the BF_DUMMY flag after loading a dummy buffer. Start
finding buffers from the end of the list.
|
|
Problem: buf_valid() can be slow when there are many buffers.
Solution: Add bufref_valid(), only go through the buffer list when a buffer
was freed.
|
|
Problem: When there are many errors adding them to the quickfix list takes
a long time.
Solution: Add BLN_NOOPT. Don't call buf_valid() in buf_copy_options().
Remember the last file name used. When going through the buffer
list start from the end of the list. Only call buf_valid() when
autocommands were executed.
|
|
Problem: There is a :cbottom command but no :lbottom command.
Solution: Add :lbottom. (Yegappan Lakshmanan)
|
|
Problem: Cannot easily scroll the quickfix window.
Solution: Add ":cbottom".
|
|
Problem: 'errorformat' is parsed for every call to ":caddexpr". Can't add
to two location lists asynchronously.
Solution: Keep the previously parsed data when appropriate. (mostly by
Yegappan Lakshmanan)
|
|
Problem: Number variables are not 64 bits while they could be.
Solution: Add the num64 feature. (Ken Takata)
|
|
Problem: On MS-Windows large files (> 2Gbyte) cause problems.
Solution: Use "off_T" instead of "off_t". Use "stat_T" instead of "struct
stat". Use 64 bit system functions if available. (Ken Takata)
|
|
Problem: It is not easy to see unrecognized error lines below the current
error position.
Solution: Add ":clist +count".
|
|
Problem: Coverity reports a resource leak.
Solution: Close "fd" also when bailing out.
|
|
Problem: The quickfix init function is too big.
Solution: Factor out parsing 'errorformat' to a separate function. (Yegappan
Lakshmanan)
|
|
Problem: Minor problems with the quickfix code.
Solution: Fix the problems. (Yegappan Lakshmanan)
|
|
Problem: Updating marks in a quickfix list is very slow when the list is
long.
Solution: Only update marks if the buffer has a quickfix entry.
|
|
Problem: Check for line break at end of line wrong. (Dominique Pelle)
Solution: Correct the logic.
|
|
Problem: Appending to a long quickfix list is slow.
Solution: Add qf_last.
|
|
Problem: Appending to the quickfix list while the quickfix window is open
is very slow.
Solution: Do not delete all the lines, only append the new ones. Avoid
using a window while updating the list. (closes #841)
|
|
Problem: The code to reallocate the buffer used for quickfix is repeated.
Solution: Move the code to a function. (Yegappan Lakshmanan, closes #831)
|
|
Problem: Warning from 64 bit compiler.
Solution: Add type cast. (Mike Williams)
|
|
Problem: Compiler warnings for unused variables. (Ajit Thakkar)
Solution: Add a dummy initialization. (Yasuhiro Matsumoto)
|
|
Problem: Memory access error when running test_quickfix.
Solution: Allocate one more byte. (Yegappan Lakshmanan)
|
|
Problem: Quickfix doesn't handle long lines well, they are split.
Solution: Drop characters after a limit. (Anton Lindqvist)
|
|
Problem: When adding to the quickfix list the current position is reset.
Solution: Do not reset the position when not needed. (Yegappan Lakshmanan)
|
|
Problem: Crash in :cgetexpr.
Solution: Check for NULL pointer. (Dominique) Add a test.
|
|
Problem: Using freed memory after setqflist() and ":caddbuffer". (Dominique)
Solution: Set qf_ptr when adding the first item to the quickfix list.
|
|
Problem: Crash when an autocommand changes a quickfix list. (Dominique)
Solution: Check wether an entry is still valid. (Yegappan Lakshmanan,
Hirohito Higashi)
|
|
Problem: The versplit feature makes the code uneccessary complicated.
Solution: Remove FEAT_VERTSPLIT, always support vertical splits when
FEAT_WINDOWS is defined.
|
|
Problem: Quickfix code using memory after being freed. (Dominique Pelle)
Solution: Detect that the window was closed. (Hirohito Higashi)
|
|
Problem: The quickfix title is truncated.
Solution: Save the command before it is truncated. (Anton Lindqvist)
|
|
Problem: Using old style function declarations.
Solution: Change to new style function declarations. (script by Hirohito
Higashi)
|
|
Problem: Still using __ARGS.
Solution: Remove __ARGS in several files. (script by Hirohito Higashi)
|
|
Problem: Autocommands triggered by quickfix cannot always get the current
title value.
Solution: Call qf_fill_buffer() later. (Christian Brabandt)
|
|
Problem: Using the local value of 'errorformat' causes more problems than
it solves.
Solution: Revert 7.4.1013.
|
|
Problem: Memory leak in :vimgrep.
Solution: Call FreeWild(). (Yegappan Lakshmanan)
|
|
Problem: New include file missing from distribution. Missing changes to
quickfix code.
Solution: Add alloc.h to the list of distributed files. Use the enum in
quickfix code.
|
|
Problem: Code will never be executed.
Solution: Remove the code.
|
|
Problem: It is not possible to test code that is only reached when memory
allocation fails.
Solution: Add the alloc_fail() function. Try it out with :vimgrep.
|
|
Problem: The local value of 'errorformat' is not used for ":lexpr" and
":cexpr".
Solution: Use the local value if it exists. (Christian Brabandt) Adjust the
help for this.
|
|
Problem: It's a bit clumsy to execute a command on a list of matches.
Solution: Add the ":ldo", ":lfdo", ":cdo" and ":cfdo" commands. (Yegappan
Lakshmanan)
|
|
Problem: ":cnext" may jump to the wrong column when setting
'virtualedit=all' (cs86661)
Solution: Reset the coladd field. (Hirohito Higashi)
|
|
Problem: Autocommands triggered by quickfix cannot get the current title
value.
Solution: Set w:quickfix_title earlier. (Yannick)
Also move the check for a title into the function.
|
