Meanwhile i found an obvious reproducible effect: horizontal navigation (e.g. to the left and to the right) along a heavily loaded by parentheses line differs in speed which depends on whether this line at the top of the window (the slowest), mid of the window (faster) and bottom of the window (fastest). I suspected that this was due to redrawing area lasted from the top-parenthesis line to the bottom window line, though the bottom line of redrawing area should be N + 1. I checked if b_mod_top and b_mod_bot is N and N + 1 respectively when the line is being drawn in win_line(): it appeared true. This means that something else forces screen to redraw to the bottom line, but i cannot still find what. I tried also to set rtype in match_add() and match_delete() to SOME_VALID: this should make redrawing line slowest at any window position: indeed this worked as i expected: the line began to redraw at the slowest speed and this should prove that the effect is really due to wrong/unexpected redrawing area (i.e. its bottom line lasts to the bottom of the window/screen).
Cheers, Alexey. 2014-06-25 13:08 GMT+04:00 Alexey Radkov <[email protected]>: > Ok, I'll do this during this or the next week. > > Cheers, Alexey. > > > 2014-06-23 23:11 GMT+04:00 Bram Moolenaar <[email protected]>: > > >> Alexey Radkov wrote: >> >> > Here is some small improvements for the patch: >> > >> > src/window.c, match_add(): >> > - removed unnecessary (after alloc() to alloc_clear() transformation) >> > zero-initializations of m->pos fields, >> > - checking that li == NULL was moved inside for-loop body >> > - decrement i when the pos_list element is a number and has value 0 to >> be >> > symmetric with sublist counterpart >> > >> > src/screen.c: >> > - simple code alignment >> > >> > runtime/doc/eval.txt, matchaddpos(): >> > this is a bit more important: actually column number and match length >> must >> > correspond to bytes as col() returns, not screen columns as virtcol() >> > returns! It does matter for Unicode texts like Cyrillics, i changed doc >> to >> > reflect this. >> >> Thanks. >> >> Now that we have one working test, perhaps you can add a few more tests? >> Would be good if we cover the alternatives in the code. >> >> >> -- >> Proverb: A nightingale that forgets the lyrics is a hummingbird. >> >> /// 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_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 [email protected]. >> For more options, visit https://groups.google.com/d/optout. >> > > -- -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
