7.3.708

   1 To: vim_dev@googlegroups.com
   2 Subject: Patch 7.3.708
   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.708
  11 Problem:    Filler lines above the first line may be hidden when opening Vim.
  12 Solution:   Change how topfill is computed. (Christian Brabandt)
  13 Files:      src/diff.c, src/testdir/test47.in, src/testdir/test47.ok
  14
  15
  16 *** ../vim-7.3.707/src/diff.c   2012-05-18 18:47:11.000000000 +0200
  17 --- src/diff.c  2012-10-21 22:08:44.000000000 +0200
  18 ***************
  19 *** 615,625 ****
  20   #endif
  21             /* A change may have made filler lines invalid, need to take care
  22              * of that for other windows. */
  23 !           if (wp != curwin && wp->w_topfill > 0)
  24             {
  25 -               n = diff_check(wp, wp->w_topline);
  26                 if (wp->w_topfill > n)
  27                     wp->w_topfill = (n < 0 ? 0 : n);
  28             }
  29         }
  30   }
  31 --- 615,627 ----
  32   #endif
  33             /* A change may have made filler lines invalid, need to take care
  34              * of that for other windows. */
  35 !           n = diff_check(wp, wp->w_topline);
  36 !           if ((wp != curwin && wp->w_topfill > 0) || n > 0)
  37             {
  38                 if (wp->w_topfill > n)
  39                     wp->w_topfill = (n < 0 ? 0 : n);
  40 +               else if (n > 0 && n > wp->w_topfill)
  41 +                   wp->w_topfill = n;
  42             }
  43         }
  44   }
  45 *** ../vim-7.3.707/src/testdir/test47.in        2010-08-15 21:57:29.000000000 +0200
  46 --- src/testdir/test47.in       2012-10-21 22:08:44.000000000 +0200
  47 ***************
  48 *** 36,42 ****
  49   :call append("$", two)
  50   :call append("$", three)
  51   :$-2,$w! test.out
  52 ! :unlet one two three
  53   :qa!
  54   ENDTEST
  55
  56 --- 36,57 ----
  57   :call append("$", two)
  58   :call append("$", three)
  59   :$-2,$w! test.out
  60 ! :" Test that diffing shows correct filler lines
  61 ! :diffoff!
  62 ! :windo :bw!
  63 ! :enew
  64 ! :put =range(4,10)
  65 ! :1d _
  66 ! :vnew
  67 ! :put =range(1,10)
  68 ! :1d _
  69 ! :windo :diffthis
  70 ! :wincmd h
  71 ! :let w0=line('w0')
  72 ! :enew
  73 ! :put =w0
  74 ! :.w >> test.out
  75 ! :unlet! one two three w0
  76   :qa!
  77   ENDTEST
  78
  79 *** ../vim-7.3.707/src/testdir/test47.ok        2010-08-15 21:57:29.000000000 +0200
  80 --- src/testdir/test47.ok       2012-10-21 22:08:44.000000000 +0200
  81 ***************
  82 *** 1,3 ****
  83 --- 1,4 ----
  84   2-4-5-6-8-9
  85   1-2-4-5-8
  86   2-3-4-5-6-7-8
  87 + 1
  88 *** ../vim-7.3.707/src/version.c        2012-10-21 21:38:42.000000000 +0200
  89 --- src/version.c       2012-10-21 22:10:42.000000000 +0200
  90 ***************
  91 *** 727,728 ****
  92 --- 727,730 ----
  93   {   /* Add new patch number below this line */
  94 + /**/
  95 +     708,
  96   /**/
  97
  98 --
  99 At some point in the project somebody will start whining about the need to
 100 determine the project "requirements".  This involves interviewing people who
 101 don't know what they want but, curiously, know exactly when they need it.
 102                                 (Scott Adams - The Dilbert principle)
 103
 104  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
 105 ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
 106 \\\  an exciting new programming language -- http://www.Zimbu.org        ///
 107  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///