From c03a0f5fc82d91cc29e81d02e4440eb8fd9cad12 Mon Sep 17 00:00:00 2001
From: "Justin M. Keyes" <justinkz@gmail.com>
Date: Thu, 25 Oct 2018 03:20:55 +0200
Subject: man.vim: avoid duplicate buffers, E95

Before this commit, man#init_pager() always tries to scrape the manpage
name and set the buffer name. That's much less important than avoiding
duplicate buffers and E95. And it doesn't seem to be necessary, usually.

Steps to reproduce:

    $ export MANPAGER="nvim -c 'set ft=man' -"
    $ man sleep
    :Man sleep

    Error detected while processing function man#init_pager:
    line   15:
    E95: Buffer with this name already exists

    :ls!
    1  h-  "man://SLEEP(1)"               line 4
    2 %a-  "man://sleep(1)"               line 1
---
 runtime/autoload/man.vim | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/runtime/autoload/man.vim b/runtime/autoload/man.vim
index 21f2dfc58a..f4c8194b65 100644
--- a/runtime/autoload/man.vim
+++ b/runtime/autoload/man.vim
@@ -400,7 +400,9 @@ function! man#init_pager() abort
   catch
     let b:man_sect = ''
   endtry
-  execute 'silent file man://'.fnameescape(ref)
+  if -1 == match(bufname('%'), 'man:\/\/')  " Avoid duplicate buffers, E95.
+    execute 'silent file man://'.fnameescape(ref)
+  endif
 endfunction
 
 call s:init()
-- 
cgit