Bug 499613 - StringIndexOutOfBoundsException thrown by class
ScriptSourceHover
Change-Id: Iec4afbe5fa33b8c37ca160023091b283f80c9a1b
Signed-off-by: Thierry BLIND <thierryblind@msn.com>
diff --git a/core/plugins/org.eclipse.dltk.ui/src/org/eclipse/dltk/internal/ui/text/hover/ScriptSourceHover.java b/core/plugins/org.eclipse.dltk.ui/src/org/eclipse/dltk/internal/ui/text/hover/ScriptSourceHover.java
index c49df08..6f7f13e 100644
--- a/core/plugins/org.eclipse.dltk.ui/src/org/eclipse/dltk/internal/ui/text/hover/ScriptSourceHover.java
+++ b/core/plugins/org.eclipse.dltk.ui/src/org/eclipse/dltk/internal/ui/text/hover/ScriptSourceHover.java
@@ -50,25 +50,18 @@
if (source == null)
return null;
+ String[] sourceLines = Strings.convertIntoLines(source);
+ if (sourceLines == null)
+ return null;
+
// source = removeLeadingComments(source);
String delim = System.getProperty("line.separator", "\n"); //$NON-NLS-1$ //$NON-NLS-2$
-
- String[] sourceLines = Strings.convertIntoLines(source);
- String firstLine = sourceLines[0];
- if (!Character.isWhitespace(firstLine.charAt(0)))
- sourceLines[0] = ""; //$NON-NLS-1$
-
- if (!Character.isWhitespace(firstLine.charAt(0)))
- sourceLines[0] = firstLine;
-
source = Strings.concatenate(sourceLines, delim);
return source;
-
} catch (ModelException ex) {
}
}
-
return null;
}