[vim/vim] Vim9: hang when a line break is used before "->(" (PR #21147)

[h_east]

Problem:  In Vim9 script a line break before "->(" makes Vim hang and
          use up memory.
Solution: Consume the line break for "->(" as well, and skip white
          space after "->" in a compiled function too.

fixes: #21144

---

You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/21147

Commit Summary

• 4ab47cc Vim9: hang when a line break is used before "->("

File Changes

(3 files)

• M src/eval.c (22)
• M src/testdir/test_vim9_expr.vim (8)
• M src/vim9expr.c (2)

Patch Links:

• https://github.com/vim/vim/pull/21147.patch
• https://github.com/vim/vim/pull/21147.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/21147@github.com>

[Christ van Willegen]

Hi,

Op di 25 aug 2026, 12:06 schreef h_east <vim-dev...@256bit.org>:

--
--
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+u...@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/vim_dev/vim/vim/pull/21147%40github.com.

The original bug report also had a case with a \ on the continuation line. Is that included in the fix? I think I didn't see it in the test, but that may be a rendering bug on the web site...

Christ van Willegen

[h_east]

h-east left a comment (vim/vim#21147)

The backslash case already worked before this patch. Continuation lines are
concatenated into one line before the expression is parsed, so they never
reach the code this patch changes. I checked on a 9.2.1001 build without the
patch: only the version without a backslash hangs.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you commented.Message ID: <vim/vim/pull/21147/c5410504476@github.com>

[Christian Brabandt]

chrisbra left a comment (vim/vim#21147)

thanks

—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!

You are receiving this because you commented.Message ID: <vim/vim/pull/21147/c5416250713@github.com>