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 / old / 5.6.060 < prev    next >
Encoding:
Internet Message Format  |  2000-04-04  |  8.4 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 5.6.060
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. ------------
  6.  
  7. Patch 5.6.060
  8. Problem:    Some bold characters spill over the the cell on the left, that
  9.         spill-over can remain sometimes.
  10. Solution:   Redraw a characters when the next character was bold and needs
  11.         redrawing. (Robert Webb)
  12. Files:        src/screen.c
  13.  
  14.  
  15. *** ../vim-5.6.59/src/screen.c    Tue Apr  4 20:52:36 2000
  16. --- src/screen.c    Wed Apr  5 16:17:34 2000
  17. ***************
  18. *** 67,72 ****
  19. --- 67,73 ----
  20.   
  21.   static void win_update __ARGS((WIN *wp));
  22.   static int win_line __ARGS((WIN *, linenr_t, int, int));
  23. + static int char_needs_redraw __ARGS((char_u *screenp_from, char_u *screenp_to, int len));
  24.   #ifdef RIGHTLEFT
  25.   static void screen_line __ARGS((int row, int endcol, int clear_rest, int rlflag));
  26.   #define SCREEN_LINE(r, e, c, rl)    screen_line((r), (e), (c), (rl))
  27. ***************
  28. *** 1875,1880 ****
  29. --- 1876,1908 ----
  30.   }
  31.   
  32.   /*
  33. +  * Check whether the given character needs redrawing:
  34. +  * - the (first byte of the) character is different
  35. +  * - the attributes are different
  36. +  * - the character is multi-byte and the next byte is different
  37. +  */
  38. +     static int
  39. + char_needs_redraw(screenp_from, screenp_to, len)
  40. +     char_u *screenp_from;
  41. +     char_u *screenp_to;
  42. +     int len;
  43. + {
  44. +     if (len > 0
  45. +         && ((*screenp_from != *screenp_to
  46. +             || *(screenp_from + Columns) != *(screenp_to + Columns))
  47. + 
  48. + #ifdef MULTI_BYTE
  49. +         || (is_dbcs
  50. +             && len > 1
  51. +             && *(screenp_from + 1) != *(screenp_to + 1)
  52. +             && IsLeadByte(*screenp_from))
  53. + #endif
  54. +            ))
  55. +     return TRUE;
  56. +     return FALSE;
  57. + }
  58. + 
  59. + /*
  60.    * Move one "cooked" screen line to the screen, but only the characters that
  61.    * have actually changed.  Handle insert/delete character.
  62.    * 'endcol' gives the columns where valid characters are.
  63. ***************
  64. *** 1899,1909 ****
  65.       char_u        *screenp_from;
  66.       char_u        *screenp_to;
  67.       int            col = 0;
  68.       int            force = FALSE;    /* force update rest of the line */
  69.   #ifdef MULTI_BYTE
  70.       int            char_bytes;        /* 1 : if normal char */
  71.                       /* 2 : if DBCS char */
  72. !     int            need_update;    /* bool variable */
  73.   #endif
  74.   
  75.       screenp_from = current_LinePointer;
  76. --- 1927,1942 ----
  77.       char_u        *screenp_from;
  78.       char_u        *screenp_to;
  79.       int            col = 0;
  80. +     int            hl;
  81.       int            force = FALSE;    /* force update rest of the line */
  82. +     int            redraw_this;    /* bool: does character need redraw? */
  83. +     int            redraw_next;    /* redraw_this for next character */
  84.   #ifdef MULTI_BYTE
  85.       int            char_bytes;        /* 1 : if normal char */
  86.                       /* 2 : if DBCS char */
  87. ! # define CHAR_BYTES char_bytes
  88. ! #else
  89. ! # define CHAR_BYTES 1
  90.   #endif
  91.   
  92.       screenp_from = current_LinePointer;
  93. ***************
  94. *** 1926,1975 ****
  95.       col = endcol + 1;
  96.       screenp_to = LinePointers[row] + col;
  97.       screenp_from += col;
  98.       }
  99.   
  100. -     while (rlflag ? (col < Columns) : (col < endcol))
  101. - #else
  102.       while (col < endcol)
  103. - #endif
  104.       {
  105.   #ifdef MULTI_BYTE
  106. !     if (is_dbcs && IsLeadByte(*screenp_from) && (col + 1 < endcol))
  107. !     {
  108.           char_bytes = 2;
  109. -         if ( force
  110. -             || *screenp_from != *screenp_to
  111. -             || *(screenp_from + 1) != *(screenp_to + 1)
  112. -             || *(screenp_from + Columns) != *(screenp_to + Columns)
  113. -         )
  114. -         need_update = TRUE;
  115. -         else
  116. -         need_update = FALSE;
  117. -     }
  118.       else
  119. -     {
  120.           char_bytes = 1;
  121. -         if ( force
  122. -             || *screenp_from != *screenp_to
  123. -             || *(screenp_from + Columns) != *(screenp_to + Columns)
  124. -         )
  125. -         need_update = TRUE;
  126. -         else
  127. -         need_update = FALSE;
  128. -     }
  129.   #endif
  130.   
  131. !     if (
  132. ! #if defined(MULTI_BYTE)
  133. !         need_update
  134. ! #else
  135. !            force
  136. !         || *screenp_from != *screenp_to
  137. !         || *(screenp_from + Columns) != *(screenp_to + Columns)
  138. ! #endif
  139. !        )
  140.       {
  141.   
  142.           /*
  143.            * Special handling when 'xs' termcap flag set (hpterm):
  144.            * Attributes for characters are stored at the position where the
  145. --- 1959,1999 ----
  146.       col = endcol + 1;
  147.       screenp_to = LinePointers[row] + col;
  148.       screenp_from += col;
  149. + 
  150. +     endcol = Columns;
  151.       }
  152. + #endif /* RIGHTLEFT */
  153. + 
  154. +     redraw_next = char_needs_redraw(screenp_from, screenp_to, endcol - col);
  155.   
  156.       while (col < endcol)
  157.       {
  158.   #ifdef MULTI_BYTE
  159. !     if (is_dbcs && (col + 1 < endcol) && IsLeadByte(*screenp_from))
  160.           char_bytes = 2;
  161.       else
  162.           char_bytes = 1;
  163.   #endif
  164.   
  165. !     redraw_this = redraw_next;
  166. !     redraw_next = force || char_needs_redraw(screenp_from + CHAR_BYTES,
  167. !               screenp_to + CHAR_BYTES, endcol - col - CHAR_BYTES);
  168. ! 
  169. ! #ifdef USE_GUI
  170. !     /* If the next character was bold, then redraw the current character to
  171. !      * remove any pixels that might have spilt over into us.  This only
  172. !      * happens in the GUI.
  173. !      */
  174. !     if (redraw_next && gui.in_use)
  175.       {
  176. +         hl = *(screenp_to + CHAR_BYTES + Columns);
  177. +         if (hl > HL_ALL || (hl & HL_BOLD))
  178. +         redraw_this = TRUE;
  179. +     }
  180. + #endif
  181.   
  182. +     if (redraw_this)
  183. +     {
  184.           /*
  185.            * Special handling when 'xs' termcap flag set (hpterm):
  186.            * Attributes for characters are stored at the position where the
  187. ***************
  188. *** 1997,2002 ****
  189. --- 2021,2027 ----
  190.           out_str(T_CE);        /* clear rest of this screen line */
  191.           screen_start();        /* don't know where cursor is now */
  192.           force = TRUE;        /* force redraw of rest of the line */
  193. +         redraw_next = TRUE;    /* or else next char would miss out */
  194.   
  195.           /*
  196.            * If the previous character was highlighted, need to stop
  197. ***************
  198. *** 2014,2027 ****
  199.   #ifdef MULTI_BYTE
  200.           if (is_dbcs)
  201.           {
  202. !         /* The trick to make a force update */
  203. !         if (!IsLeadByte(*screenp_from) && IsLeadByte(*screenp_to))
  204. !             *(screenp_to + 1) = 0;
  205. !         else if ((char_bytes == 2)
  206. !             && mb_isbyte1(LinePointers[row], col + 1))
  207.           {
  208.               *(screenp_to + 1) = 0;
  209.               *(screenp_to + 2) = 0;
  210.           }
  211.           }
  212.   #endif
  213. --- 2039,2059 ----
  214.   #ifdef MULTI_BYTE
  215.           if (is_dbcs)
  216.           {
  217. !         if (char_bytes == 1 && col + 1 < endcol
  218. !             && IsLeadByte(*screenp_to))
  219.           {
  220. +             /* When: !DBCS(*screenp_from) && DBCS(*screenp_to) */
  221.               *(screenp_to + 1) = 0;
  222. +             redraw_next = TRUE;
  223. +         }
  224. +         else if (char_bytes == 2 && col + 2 < endcol
  225. +             && !IsLeadByte(*screenp_to)
  226. +             && IsLeadByte(*(screenp_to + 1)))
  227. +         {
  228. +             /* When: DBCS(*screenp_from) && DBCS(*screenp_to+1) */
  229. +             /* *(screenp_to + 1) = 0; redundant code */
  230.               *(screenp_to + 2) = 0;
  231. +             redraw_next = TRUE;
  232.           }
  233.           }
  234.   #endif
  235. ***************
  236. *** 2049,2069 ****
  237.   # endif
  238.               )
  239.           {
  240. !         int        n;
  241. ! 
  242. !         n = *(screenp_to + Columns);
  243. ! # ifdef MULTI_BYTE
  244. !         if (col + char_bytes < Columns && (n > HL_ALL || (n & HL_BOLD)))
  245. !             *(screenp_to + char_bytes) = 0;
  246. ! # else
  247. !         if (col + 1 < Columns && (n > HL_ALL || (n & HL_BOLD)))
  248. !             *(screenp_to + 1) = 0;
  249. ! # endif
  250.           }
  251.   #endif
  252.           *(screenp_to + Columns) = *(screenp_from + Columns);
  253.   #ifdef MULTI_BYTE
  254. !         if (char_bytes == 2)    /* just a hack */
  255.           *(screenp_to + Columns + 1) = *(screenp_from + Columns);
  256.           screen_char_n(screenp_to, char_bytes, row, col);
  257.   #else
  258. --- 2081,2095 ----
  259.   # endif
  260.               )
  261.           {
  262. !         hl = *(screenp_to + Columns);
  263. !         if (hl > HL_ALL || (hl & HL_BOLD))
  264. !             redraw_next = TRUE;
  265.           }
  266.   #endif
  267.           *(screenp_to + Columns) = *(screenp_from + Columns);
  268.   #ifdef MULTI_BYTE
  269. !         /* just a hack: It makes two bytes of DBCS have same attributes */
  270. !         if (char_bytes == 2)
  271.           *(screenp_to + Columns + 1) = *(screenp_from + Columns);
  272.           screen_char_n(screenp_to, char_bytes, row, col);
  273.   #else
  274. ***************
  275. *** 2090,2104 ****
  276.           }
  277.       }
  278.   
  279. ! #ifdef MULTI_BYTE
  280. !     screenp_to += char_bytes;
  281. !     screenp_from += char_bytes;
  282. !     col += char_bytes;
  283. ! #else
  284. !     ++screenp_to;
  285. !     ++screenp_from;
  286. !     ++col;
  287. ! #endif
  288.       }
  289.   
  290.       if (clear_rest
  291. --- 2116,2124 ----
  292.           }
  293.       }
  294.   
  295. !     screenp_to += CHAR_BYTES;
  296. !     screenp_from += CHAR_BYTES;
  297. !     col += CHAR_BYTES;
  298.       }
  299.   
  300.       if (clear_rest
  301. *** ../vim-5.6.59/src/version.c    Wed Apr  5 16:30:51 2000
  302. --- src/version.c    Wed Apr  5 16:31:16 2000
  303. ***************
  304. *** 420,421 ****
  305. --- 420,423 ----
  306.   {   /* Add new patch number below this line */
  307. + /**/
  308. +     60,
  309.   /**/
  310.  
  311. -- 
  312. SECOND SOLDIER: It could be carried by an African swallow!
  313. FIRST SOLDIER:  Oh  yes! An African swallow maybe ... but not a European
  314.                 swallow. that's my point.
  315.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  316.  
  317. /-/-- Bram Moolenaar --- Bram@moolenaar.net --- http://www.moolenaar.net --\-\
  318. \-\-- Vim: http://www.vim.org ---- ICCF Holland: http://www.vim.org/iccf --/-/
  319.