[vim/vim] runtime(c): set 'keywordprg' to :Man (PR #19222)
Mateo Gjika
This is mostly useful for the gui since :!man in the builtin terminal is unusable. Maybe it should be done only when the gui is running? Although I think also terminal users would prefer the man.vim plugin
I also added an option to man to only search in relevant sections, for example man -s2,3 open opens the man page for the syscall instead of the user program with the same name.
You can view, comment on, or merge this pull request online at:
https://github.com/vim/vim/pull/19222
Commit Summary
- f6da64b runtime(c): set 'keywordprg' to :Man
File Changes
(1 file)
- M runtime/ftplugin/c.vim (11)
Patch Links:
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Christian Brabandt
chrisbra left a comment (vim/vim#19222)I don't think this makes sense. You can already use <count>K on any function to open the manpage for the identifier under the cursor (in section <count>) it. This sounds more like something for user configuration, but I don't think we should add this by default.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Maxim Kim
habamax left a comment (vim/vim#19222)Even if it was considered for inclusion, sourcing man.vim every time c filetype is detected is wrong.
I agree, this should be user configuration.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
mateoxh left a comment (vim/vim#19222)I agree with both of you, I didn't think this through.
But K is still not usable in gui, maybe a solution is sourcing man.vim somewhere else and add an autocmd in c.vim?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
mateoxh left a comment (vim/vim#19222)I opted for another approach using :term but only for the gui. I added ++close because closing the terminal window after exiting the pager was annoying, the downside is that in case of an error nothing is shown.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
@mateoxh pushed 1 commit.
- 8c9900f add check for terminal feature
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
dkearns
dkearns left a comment (vim/vim#19222)We already have this sort of behaviour implemented in some other ftplugins. E.g.,
The approach should probably be generalised.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Mateo Gjika
mateoxh left a comment (vim/vim#19222)Yes, same thing for go
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
