# HG changeset patch # User Christian Brabandt # Date 1466415006 -7200 # Node ID cd9c4bbe1d033c4f001a17d7f9f374776f3a4fcc # Parent 710e5276a348d913df38bca808a32d227e465732 commit https://github.com/vim/vim/commit/ddf8d1c746ee081d15c9d7e0515f6ac43adbf363 Author: Bram Moolenaar Date: Mon Jun 20 11:22:54 2016 +0200 patch 7.4.1945 Problem: The Man plugin doesn't work that well. Solution: Use "g:ft_man_open_mode" to be able open man pages in vert split or separate tab. Set nomodifiable for buffer with man content. Add a test. (Andrey Starodubtsev, closes #873) diff --git a/runtime/ftplugin/man.vim b/runtime/ftplugin/man.vim --- a/runtime/ftplugin/man.vim +++ b/runtime/ftplugin/man.vim @@ -1,7 +1,7 @@ " Vim filetype plugin file " Language: man " Maintainer: SungHyun Nam -" Last Change: 2016 Feb 04 +" Last Change: 2016 Jun 20 " To make the ":Man" command available before editing a manual page, source " this script from your startup vimrc file. @@ -150,7 +150,17 @@ func GetPage(...) endwhile endif if &filetype != "man" - new + if exists("g:ft_man_open_mode") + if g:ft_man_open_mode == "vert" + vnew + elseif g:ft_man_open_mode == "tab" + tabnew + else + new + endif + else + new + endif setl nonu fdc=0 endif endif @@ -160,10 +170,15 @@ func GetPage(...) setl ma nonu nornu nofen silent exec "norm 1GdG" + let unsetwidth = 0 if empty($MANWIDTH) let $MANWIDTH = winwidth(0) + let unsetwidth = 1 endif silent exec "r!/usr/bin/man ".s:GetCmdArg(sect, page)." | col -b" + if unsetwidth + let $MANWIDTH = '' + endif " Remove blank lines from top and bottom. while getline(1) =~ '^\s*$' silent keepj norm ggdd @@ -175,6 +190,7 @@ func GetPage(...) setl ft=man nomod setl bufhidden=hide setl nobuflisted + setl noma endfunc func PopPage() @@ -195,4 +211,4 @@ endfunc endif -" vim: set sw=2: +" vim: set sw=2 ts=8 noet: diff --git a/src/testdir/Make_all.mak b/src/testdir/Make_all.mak --- a/src/testdir/Make_all.mak +++ b/src/testdir/Make_all.mak @@ -175,6 +175,7 @@ NEW_TESTS = test_arglist.res \ test_increment.res \ test_json.res \ test_langmap.res \ + test_man.res \ test_matchadd_conceal.res \ test_packadd.res \ test_perl.res \ diff --git a/src/testdir/test_man.vim b/src/testdir/test_man.vim new file mode 100644 --- /dev/null +++ b/src/testdir/test_man.vim @@ -0,0 +1,59 @@ +runtime ftplugin/man.vim + +function Test_g_ft_man_open_mode() + let l:w = winwidth(1) + vnew + let l:h = winheight(1) + q + + " split horizontally + let wincnt = winnr('$') + Man 'vim' + if wincnt == winnr('$') + " Vim manual page cannot be found. + return + endif + call assert_equal(l:w, winwidth(1)) + call assert_true(l:h > winheight(1)) + call assert_equal(1, tabpagenr('$')) + call assert_equal(1, tabpagenr()) + q + + " split horizontally + let g:ft_man_open_mode = "horz" + Man 'vim' + call assert_equal(l:w, winwidth(1)) + call assert_true(l:h > winheight(1)) + call assert_equal(1, tabpagenr('$')) + call assert_equal(1, tabpagenr()) + q + + " split vertically + let g:ft_man_open_mode = "vert" + Man 'vim' + call assert_true(l:w > winwidth(1)) + call assert_equal(l:h, winheight(1)) + call assert_equal(1, tabpagenr('$')) + call assert_equal(1, tabpagenr()) + q + + " separate tab + let g:ft_man_open_mode = "tab" + Man 'vim' + call assert_equal(l:w, winwidth(1)) + call assert_equal(l:h, winheight(1)) + call assert_equal(2, tabpagenr('$')) + call assert_equal(2, tabpagenr()) + q +endfunction + +function Test_nomodifiable() + let wincnt = winnr('$') + Man 'vim' + if wincnt == winnr('$') + " Vim manual page cannot be found. + return + endif + call assert_false(&l:modifiable) + q +endfunction diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -754,6 +754,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1945, +/**/ 1944, /**/ 1943,