On Tue, Sep 08, 2015 at 05:31:53PM +0200, Bram Moolenaar wrote: > > Patch 7.4.856 > Problem: "zt" still doesn't work well with filler lines. (Gary Johnson) > Solution: Check for filler lines above the cursor. (Christian Brabandt) > Files: src/move.c > > > *** ../vim-7.4.855/src/move.c 2015-09-01 20:53:20.294030821 +0200 > --- src/move.c 2015-09-08 17:18:09.315751849 +0200 > *************** > *** 1751,1762 **** > new_topline = top + 1; > > #ifdef FEAT_DIFF > ! /* used already contains the number of filler lines above, don't add it > * again. > ! * TODO: if filler lines above new top are to be considered as context > for > ! * the current window, leave next statement commented, else hide filler > ! * lines above cursor line, by adding them to extra */ > ! /* extra += diff_check_fill(curwin, curwin->w_cursor.lnum); */ > #endif > > /* > --- 1751,1760 ---- > new_topline = top + 1; > > #ifdef FEAT_DIFF > ! /* "used" already contains the number of filler lines above, don't add > it > * again. > ! * Hide filler lines above cursor line by adding them to "extra". */ > ! extra += diff_check_fill(curwin, curwin->w_cursor.lnum); > #endif > > /* > *************** > *** 1771,1777 **** > i = 1; > else > #endif > ! i = plines(top); > used += i; > if (extra + i <= off && bot < curbuf->b_ml.ml_line_count) > { > --- 1769,1775 ---- > i = 1; > else > #endif > ! i = plines_nofill(top); > used += i; > if (extra + i <= off && bot < curbuf->b_ml.ml_line_count) > { > *** ../vim-7.4.855/src/version.c 2015-09-08 16:31:01.673123014 +0200 > --- src/version.c 2015-09-08 17:15:18.005529911 +0200 > *************** > *** 743,744 **** > --- 743,746 ---- > { /* Add new patch number below this line */ > + /**/ > + 856, > /**/ > > -- > I used to wonder about the meaning of life. But I looked it > up in the dictionary under "L" and there it was - the meaning > of life. It was less than I expected. - Dogbert > > /// Bram Moolenaar -- b...@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_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 vim_dev+unsubscr...@googlegroups.com. > For more options, visit https://groups.google.com/d/optout.
tiny build breaks with this patch, I've opened a pull request on github that only uses plines_nofill when FEAT_DIFF or PROTO is defined. https://github.com/vim/vim/pull/416 -- Ike -- -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
pgp2KJRYQLVhN.pgp
Description: PGP signature