mirror of
https://github.com/vim/vim.git
synced 2025-07-26 11:04:33 -04:00
patch 8.1.1758: count of g$ not used correctly when text is not wrapped
Problem: Count of g$ not used correctly when text is not wrapped. Solution: Do use the count. (Christian Brabandt, closes #4729, closes #4566)
This commit is contained in:
parent
2984666291
commit
d5c8234517
@ -8062,10 +8062,14 @@ nv_g_cmd(cmdarg_T *cap)
|
|||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
|
if (cap->count1 > 1)
|
||||||
|
// if it fails, let the cursor still move to the last char
|
||||||
|
cursor_down(cap->count1 - 1, FALSE);
|
||||||
|
|
||||||
i = curwin->w_leftcol + curwin->w_width - col_off - 1;
|
i = curwin->w_leftcol + curwin->w_width - col_off - 1;
|
||||||
coladvance((colnr_T)i);
|
coladvance((colnr_T)i);
|
||||||
|
|
||||||
/* Make sure we stick in this column. */
|
// Make sure we stick in this column.
|
||||||
validate_virtcol();
|
validate_virtcol();
|
||||||
curwin->w_curswant = curwin->w_virtcol;
|
curwin->w_curswant = curwin->w_virtcol;
|
||||||
curwin->w_set_curswant = FALSE;
|
curwin->w_set_curswant = FALSE;
|
||||||
|
@ -2732,3 +2732,58 @@ Piece of Java
|
|||||||
|
|
||||||
close!
|
close!
|
||||||
endfunc
|
endfunc
|
||||||
|
|
||||||
|
fun! Test_normal_gdollar_cmd()
|
||||||
|
if !has("jumplist")
|
||||||
|
return
|
||||||
|
endif
|
||||||
|
" Tests for g cmds
|
||||||
|
call Setup_NewWindow()
|
||||||
|
" Make long lines that will wrap
|
||||||
|
%s/$/\=repeat(' foobar', 10)/
|
||||||
|
20vsp
|
||||||
|
set wrap
|
||||||
|
" Test for g$ with count
|
||||||
|
norm! gg
|
||||||
|
norm! 0vg$y
|
||||||
|
call assert_equal(20, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foob', getreg(0))
|
||||||
|
norm! gg
|
||||||
|
norm! 0v4g$y
|
||||||
|
call assert_equal(72, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.."\n", getreg(0))
|
||||||
|
norm! gg
|
||||||
|
norm! 0v6g$y
|
||||||
|
call assert_equal(40, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '2 foobar foobar foobar foobar foobar foo', getreg(0))
|
||||||
|
set nowrap
|
||||||
|
" clean up
|
||||||
|
norm! gg
|
||||||
|
norm! 0vg$y
|
||||||
|
call assert_equal(20, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foob', getreg(0))
|
||||||
|
norm! gg
|
||||||
|
norm! 0v4g$y
|
||||||
|
call assert_equal(20, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '2 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '3 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '4 foobar foobar foob', getreg(0))
|
||||||
|
norm! gg
|
||||||
|
norm! 0v6g$y
|
||||||
|
call assert_equal(20, col("'>"))
|
||||||
|
call assert_equal('1 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '2 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '3 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '4 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '5 foobar foobar foobar foobar foobar foobar foobar foobar foobar foobar'.. "\n"..
|
||||||
|
\ '6 foobar foobar foob', getreg(0))
|
||||||
|
" Move to last line, also down movement is not possible, should still move
|
||||||
|
" the cursor to the last visible char
|
||||||
|
norm! G
|
||||||
|
norm! 0v6g$y
|
||||||
|
call assert_equal(20, col("'>"))
|
||||||
|
call assert_equal('100 foobar foobar fo', getreg(0))
|
||||||
|
bw!
|
||||||
|
endfunc
|
||||||
|
@ -777,6 +777,8 @@ static char *(features[]) =
|
|||||||
|
|
||||||
static int included_patches[] =
|
static int included_patches[] =
|
||||||
{ /* Add new patch number below this line */
|
{ /* Add new patch number below this line */
|
||||||
|
/**/
|
||||||
|
1758,
|
||||||
/**/
|
/**/
|
||||||
1757,
|
1757,
|
||||||
/**/
|
/**/
|
||||||
|
Loading…
x
Reference in New Issue
Block a user