Skip to main content

This CGIT instance is deprecated, and repositories have been moved to Gitlab or Github. See the repository descriptions for specific locations.

summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordavid_williams2006-06-13 02:43:02 +0000
committerdavid_williams2006-06-13 02:43:02 +0000
commitbf7ad8e397213f6a1864a8625df6b1454c86cb52 (patch)
treed0a3fa04d286b3fde2b000cf80463fb3b702c9f7 /bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core
parent837baf8d23af530f28fb0e53a695b0ae6fe293c4 (diff)
downloadwebtools.sourceediting-bf7ad8e397213f6a1864a8625df6b1454c86cb52.tar.gz
webtools.sourceediting-bf7ad8e397213f6a1864a8625df6b1454c86cb52.tar.xz
webtools.sourceediting-bf7ad8e397213f6a1864a8625df6b1454c86cb52.zip
[146390] String Index Out Of Bounds exception in JSP-CSS parsing -- fixed comment
Diffstat (limited to 'bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core')
-rw-r--r--bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core/internal/document/JSPedCSSModelParser.java7
1 files changed, 4 insertions, 3 deletions
diff --git a/bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core/internal/document/JSPedCSSModelParser.java b/bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core/internal/document/JSPedCSSModelParser.java
index d14f1960b5..0aa723fadb 100644
--- a/bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core/internal/document/JSPedCSSModelParser.java
+++ b/bundles/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/css/core/internal/document/JSPedCSSModelParser.java
@@ -86,9 +86,10 @@ class JSPedCSSModelParser extends CSSModelParser {
}
else {
/*
- * ISSUE: this handles casese where, e.g. "file://" has so
- * subsequent 'value' ... but should it return null or empty
- * string?
+ * ISSUE: this handles cases where, e.g. "file=" has no
+ * subsequent 'value' ... and from code in insertStructuredDocumentRegion
+ * I believe should return null, rather than empty string, but, this may
+ * need some fine tuning eventually.
*/
hrefValue = null;
}

Back to the top