[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [1839] trunk/org.rubypeople.rdt.ui/src/org/ rubypeople/rdt/internal/ui/text/ruby/hover/RiDocHoverProvider.java

[<caw...@us...>]

Revision: 1839
          http://svn.sourceforge.net/rubyeclipse/?rev=1839&view=rev
Author:   cawilliams
Date:     2007-01-22 11:08:10 -0800 (Mon, 22 Jan 2007)

Log Message:
-----------
do a check to see if ri exists and is a file before trying to execute it

Modified Paths:
--------------
    trunk/org.rubypeople.rdt.ui/src/org/rubypeople/rdt/internal/ui/text/ruby/hover/RiDocHoverProvider.java

Modified: trunk/org.rubypeople.rdt.ui/src/org/rubypeople/rdt/internal/ui/text/ruby/hover/RiDocHoverProvider.java
===================================================================
--- trunk/org.rubypeople.rdt.ui/src/org/rubypeople/rdt/internal/ui/text/ruby/hover/RiDocHoverProvider.java	2007-01-22 18:22:20 UTC (rev 1838)
+++ trunk/org.rubypeople.rdt.ui/src/org/rubypeople/rdt/internal/ui/text/ruby/hover/RiDocHoverProvider.java	2007-01-22 19:08:10 UTC (rev 1839)
@@ -1,6 +1,7 @@
 package org.rubypeople.rdt.internal.ui.text.ruby.hover;
 
 import java.io.BufferedReader;
+import java.io.File;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.util.ArrayList;
@@ -20,6 +21,9 @@
 public class RiDocHoverProvider implements ITextHoverProvider {
 	public String getHoverInfo(IEditorInput input, ITextViewer textViewer, IRegion hoverRegion){
     	IPath riPath = new Path( RubyPlugin.getDefault().getPreferenceStore().getString( PreferenceConstants.RI_PATH ) );
+    	File ri = riPath.toFile();
+    	if (!ri.exists() || !ri.isFile()) return "RI executable doesn't exist at given path";
+    	
     	List<String> args = new ArrayList<String>();
     	args.add(0, riPath.toString());
     	// these will get rid of some of the overhead formatting


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.