Browse files

Guess it was really meant not to break execution, thus no :throw

  • Loading branch information...
1 parent 04ac1cd commit 5fee544be45455d29ddc8fab0838aad9cb623d68 @ZyX-I ZyX-I committed Jun 18, 2012
Showing with 4 additions and 1 deletion.
  1. +4 −1 doc/vim-addon-manager-getting-started.txt
@@ -153,7 +153,10 @@ recommended setup ~
" VAM install location:
let vam_install_path = expand('$HOME') . '/.vim/vim-addons'
if !EnsureVamIsOnDisk(vam_install_path)
- throw "NO VAM FOUND!"
+ echohl ErrorMsg
+ echomsg "No VAM found!"
+ echohl NONE
+ return
exec 'set runtimepath+='.vam_install_path.'/vim-addon-manager'

2 comments on commit 5fee544


MarcWeber commented on 5fee544 Jun 26, 2012

I don't see a reason why that much fuzz must be made about a error message which should be the exceptional case anyway. The SetupVAM should focus on setting up VAM, not error handling IMHO.

I agree. That's why you should have just gotten rid of the confirmation message to clone VAM (i.e. removing a single line), as I said in #75.

Please sign in to comment.