Skip to content

Fix: Memory leak related to cmdcomplete_info() - #17190

Closed
girishji wants to merge 3 commits into
vim:masterfrom
girishji:pr_cmdline2
Closed

girishji wants to merge 3 commits into
vim:masterfrom
girishji:pr_cmdline2

Conversation

@girishji

Copy link
Copy Markdown
Contributor

Fix issues mentioned in #17115 (comment)

M  src/cmdexpand.c
M  src/ex_getln.c
M  src/testdir/test_autocmd.vim
M  runtime/doc/builtin.txt
@chrisbra

Copy link
Copy Markdown
Member

thanks, I had also a fix for the leak issue in #17189 but I'll drop it over there then again.

@chrisbra chrisbra closed this in 5c3d1e3 Apr 22, 2025
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Apr 22, 2025
Problem:  Leaking memory with cmdcomplete()
          (zeertzjq, after v9.1.1329)
Solution: free the memory (Girish Palya)

closes: vim/vim#17190

vim/vim@5c3d1e3

Co-authored-by: Girish Palya <[email protected]>
@girishji

Copy link
Copy Markdown
Contributor Author

thanks, I had also a fix for the leak issue in #17189 but I'll drop it over there then again.

Thanks, I saw that after I created the PR!

zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Apr 23, 2025
Problem:  Leaking memory with cmdcomplete()
          (zeertzjq, after v9.1.1329)
Solution: free the memory (Girish Palya)

closes: vim/vim#17190

vim/vim@5c3d1e3

Co-authored-by: Girish Palya <[email protected]>
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Apr 23, 2025
Problem:  Leaking memory with cmdcomplete()
          (zeertzjq, after v9.1.1329)
Solution: free the memory (Girish Palya)

closes: vim/vim#17190

vim/vim@5c3d1e3

Co-authored-by: Girish Palya <[email protected]>
@girishji
girishji deleted the pr_cmdline2 branch April 24, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants