Hi Bram,
On Sat, Apr 22, 2017 at 6:43 AM, Bram Moolenaar <b...@moolenaar.net> wrote:
>
> Yegappan Lakshmanan wrote:
>
>> On Thu, Apr 20, 2017 at 1:45 PM, Bram Moolenaar <b...@moolenaar.net> wrote:
>> >
>> > Patch 8.0.0574
>> > Problem: Get only one quickfix list after :caddbuf.
>> > Solution: Reset qf_multiline. (Yegappan Lakshmanan)
>> > Files: src/quickfix.c, src/testdir/test_quickfix.vim
>>
>> This new function is not needed and is a duplicate. I have already
>> added these commands above with asserts.
>
> OK, I'll remove this function then.
>
> I'm still trying to figure out a memory leak in Test_qf_free().
> Didn't get further than pinpointing it to this function.
> Not sure when I'll have time to dig deeper.
>
I have located the memory leak when freeing a location list from the
location list window and the fix for it is attached.
- Yegappan
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
diff --git a/src/quickfix.c b/src/quickfix.c
index b28a321..b7b6129 100644
--- a/src/quickfix.c
+++ b/src/quickfix.c
@@ -4930,6 +4930,9 @@ qf_free_stack(win_T *wp, qf_info_T *qi)
* location list */
qf_info_T *new_ll = ll_new_list();
+ /* first free the list reference in location list window */
+ ll_free_all(&orig_wp->w_llist_ref);
+
orig_wp->w_llist_ref = new_ll;
if (llwin != NULL)
{
