[vim/vim] WIP: Fix passed timeout with backwards searchpair (#2552)

23 views
Skip to first unread message

Daniel Hahler

unread,
Jan 15, 2018, 1:49:32 AM1/15/18
to vim/vim, Subscribed

searchpair(pos) might return an invalid value in case a timeout is used.

This can be triggered using searchpairpos('(', '', ')', 'bnW', '', 1, 500) (not reliably, but very often from line 1818 of
https://github.com/vim-jp/vim-vimlparser/blob/9eb216ff880d7b7fee036f096b6ad822651ea4db/py/vimlparser.py#L1818).

This is a WIP PR to check if any existing tests are failing.
For Neovim: neovim/neovim#7852

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

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

Commit Summary

  • WIP: Fix passed timeout with backwards searchpair

File Changes

Patch Links:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub

Daniel Hahler

unread,
Jan 15, 2018, 10:24:32 AM1/15/18
to vim/vim, Push

@blueyed pushed 1 commit.

  • 9890e5a fixup! WIP: Fix passed timeout with backwards searchpair

—
You are receiving this because you are subscribed to this thread.

View it on GitHub

Christian Brabandt

unread,
Feb 3, 2018, 4:06:35 PM2/3/18
to vim/vim, Subscribed

can you please write a test for the change?

Reply all
Reply to author
Forward
0 new messages