diff options
Diffstat (limited to 'runtime/autoload/man.vim')
-rw-r--r-- | runtime/autoload/man.vim | 157 |
1 files changed, 66 insertions, 91 deletions
diff --git a/runtime/autoload/man.vim b/runtime/autoload/man.vim index 6c2d4eae8e..bac88fc99e 100644 --- a/runtime/autoload/man.vim +++ b/runtime/autoload/man.vim @@ -1,19 +1,15 @@ " Maintainer: Anmol Sethi <anmol@aubble.com> -" Ensure Vim is not recursively invoked (man-db does this) -" by forcing man to use cat as the pager. -" More info here http://comments.gmane.org/gmane.editors.vim.devel/29085 if &shell =~# 'fish$' - let s:man_cmd = 'man -P cat ^/dev/null' + let s:man_cmd = 'man ^/dev/null' else - let s:man_cmd = 'man -P cat 2>/dev/null' + let s:man_cmd = 'man 2>/dev/null' endif let s:man_find_arg = "-w" -" TODO(nhooyr) I do not think completion will work on SunOS because I'm not sure if `man -l` -" displays the list of directories that are searched by man for manpages. -" I also do not think Solaris supports the '-P' flag used above and uses only $PAGER. +" TODO(nhooyr) Completion may work on SunOS; I'm not sure if `man -l` displays +" the list of searched directories. try if !has('win32') && $OSTYPE !~? 'cygwin\|linux' && system('uname -s') =~? 'SunOS' && system('uname -r') =~# '^5' let s:man_find_arg = '-l' @@ -22,77 +18,71 @@ catch /E145:/ " Ignore the error in restricted mode endtry -" We need count and count1 to ensure the section was explicitly set -" by the user. count defaults to 0 which is a valid section and -" count1 defaults to 1 which is also a valid section. Only when they -" are equal was the count explicitly set. -function! man#open_page(count, count1, ...) abort +function! man#open_page(count, count1, mods, ...) abort if a:0 > 2 call s:error('too many arguments') return - elseif a:0 ==# 1 - if empty(a:1) + elseif a:0 == 0 + let ref = &filetype ==# 'man' ? expand('<cWORD>') : expand('<cword>') + if empty(ref) call s:error('no identifier under cursor') return endif + elseif a:0 ==# 1 let ref = a:1 else - " We combine the name and sect into a manpage reference so that all + " Combine the name and sect into a manpage reference so that all " verification/extraction can be kept in a single function. " If a:2 is a reference as well, that is fine because it is the only " reference that will match. let ref = a:2.'('.a:1.')' endif try - let [sect, name] = s:extract_sect_and_name_ref(ref) + let [sect, name] = man#extract_sect_and_name_ref(ref) if a:count ==# a:count1 - " user explicitly set a count + " v:count defaults to 0 which is a valid section, and v:count1 defaults to + " 1, also a valid section. If they are equal, count explicitly set. let sect = string(a:count) endif - let [sect, name] = s:verify_exists(sect, name) + let [sect, name, path] = s:verify_exists(sect, name) catch call s:error(v:exception) return endtry call s:push_tag() let bufname = 'man://'.name.(empty(sect)?'':'('.sect.')') - let found_man = s:find_man() - if getbufvar(bufname, 'manwidth') ==# s:manwidth() - if found_man - silent execute 'buf' bufnr(bufname) - else - execute 'split' bufname - endif - keepjumps 1 - return - endif - if found_man - noautocmd execute 'edit' bufname + if a:mods !~# 'tab' && s:find_man() + noautocmd execute 'silent edit' bufname else - noautocmd execute 'split' bufname + noautocmd execute 'silent' a:mods 'split' bufname endif - call s:read_page(sect, name) + let b:man_sect = sect + call s:read_page(path) endfunction function! man#read_page(ref) abort try - let [sect, name] = s:extract_sect_and_name_ref(a:ref) - let [sect, name] = s:verify_exists(sect, name) + let [sect, name] = man#extract_sect_and_name_ref(a:ref) + let [b:man_sect, name, path] = s:verify_exists(sect, name) catch - call s:error(v:exception) + " call to s:error() is unnecessary return endtry - call s:read_page(sect, name) + call s:read_page(path) endfunction -function! s:read_page(sect, name) abort +function! s:read_page(path) abort setlocal modifiable setlocal noreadonly - keepjumps %delete _ - let b:manwidth = s:manwidth() - silent execute 'read!env MANWIDTH='.b:manwidth s:man_cmd s:man_args(a:sect, a:name) + silent keepjumps %delete _ + " Force MANPAGER=cat to ensure Vim is not recursively invoked (by man-db). + " http://comments.gmane.org/gmane.editors.vim.devel/29085 + " Respect $MANWIDTH, or default to window width. + let cmd = 'env MANPAGER=cat'.(empty($MANWIDTH) ? ' MANWIDTH='.winwidth(0) : '') + let cmd .= ' '.s:man_cmd.' '.shellescape(a:path) + silent put =system(cmd) " remove all the backspaced text - silent execute 'keeppatterns keepjumps %substitute,.\b,,e'.(&gdefault?'':'g') + execute 'silent keeppatterns keepjumps %substitute,.\b,,e'.(&gdefault?'':'g') while getline(1) =~# '^\s*$' silent keepjumps 1delete _ endwhile @@ -101,17 +91,17 @@ endfunction " attempt to extract the name and sect out of 'name(sect)' " otherwise just return the largest string of valid characters in ref -function! s:extract_sect_and_name_ref(ref) abort +function! man#extract_sect_and_name_ref(ref) abort if a:ref[0] ==# '-' " try ':Man -pandoc' with this disabled. - throw 'manpage name starts with ''-''' + throw 'manpage name cannot start with ''-''' endif let ref = matchstr(a:ref, '[^()]\+([^()]\+)') if empty(ref) let name = matchstr(a:ref, '[^()]\+') if empty(name) - throw 'manpage reference contains only parantheses' + throw 'manpage reference cannot contain only parentheses' endif - return ['', name] + return [get(b:, 'man_default_sects', ''), name] endif let left = split(ref, '(') " see ':Man 3X curses' on why tolower. @@ -120,21 +110,29 @@ function! s:extract_sect_and_name_ref(ref) abort return [tolower(split(left[1], ')')[0]), left[0]] endfunction -function! s:verify_exists(sect, name) abort - let path = system(s:man_cmd.' '.s:man_find_arg.' '.s:man_args(a:sect, a:name)) - if path !~# '^\/' - if empty(a:sect) - throw 'no manual entry for '.a:name - endif +function! s:get_path(sect, name) abort + if empty(a:sect) let path = system(s:man_cmd.' '.s:man_find_arg.' '.shellescape(a:name)) if path !~# '^\/' - throw 'no manual entry for '.a:name.'('.a:sect.') or '.a:name + throw 'no manual entry for '.a:name endif + return path endif - if a:name =~# '\/' - " We do not need to extract the section/name from the path if the name is - " just a path. - return ['', a:name] + " '-s' flag handles: + " - tokens like 'printf(echo)' + " - sections starting with '-' + " - 3pcap section (found on macOS) + " - commas between sections (for section priority) + return system(s:man_cmd.' '.s:man_find_arg.' -s '.shellescape(a:sect).' '.shellescape(a:name)) +endfunction + +function! s:verify_exists(sect, name) abort + let path = s:get_path(a:sect, a:name) + if path !~# '^\/' + let path = s:get_path(get(b:, 'man_default_sects', ''), a:name) + if path !~# '^\/' + let path = s:get_path('', a:name) + endif endif " We need to extract the section from the path because sometimes " the actual section of the manpage is more specific than the section @@ -142,7 +140,8 @@ function! s:verify_exists(sect, name) abort " Also on linux, it seems that the name is case insensitive. So if one does " ':Man PRIntf', we still want the name of the buffer to be 'printf' or " whatever the correct capitilization is. - return s:extract_sect_and_name_path(path[:len(path)-2]) + let path = path[:len(path)-2] + return s:extract_sect_and_name_path(path) + [path] endfunction let s:tag_stack = [] @@ -158,7 +157,7 @@ endfunction function! man#pop_tag() abort if !empty(s:tag_stack) let tag = remove(s:tag_stack, -1) - execute tag['buf'].'b' + silent execute tag['buf'].'buffer' call cursor(tag['lnum'], tag['col']) endif endfunction @@ -170,13 +169,15 @@ function! s:extract_sect_and_name_path(path) abort let tail = fnamemodify(tail, ':r') endif let sect = matchstr(tail, '\.\zs[^.]\+$') - let name = matchstr(tail, '^.\+\ze\.[^.]\+$') + let name = matchstr(tail, '^.\+\ze\.') return [sect, name] endfunction function! s:find_man() abort if &filetype ==# 'man' return 1 + elseif winnr('$') ==# 1 + return 0 endif let thiswin = winnr() while 1 @@ -189,32 +190,6 @@ function! s:find_man() abort endwhile endfunction -function! s:manwidth() abort - " The reason for respecting $MANWIDTH even if it is wider/smaller than the - " current window is that the current window might only be temporarily - " narrow/wide. Since we don't reflow, we should just assume the - " user knows what they're doing and respect $MANWIDTH. - if empty($MANWIDTH) - " If $MANWIDTH is not set, we do not assign directly to $MANWIDTH because - " then $MANWIDTH will always stay the same value as we only use - " winwidth(0) when $MANWIDTH is empty. Instead we set it locally for the command. - return winwidth(0) - endif - return $MANWIDTH -endfunction - -function! s:man_args(sect, name) abort - if empty(a:sect) - return shellescape(a:name) - endif - " The '-s' flag is very useful. - " We do not need to worry about stuff like 'printf(echo)' - " (two manpages would be interpreted by man without -s) - " We do not need to check if the sect starts with '-' - " Lastly, the 3pcap section on macOS doesn't work without -s - return '-s '.shellescape(a:sect).' '.shellescape(a:name) -endfunction - function! s:error(msg) abort redraw echohl ErrorMsg @@ -224,7 +199,7 @@ endfunction let s:mandirs = join(split(system(s:man_cmd.' '.s:man_find_arg), ':\|\n'), ',') -" see s:extract_sect_and_name_ref on why tolower(sect) +" see man#extract_sect_and_name_ref on why tolower(sect) function! man#complete(arg_lead, cmd_line, cursor_pos) abort let args = split(a:cmd_line) let l = len(args) @@ -271,14 +246,14 @@ function! man#complete(arg_lead, cmd_line, cursor_pos) abort return uniq(sort(map(globpath(s:mandirs,'man?/'.name.'*.'.sect.'*', 0, 1), 's:format_candidate(v:val, sect)'), 'i')) endfunction -function! s:format_candidate(c, sect) abort - if a:c =~# '\.\%(pdf\|in\)$' " invalid extensions +function! s:format_candidate(path, sect) abort + if a:path =~# '\.\%(pdf\|in\)$' " invalid extensions return endif - let [sect, name] = s:extract_sect_and_name_path(a:c) + let [sect, name] = s:extract_sect_and_name_path(a:path) if sect ==# a:sect return name - elseif sect =~# a:sect.'[^.]\+$' + elseif sect =~# a:sect.'.\+$' " We include the section if the user provided section is a prefix " of the actual section. return name.'('.sect.')' |