Bug#487120: vim-runtime: Screen is not redrawn after running external file handler from netrw plugin

[Teemu Likonen]

Package: vim-runtime
Version: 1:7.1.314-3
Severity: normal
Tags: patch

The netrw explorer has a possibility to view files with external program
(the "x" key in the explorer). Such external programs are launched with
a command like

  exe "silent !kfmclient exec " [...]

Problem is that Vim's screen is not redrawn after the operation. For
example after running kfmclient (KDE's general file handler) Vim's
screen is pretty much messed up. Vim's manual itself warns about the
possibility after :silent command (see :help :silent). Previously netrw
plugin ran :redraw! but for some reason the line in the code was
commented out later. This patch resurrects it.

Signed-off-by: Teemu Likonen <tlikonen at iki.fi>


-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.18-6-k7 (SMP w/1 CPU core)
Locale: LANG=fi_FI.UTF-8, LC_CTYPE=fi_FI.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

vim-runtime depends on no packages.

Versions of packages vim-runtime recommends:
ii  vim-gtk [vim]                1:7.1.314-3 Vi IMproved - enhanced vi editor -

-- no debconf information
-------------- next part --------------
>From ea71352db7d134fce778eb148efae37284f40a3a Mon Sep 17 00:00:00 2001
From: Teemu Likonen <tlikonen at iki.fi>
Date: Thu, 19 Jun 2008 20:08:11 +0300
Subject: [PATCH] Redraw screen after running external file handler from netrw explorer

The netrw explorer has a possibility to view files with external program
(the "x" key in the explorer). Such external programs are launched with
a command like

  exe "silent !kfmclient exec " [...]

Problem is that Vim's screen is not redrawn after the operation. For
example after running kfmclient (KDE's general file handler) Vim's
screen is pretty much messed up. Vim's manual itself warns about the
possibility after :silent command (see :help :silent). Previously netrw
plugin ran :redraw! but for some reason the line in the code was
commented out later. This patch resurrects it.

Signed-off-by: Teemu Likonen <tlikonen at iki.fi>
---
 runtime/autoload/netrw.vim |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/runtime/autoload/netrw.vim b/runtime/autoload/netrw.vim
index 6064a65..a22b44e 100644
--- a/runtime/autoload/netrw.vim
+++ b/runtime/autoload/netrw.vim
@@ -3007,7 +3007,7 @@ fun! netrw#NetrwBrowseX(fname,remote)
    let ret= netrwFileHandlers#Invoke(exten,fname)
   endif
 
-"  redraw!
+  redraw!
 
   " cleanup: remove temporary file,
   "          delete current buffer if success with handler,
-- 
1.5.6.2.g3c3a