Minor missing functionality from Vim `%` handling about zep HOT 4 CLOSED

A few more (single-line) test cases that currently fail:

// More `%` test cases
CURSOR_TEST(motion_percent_multiple, "vec2(cos(x), sin(y))", "9l%", 8, 0); // fail: jumps to 18
CURSOR_TEST(motion_percent_multiple, "vec2(cos(x), sin(y))", "10l%", 8, 0); // should be same as previous (starting on paren) - fail: jumps to 18
CURSOR_TEST(motion_percent_multiple, "vec2(cos(x), sin(y))", "9l%%", 10, 0); // fail: jumps to 16
CURSOR_TEST(motion_percent_multiple, "vec2(cos(x), sin(y))", "17l%", 16, 0); // fail: jumps back to 8
CURSOR_TEST(motion_percent_multiple, "vec2(cos(x), sin(y))", "17l%%", 18, 0); // fail: jumps to 10

from zep.

Thanks for the report; I made this more complicated than it needed to be :) I've added the new unit tests and everything passes. Regarding your comment "Zep does not find matching parens when they're on a previous line (unlike Vim)." I think you may have been referring to the character on a final bracket matching a previous one on a different line:

{
} -< Matches to above

This seems to work too; but note that Zep will not search forward beyond the current line to find the bracket to start on. This matches neovim and VsVim as far as I can see.

from zep.

Yes, that's what I meant, and what you've done seems correct (especially if all those tests now pass...I've not tried it yet but it sounds good). Yes, it's not supposed to search forward beyond the current line to find a bracket to start on, according to the Vim "spec":

Find the next item in this line after or under the cursor and jump to its match. (from :help %)

The matched item may be on a different line (earlier or later), but the starting item should only be after the cursor, and only considering the current line.

Thanks again for this!

from zep.

Tested it out with the latest version and all seems great with %, thanks.

from zep.