PATCH: default installation directories

Doug Kearns dougkearns at gmail.com
Mon Aug 15 11:58:10 EDT 2005


On Mon, Aug 15, 2005 at 04:29:36PM +0100, Hugh Sasse wrote:
> On Tue, 16 Aug 2005, Doug Kearns wrote:
> 
> >This is just a quick patch to use $VIM/vimfiles and
> >$HOME/{.vim,vimfiles} rather than $VIMRUNTIME as the two default
> >installation directories offered by vim-ruby-install.rb.
> 
> Looks like what I'd wish for.  I'll `cvs update -d` and try this.

I thought you'd be glued to the cricket Hugh? ;-)
 
> >-      end
> >-    end
> >-      # 3. We don't get here; every path contains a return statement.
> >+    entered_directory = Env.ask_user "Please enter the full path to your 
> >Vim installation directory: "
> >+    entered_directory = File.expand_path(entered_directory)
> >+    return entered_directory
> >+      # 4. We don't get here; every path contains a return statement.
> >  end
> 
> Isn't the statement # 4 redundant now that it is *straight after* a
> return statement? 

Yes.

> (I suppose it should have been a raise or assert before, but
> still....)

Regards,
Doug


More information about the vim-ruby-devel mailing list