Friday, August 29, 2014

Re: vim-7.4.389 UI extremely sluggish with custom syntax highlighting plugins

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 <Bram@moolenaar.net> 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 -- Bram@Moolenaar.net -- 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 vim_use+unsubscribe@googlegroups.com.
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 vim_use+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

No comments: