Improve mvim script's finding of MacVim.app #2
Conversation
"/Applications", which is where 99.9% of all users will have their MacVim.app located, was the seventh item to be checked for the presence of MacVim.app.
|
Probably the items in the list after the fourth one should be removed, since it's hard to imagine someone actually puts MacVim.app there. But I left them in for backward compatibility. EDIT: I meant to say, the entries of the form "*/vim"; I suspect those are cruft from the days of the Carbon gui. |
|
Thank you @chdiza, I'm a bit busy doing overtime work this weekend, but as soon as I can I'll look into it and merge. |
|
ping |
Improve mvim script's finding of MacVim.app
|
The relative priorities seem wrong to me in the new script; if you have MacVim.app both in |
|
It's officially advised to not have two copies of MacVim on one's machine. Having two copies has been the cause of plenty of weirdness over the years. |
For some reason, the mvim script had "/Applications" as the seventh item in the list of places to look for MacVim.app. Given that the overwhelming majority of all users are bound to have MacVim.app in /Applications, put it first in the list.
Also, eliminate a needlessly long line by making an array out of the places to check.