Hi

Changeset 2266:fb627e94e6c6 introduced a call to update_single_line()
when 'modifiable' option is changed.  At the time this change was needed
since 'modifiable' option had a influence on the way concealable items
were shown.

But now this change is useless I think since 'modifiable' no longer has
an influence on concealable items.  Attached patch rolls back part of
2266:fb627e94e6c6 to avoid a useless redraw when changing option
'modifiable' and to simplify code slightly.

Regards
-- Dominique

-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
diff -r 628147a4b3b2 src/option.c
--- a/src/option.c	Sat Jul 31 20:53:54 2010 +0200
+++ b/src/option.c	Sat Jul 31 22:03:17 2010 +0200
@@ -7516,21 +7516,12 @@
     }
 #endif
 
-#if defined(FEAT_TITLE) || defined(FEAT_CONCEAL)
-    /* when 'modifiable' is changed, redraw the window title and
-     * update current line for concealable items */
-    else if ((int *)varp == &curbuf->b_p_ma)
-    {
-# ifdef FEAT_TITLE
+#ifdef FEAT_TITLE
+     /* when 'modifiable' is changed, redraw the window title */
+     else if ((int *)varp == &curbuf->b_p_ma)
+     {
 	redraw_titles();
-# endif
-# ifdef FEAT_CONCEAL
-	if (curwin->w_p_cole > 0)
-	    update_single_line(curwin, curwin->w_cursor.lnum);
-# endif
-    }
-#endif
-#ifdef FEAT_TITLE
+     }
     /* when 'endofline' is changed, redraw the window title */
     else if ((int *)varp == &curbuf->b_p_eol)
     {

Raspunde prin e-mail lui