Skip to content

Conversation

@girishji
Copy link
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
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 <girishji@gmail.com>
@girishji
Copy link
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 <girishji@gmail.com>
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 <girishji@gmail.com>
@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