b:did_ftplugin doesn't do anything, K is silently overridden

[AndrewRadev]

I'm not entirely sure if this is a mirror from "official" sources, as in, whether this is the place to complain, but I found some problems with these files:

• The b:did_ftplugin lines don't really work, because no let b:did_ftplugin = 1 is ever executed. As a result, with my setup, some of the files are executed twice, since I have both code from github in ~/.vim/bundle and some built-in files from the go arch linux package.

  One solution I implemented in my own fork can be seen here. The fork is not of this repo, because there are several mirrors and I noticed that this one is mentioned in the official documentation after I'd made the changes. They're not very complicated, so I hope this won't be a problem.

• The K mapping is silently overridden, which is an annoying problem with my setup -- I override K myself for something completely different. I believe it would be much better to use maparg to prevent overriding if a mapping exists. This does mean users won't get the mapping in that case, but this could be remedied by a line in the readme mentioning the mapping and telling them to figure something out if they already have K mapped.

  Incidentally, why not just use keywordprg instead of overriding K? Is there a specific reason for that?

[jnwhiteh]

This is just a mirror that's organized properly for use with pathogen. I'd
suggest posting on the gonuts mailing list. I think this should be up to
date with the main repository, but it's worth looking at.
On Oct 26, 2012 5:03 PM, "Andrew Radev" notifications@github.com wrote:

I'm not entirely sure if this is a mirror from "official" sources, as in,
whether this is the place to complain, but I found some problems with these
files:

The b:did_ftplugin lines don't really work, because no let
b:did_ftplugin = 1 is ever executed. As a result, with my setup, some
of the files are executed twice, since I have both code from github in
~/.vim/bundle and some built-in files from the go arch linux package.

One solution I implemented in my own fork can be seen herehttps://github.com/AndrewRadev/go-vim/blob/master/ftplugin/go.vim.
The fork is not of this repo, because there are several mirrors and I
noticed that this one is mentioned in the official documentation after I'd
made the changes. They're not very complicated, so I hope this won't be a
problem.
-

The K mapping is silently overridden, which is an annoying problem
with my setup -- I override K myself for something completely
different. I believe it would be much better to use maparg to prevent
overriding if a mapping exists. This does mean users won't get the mapping
in that case, but this could be remedied by a line in the readme mentioning
the mapping and telling them to figure something out if they already have
K mapped.

Incidentally, why not just use keywordprg instead of overriding K? Is
there a specific reason for that?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/2.

[AndrewRadev]

Thanks, I'll go over there and see what I can find.