home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 7.3 / 7.3.1115 < prev    next >
Encoding:
Internet Message Format  |  2013-06-03  |  8.1 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.1115
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.3.1115
  11. Problem:    Many users don't like the cursor line number when 'relativenumber'
  12.         is set.
  13. Solution:   Have four combinations with 'number' and 'relativenumber'.
  14.         (Christian Brabandt)
  15. Files:        runtime/doc/options.txt, src/option.c, src/screen.c,
  16.         src/testdir/test89.in, src/testdir/test89.ok
  17.  
  18.  
  19. *** ../vim-7.3.1114/runtime/doc/options.txt    2013-03-19 16:46:59.000000000 +0100
  20. --- runtime/doc/options.txt    2013-06-04 22:01:35.000000000 +0200
  21. ***************
  22. *** 5077,5084 ****
  23.       number.
  24.       When a long, wrapped line doesn't start with the first character, '-'
  25.       characters are put before the number.
  26. !     See |hl-LineNr| for the highlighting used for the number.
  27. !     When setting this option, 'relativenumber' is reset.
  28.   
  29.                           *'numberwidth'* *'nuw'*
  30.   'numberwidth' 'nuw'    number    (Vim default: 4  Vi default: 8)
  31. --- 5130,5149 ----
  32.       number.
  33.       When a long, wrapped line doesn't start with the first character, '-'
  34.       characters are put before the number.
  35. !     See |hl-LineNr|  and |hl-CursorLineNr| for the highlighting used for
  36. !     the number.
  37. !                         *number_relativenumber*
  38. !     The 'relativenumber' option changes the displayed number to be
  39. !     relative to the cursor.  Together with 'number' there are these
  40. !     four combinations (cursor in line 3):
  41. ! 
  42. !                   'nonu'          'nu'            'nonu'          'nu'
  43. !         'nornu'         'nornu'         'rnu'           'rnu'
  44. ! 
  45. !         |apple          |  1 apple      |  2 apple      |  2 apple
  46. !         |pear           |  2 pear       |  1 pear       |  1 pear
  47. !         |nobody         |  3 nobody     |  0 nobody     |3   nobody
  48. !         |there          |  4 there      |  1 there      |  1 there
  49.   
  50.                           *'numberwidth'* *'nuw'*
  51.   'numberwidth' 'nuw'    number    (Vim default: 4  Vi default: 8)
  52. ***************
  53. *** 5484,5491 ****
  54.       number.
  55.       When a long, wrapped line doesn't start with the first character, '-'
  56.       characters are put before the number.
  57. !     See |hl-LineNr| for the highlighting used for the number.
  58. !     When setting this option, 'number' is reset.
  59.   
  60.                           *'remap'* *'noremap'*
  61.   'remap'            boolean    (default on)
  62. --- 5556,5567 ----
  63.       number.
  64.       When a long, wrapped line doesn't start with the first character, '-'
  65.       characters are put before the number.
  66. !     See |hl-LineNr|  and |hl-CursorLineNr| for the highlighting used for
  67. !     the number.
  68. !     
  69. !     The number in front of the cursor line also depends on the value of
  70. !     'number', see |number_relativenumber| for all combinations of the two
  71. !     options.
  72.   
  73.                           *'remap'* *'noremap'*
  74.   'remap'            boolean    (default on)
  75. *** ../vim-7.3.1114/src/option.c    2013-05-21 22:38:14.000000000 +0200
  76. --- src/option.c    2013-06-04 21:52:13.000000000 +0200
  77. ***************
  78. *** 7647,7681 ****
  79.       }
  80.   #endif
  81.   
  82. -     /* If 'number' is set, reset 'relativenumber'. */
  83. -     /* If 'relativenumber' is set, reset 'number'. */
  84. -     else if ((int *)varp == &curwin->w_p_nu && curwin->w_p_nu)
  85. -     {
  86. -     curwin->w_p_rnu = FALSE;
  87. - 
  88. -     /* Only reset the global value if the own value is set globally. */
  89. -     if (((opt_flags & (OPT_LOCAL | OPT_GLOBAL)) == 0))
  90. -         curwin->w_allbuf_opt.wo_rnu = FALSE;
  91. -     }
  92. -     else if ((int *)varp == &curwin->w_p_rnu && curwin->w_p_rnu)
  93. -     {
  94. -     curwin->w_p_nu = FALSE;
  95. - 
  96. -     /* Only reset the global value if the own value is set globally. */
  97. -     if (((opt_flags & (OPT_LOCAL | OPT_GLOBAL)) == 0))
  98. -         curwin->w_allbuf_opt.wo_nu = FALSE;
  99. -     }
  100. -     else if ((int *)varp == &curwin->w_allbuf_opt.wo_nu
  101. -                         && curwin->w_allbuf_opt.wo_nu)
  102. -     {
  103. -         curwin->w_allbuf_opt.wo_rnu = FALSE;
  104. -     }
  105. -     else if ((int *)varp == &curwin->w_allbuf_opt.wo_rnu
  106. -                            && curwin->w_allbuf_opt.wo_rnu)
  107. -     {
  108. -         curwin->w_allbuf_opt.wo_nu = FALSE;
  109. -     }
  110. - 
  111.       else if ((int *)varp == &curbuf->b_p_ro)
  112.       {
  113.       /* when 'readonly' is reset globally, also reset readonlymode */
  114. --- 7647,7652 ----
  115. *** ../vim-7.3.1114/src/screen.c    2013-05-06 04:21:35.000000000 +0200
  116. --- src/screen.c    2013-06-04 22:04:39.000000000 +0200
  117. ***************
  118. *** 2329,2343 ****
  119.           if (len > w + 1)
  120.           len = w + 1;
  121.   
  122. !         if (wp->w_p_nu)
  123. !         /* 'number' */
  124.           num = (long)lnum;
  125.           else
  126.           {
  127.           /* 'relativenumber', don't use negative numbers */
  128.           num = labs((long)get_cursor_rel_lnum(wp, lnum));
  129. !         if (num == 0)
  130.           {
  131.               num = lnum;
  132.               fmt = "%-*ld ";
  133.           }
  134. --- 2329,2345 ----
  135.           if (len > w + 1)
  136.           len = w + 1;
  137.   
  138. !         if (wp->w_p_nu && !wp->w_p_rnu)
  139. !         /* 'number' + 'norelativenumber' */
  140.           num = (long)lnum;
  141.           else
  142.           {
  143.           /* 'relativenumber', don't use negative numbers */
  144.           num = labs((long)get_cursor_rel_lnum(wp, lnum));
  145. !         if (num == 0 && wp->w_p_nu && wp->w_p_rnu)
  146.           {
  147. +             /* 'number' + 'relativenumber': cursor line shows absolute
  148. +              * line number */
  149.               num = lnum;
  150.               fmt = "%-*ld ";
  151.           }
  152. ***************
  153. *** 3499,3513 ****
  154.               long num;
  155.               char *fmt = "%*ld ";
  156.   
  157. !             if (wp->w_p_nu)
  158. !                 /* 'number' */
  159.                   num = (long)lnum;
  160.               else
  161.               {
  162.                   /* 'relativenumber', don't use negative numbers */
  163.                   num = labs((long)get_cursor_rel_lnum(wp, lnum));
  164. !                 if (num == 0)
  165.                   {
  166.                   num = lnum;
  167.                   fmt = "%-*ld ";
  168.                   }
  169. --- 3501,3516 ----
  170.               long num;
  171.               char *fmt = "%*ld ";
  172.   
  173. !             if (wp->w_p_nu && !wp->w_p_rnu)
  174. !                 /* 'number' + 'norelativenumber' */
  175.                   num = (long)lnum;
  176.               else
  177.               {
  178.                   /* 'relativenumber', don't use negative numbers */
  179.                   num = labs((long)get_cursor_rel_lnum(wp, lnum));
  180. !                 if (num == 0 && wp->w_p_nu && wp->w_p_rnu)
  181.                   {
  182. +                 /* 'number' + 'relativenumber' */
  183.                   num = lnum;
  184.                   fmt = "%-*ld ";
  185.                   }
  186. ***************
  187. *** 10260,10266 ****
  188.       int        n;
  189.       linenr_T    lnum;
  190.   
  191. !     lnum = wp->w_buffer->b_ml.ml_line_count;
  192.   
  193.       if (lnum == wp->w_nrwidth_line_count)
  194.       return wp->w_nrwidth_width;
  195. --- 10263,10274 ----
  196.       int        n;
  197.       linenr_T    lnum;
  198.   
  199. !     if (wp->w_p_rnu && !wp->w_p_nu)
  200. !     /* cursor line shows "0" */
  201. !     lnum = wp->w_height;
  202. !     else
  203. !     /* cursor line shows absolute line number */
  204. !     lnum = wp->w_buffer->b_ml.ml_line_count;
  205.   
  206.       if (lnum == wp->w_nrwidth_line_count)
  207.       return wp->w_nrwidth_width;
  208. *** ../vim-7.3.1114/src/testdir/test89.in    2013-04-24 15:47:11.000000000 +0200
  209. --- src/testdir/test89.in    2013-06-04 21:59:01.000000000 +0200
  210. ***************
  211. *** 1,4 ****
  212. --- 1,6 ----
  213.   Some tests for setting 'number' and 'relativenumber'
  214. + This is not all that useful now that the options are no longer reset when
  215. + setting the other.
  216.   
  217.   STARTTEST
  218.   :so small.vim
  219. *** ../vim-7.3.1114/src/testdir/test89.ok    2013-03-13 20:42:28.000000000 +0100
  220. --- src/testdir/test89.ok    2013-06-04 21:58:09.000000000 +0200
  221. ***************
  222. *** 1,9 ****
  223.   results:
  224.   
  225. ! nonumber
  226.     relativenumber
  227.   
  228. ! nonumber
  229.     relativenumber
  230.   :setlocal must NOT reset the other global value
  231.   
  232. --- 1,9 ----
  233.   results:
  234.   
  235. !   number
  236.     relativenumber
  237.   
  238. !   number
  239.     relativenumber
  240.   :setlocal must NOT reset the other global value
  241.   
  242. ***************
  243. *** 12,22 ****
  244.     relativenumber
  245.   :setglobal MUST reset the other global value
  246.   
  247. ! nonumber
  248.   
  249. ! norelativenumber
  250.   :set MUST reset the other global value
  251.   
  252. ! nonumber
  253.   
  254. ! norelativenumber
  255. --- 12,22 ----
  256.     relativenumber
  257.   :setglobal MUST reset the other global value
  258.   
  259. !   number
  260.   
  261. !   relativenumber
  262.   :set MUST reset the other global value
  263.   
  264. !   number
  265.   
  266. !   relativenumber
  267. *** ../vim-7.3.1114/src/version.c    2013-06-04 21:42:19.000000000 +0200
  268. --- src/version.c    2013-06-04 22:11:05.000000000 +0200
  269. ***************
  270. *** 730,731 ****
  271. --- 730,733 ----
  272.   {   /* Add new patch number below this line */
  273. + /**/
  274. +     1115,
  275.   /**/
  276.  
  277. -- 
  278. From "know your smileys":
  279.  :----}  You lie like Pinocchio
  280.  
  281.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  282. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  283. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  284.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  285.