Any updates here, or is it still broken in HEAD? If it's still not fixed, it would be great to have the offending change reverted while waiting for the complete solution.
David On Sat, Aug 16, 2014 at 7:23 AM, Bram Moolenaar <[email protected]> wrote: > > Christian Brabandt wrote: > > > On Do, 14 Aug 2014, Bram Moolenaar wrote: > > > > > Christian Brabandt wrote: > > > > > > > >> But anyway, this raises the question again, why matchaddpos() > > > > >> accepts a list of positions (but only 8 items), while all other > > > > >> functions usually only accept 1 position. > > > > > > > > > > It's used for the matchparen plugin. The idea is that it's useful > for > > > > > other plugins as well. > > > > > > > > Yes, but for consistency with the other match() functions and the > VimL > > > > API it might still be a good idea to only accept a single position > per > > > > matchaddpos() call. It shouldn't be too hard, to adjust the > matchparen > > > > plugin to call matchaddpos() twice, right? > > > > > > It would require another way to deal with IDs. Or allow for adding > > > another match with the same ID. > > > > Okay, here is another patch. This should fix all known problems, that > > have been mentioned so far. BTW, perhaps we could use h_east (sorry > > don't remember your name) test and include it as performance test? > > Thanks. It's a bit strange to use mb_l here, it's hard to see it has the > correct value. > > I found another solution: check if shl->lnum is equal to lnum. That's > how the match is disabled by next_search_hl(). > > -- > Apparently, 1 in 5 people in the world are Chinese. And there are 5 > people in my family, so it must be one of them. It's either my mum > or my dad. Or my older brother Colin. Or my younger brother > Ho-Cha-Chu. But I think it's Colin. > > /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net > \\\ > /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ > \\\ > \\\ an exciting new programming language -- http://www.Zimbu.org > /// > \\\ help me help AIDS victims -- http://ICCF-Holland.org > /// > > -- > -- > You received this message from the "vim_use" 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 a topic in the > Google Groups "vim_use" group. > To unsubscribe from this topic, visit > https://groups.google.com/d/topic/vim_use/V6cRWX4c13E/unsubscribe. > To unsubscribe from this group and all its topics, send an email to > [email protected]. > For more options, visit https://groups.google.com/d/optout. > -- -- You received this message from the "vim_use" 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_use" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
