- patchlevel 1115
This commit is contained in:
parent
ce7d1bf4a2
commit
6012b0a305
284
7.3.1115
Normal file
284
7.3.1115
Normal file
@ -0,0 +1,284 @@
|
||||
To: vim_dev@googlegroups.com
|
||||
Subject: Patch 7.3.1115
|
||||
Fcc: outbox
|
||||
From: Bram Moolenaar <Bram@moolenaar.net>
|
||||
Mime-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
------------
|
||||
|
||||
Patch 7.3.1115
|
||||
Problem: Many users don't like the cursor line number when 'relativenumber'
|
||||
is set.
|
||||
Solution: Have four combinations with 'number' and 'relativenumber'.
|
||||
(Christian Brabandt)
|
||||
Files: runtime/doc/options.txt, src/option.c, src/screen.c,
|
||||
src/testdir/test89.in, src/testdir/test89.ok
|
||||
|
||||
|
||||
*** ../vim-7.3.1114/runtime/doc/options.txt 2013-03-19 16:46:59.000000000 +0100
|
||||
--- runtime/doc/options.txt 2013-06-04 22:01:35.000000000 +0200
|
||||
***************
|
||||
*** 5077,5084 ****
|
||||
number.
|
||||
When a long, wrapped line doesn't start with the first character, '-'
|
||||
characters are put before the number.
|
||||
! See |hl-LineNr| for the highlighting used for the number.
|
||||
! When setting this option, 'relativenumber' is reset.
|
||||
|
||||
*'numberwidth'* *'nuw'*
|
||||
'numberwidth' 'nuw' number (Vim default: 4 Vi default: 8)
|
||||
--- 5130,5149 ----
|
||||
number.
|
||||
When a long, wrapped line doesn't start with the first character, '-'
|
||||
characters are put before the number.
|
||||
! See |hl-LineNr| and |hl-CursorLineNr| for the highlighting used for
|
||||
! the number.
|
||||
! *number_relativenumber*
|
||||
! The 'relativenumber' option changes the displayed number to be
|
||||
! relative to the cursor. Together with 'number' there are these
|
||||
! four combinations (cursor in line 3):
|
||||
!
|
||||
! 'nonu' 'nu' 'nonu' 'nu'
|
||||
! 'nornu' 'nornu' 'rnu' 'rnu'
|
||||
!
|
||||
! |apple | 1 apple | 2 apple | 2 apple
|
||||
! |pear | 2 pear | 1 pear | 1 pear
|
||||
! |nobody | 3 nobody | 0 nobody |3 nobody
|
||||
! |there | 4 there | 1 there | 1 there
|
||||
|
||||
*'numberwidth'* *'nuw'*
|
||||
'numberwidth' 'nuw' number (Vim default: 4 Vi default: 8)
|
||||
***************
|
||||
*** 5484,5491 ****
|
||||
number.
|
||||
When a long, wrapped line doesn't start with the first character, '-'
|
||||
characters are put before the number.
|
||||
! See |hl-LineNr| for the highlighting used for the number.
|
||||
! When setting this option, 'number' is reset.
|
||||
|
||||
*'remap'* *'noremap'*
|
||||
'remap' boolean (default on)
|
||||
--- 5556,5567 ----
|
||||
number.
|
||||
When a long, wrapped line doesn't start with the first character, '-'
|
||||
characters are put before the number.
|
||||
! See |hl-LineNr| and |hl-CursorLineNr| for the highlighting used for
|
||||
! the number.
|
||||
!
|
||||
! The number in front of the cursor line also depends on the value of
|
||||
! 'number', see |number_relativenumber| for all combinations of the two
|
||||
! options.
|
||||
|
||||
*'remap'* *'noremap'*
|
||||
'remap' boolean (default on)
|
||||
*** ../vim-7.3.1114/src/option.c 2013-05-21 22:38:14.000000000 +0200
|
||||
--- src/option.c 2013-06-04 21:52:13.000000000 +0200
|
||||
***************
|
||||
*** 7647,7681 ****
|
||||
}
|
||||
#endif
|
||||
|
||||
- /* If 'number' is set, reset 'relativenumber'. */
|
||||
- /* If 'relativenumber' is set, reset 'number'. */
|
||||
- else if ((int *)varp == &curwin->w_p_nu && curwin->w_p_nu)
|
||||
- {
|
||||
- curwin->w_p_rnu = FALSE;
|
||||
-
|
||||
- /* Only reset the global value if the own value is set globally. */
|
||||
- if (((opt_flags & (OPT_LOCAL | OPT_GLOBAL)) == 0))
|
||||
- curwin->w_allbuf_opt.wo_rnu = FALSE;
|
||||
- }
|
||||
- else if ((int *)varp == &curwin->w_p_rnu && curwin->w_p_rnu)
|
||||
- {
|
||||
- curwin->w_p_nu = FALSE;
|
||||
-
|
||||
- /* Only reset the global value if the own value is set globally. */
|
||||
- if (((opt_flags & (OPT_LOCAL | OPT_GLOBAL)) == 0))
|
||||
- curwin->w_allbuf_opt.wo_nu = FALSE;
|
||||
- }
|
||||
- else if ((int *)varp == &curwin->w_allbuf_opt.wo_nu
|
||||
- && curwin->w_allbuf_opt.wo_nu)
|
||||
- {
|
||||
- curwin->w_allbuf_opt.wo_rnu = FALSE;
|
||||
- }
|
||||
- else if ((int *)varp == &curwin->w_allbuf_opt.wo_rnu
|
||||
- && curwin->w_allbuf_opt.wo_rnu)
|
||||
- {
|
||||
- curwin->w_allbuf_opt.wo_nu = FALSE;
|
||||
- }
|
||||
-
|
||||
else if ((int *)varp == &curbuf->b_p_ro)
|
||||
{
|
||||
/* when 'readonly' is reset globally, also reset readonlymode */
|
||||
--- 7647,7652 ----
|
||||
*** ../vim-7.3.1114/src/screen.c 2013-05-06 04:21:35.000000000 +0200
|
||||
--- src/screen.c 2013-06-04 22:04:39.000000000 +0200
|
||||
***************
|
||||
*** 2329,2343 ****
|
||||
if (len > w + 1)
|
||||
len = w + 1;
|
||||
|
||||
! if (wp->w_p_nu)
|
||||
! /* 'number' */
|
||||
num = (long)lnum;
|
||||
else
|
||||
{
|
||||
/* 'relativenumber', don't use negative numbers */
|
||||
num = labs((long)get_cursor_rel_lnum(wp, lnum));
|
||||
! if (num == 0)
|
||||
{
|
||||
num = lnum;
|
||||
fmt = "%-*ld ";
|
||||
}
|
||||
--- 2329,2345 ----
|
||||
if (len > w + 1)
|
||||
len = w + 1;
|
||||
|
||||
! if (wp->w_p_nu && !wp->w_p_rnu)
|
||||
! /* 'number' + 'norelativenumber' */
|
||||
num = (long)lnum;
|
||||
else
|
||||
{
|
||||
/* 'relativenumber', don't use negative numbers */
|
||||
num = labs((long)get_cursor_rel_lnum(wp, lnum));
|
||||
! if (num == 0 && wp->w_p_nu && wp->w_p_rnu)
|
||||
{
|
||||
+ /* 'number' + 'relativenumber': cursor line shows absolute
|
||||
+ * line number */
|
||||
num = lnum;
|
||||
fmt = "%-*ld ";
|
||||
}
|
||||
***************
|
||||
*** 3499,3513 ****
|
||||
long num;
|
||||
char *fmt = "%*ld ";
|
||||
|
||||
! if (wp->w_p_nu)
|
||||
! /* 'number' */
|
||||
num = (long)lnum;
|
||||
else
|
||||
{
|
||||
/* 'relativenumber', don't use negative numbers */
|
||||
num = labs((long)get_cursor_rel_lnum(wp, lnum));
|
||||
! if (num == 0)
|
||||
{
|
||||
num = lnum;
|
||||
fmt = "%-*ld ";
|
||||
}
|
||||
--- 3501,3516 ----
|
||||
long num;
|
||||
char *fmt = "%*ld ";
|
||||
|
||||
! if (wp->w_p_nu && !wp->w_p_rnu)
|
||||
! /* 'number' + 'norelativenumber' */
|
||||
num = (long)lnum;
|
||||
else
|
||||
{
|
||||
/* 'relativenumber', don't use negative numbers */
|
||||
num = labs((long)get_cursor_rel_lnum(wp, lnum));
|
||||
! if (num == 0 && wp->w_p_nu && wp->w_p_rnu)
|
||||
{
|
||||
+ /* 'number' + 'relativenumber' */
|
||||
num = lnum;
|
||||
fmt = "%-*ld ";
|
||||
}
|
||||
***************
|
||||
*** 10260,10266 ****
|
||||
int n;
|
||||
linenr_T lnum;
|
||||
|
||||
! lnum = wp->w_buffer->b_ml.ml_line_count;
|
||||
|
||||
if (lnum == wp->w_nrwidth_line_count)
|
||||
return wp->w_nrwidth_width;
|
||||
--- 10263,10274 ----
|
||||
int n;
|
||||
linenr_T lnum;
|
||||
|
||||
! if (wp->w_p_rnu && !wp->w_p_nu)
|
||||
! /* cursor line shows "0" */
|
||||
! lnum = wp->w_height;
|
||||
! else
|
||||
! /* cursor line shows absolute line number */
|
||||
! lnum = wp->w_buffer->b_ml.ml_line_count;
|
||||
|
||||
if (lnum == wp->w_nrwidth_line_count)
|
||||
return wp->w_nrwidth_width;
|
||||
*** ../vim-7.3.1114/src/testdir/test89.in 2013-04-24 15:47:11.000000000 +0200
|
||||
--- src/testdir/test89.in 2013-06-04 21:59:01.000000000 +0200
|
||||
***************
|
||||
*** 1,4 ****
|
||||
--- 1,6 ----
|
||||
Some tests for setting 'number' and 'relativenumber'
|
||||
+ This is not all that useful now that the options are no longer reset when
|
||||
+ setting the other.
|
||||
|
||||
STARTTEST
|
||||
:so small.vim
|
||||
*** ../vim-7.3.1114/src/testdir/test89.ok 2013-03-13 20:42:28.000000000 +0100
|
||||
--- src/testdir/test89.ok 2013-06-04 21:58:09.000000000 +0200
|
||||
***************
|
||||
*** 1,9 ****
|
||||
results:
|
||||
|
||||
! nonumber
|
||||
relativenumber
|
||||
|
||||
! nonumber
|
||||
relativenumber
|
||||
:setlocal must NOT reset the other global value
|
||||
|
||||
--- 1,9 ----
|
||||
results:
|
||||
|
||||
! number
|
||||
relativenumber
|
||||
|
||||
! number
|
||||
relativenumber
|
||||
:setlocal must NOT reset the other global value
|
||||
|
||||
***************
|
||||
*** 12,22 ****
|
||||
relativenumber
|
||||
:setglobal MUST reset the other global value
|
||||
|
||||
! nonumber
|
||||
|
||||
! norelativenumber
|
||||
:set MUST reset the other global value
|
||||
|
||||
! nonumber
|
||||
|
||||
! norelativenumber
|
||||
--- 12,22 ----
|
||||
relativenumber
|
||||
:setglobal MUST reset the other global value
|
||||
|
||||
! number
|
||||
|
||||
! relativenumber
|
||||
:set MUST reset the other global value
|
||||
|
||||
! number
|
||||
|
||||
! relativenumber
|
||||
*** ../vim-7.3.1114/src/version.c 2013-06-04 21:42:19.000000000 +0200
|
||||
--- src/version.c 2013-06-04 22:11:05.000000000 +0200
|
||||
***************
|
||||
*** 730,731 ****
|
||||
--- 730,733 ----
|
||||
{ /* Add new patch number below this line */
|
||||
+ /**/
|
||||
+ 1115,
|
||||
/**/
|
||||
|
||||
--
|
||||
From "know your smileys":
|
||||
:----} You lie like Pinocchio
|
||||
|
||||
/// 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 ///
|
Loading…
Reference in New Issue
Block a user