[jbosstools-commits] JBoss Tools SVN: r42489 - trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util.

Author: yradtsevich Date: 2012-07-10 14:14:58 -0400 (Tue, 10 Jul 2012) New Revision: 42489 Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java Log: https://issues.jboss.org/browse/JBIDE-12308 : VPE throws StackOverflowError and visual part is blank Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java =================================================================== --- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java 2012-07-10 15:22:19 UTC (rev 42488) +++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/util/VpeStyleUtil.java 2012-07-10 18:14:58 UTC (rev 42489) @@ -89,7 +89,8 @@ /* * Pattern "|(//.*)" could be added at the end to remove single line comments. */ - public static final Pattern CSS_COMMENT_PATTERN = Pattern.compile("(/\\*([^*]|["+LINE_SEPARATOR+"]|(\\*+([^*/]|["+LINE_SEPARATOR+"])))*\\*+/)"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ + private static final String CSS_COMMENT_END = "*/"; //$NON-NLS-1$ + private static final String CSS_COMMENT_START = "/*"; //$NON-NLS-1$ public static String ATTR_URL = "url"; //$NON-NLS-1$ public static String OPEN_BRACKET = "("; //$NON-NLS-1$ @@ -949,12 +950,27 @@ } /** - * Removes all the CSS comments from the text + * Removes all the CSS comments from the text. * - * @param cssText the css text + * @param css the CSS text * @return updated string */ public static String removeAllCssComments(String cssText) { - return CSS_COMMENT_PATTERN.matcher(cssText).replaceAll(Constants.EMPTY); + // It is too expensive to use regular expressions here, this is why they are not used. See JBIDE-12308 + + StringBuilder cssBuilder = new StringBuilder(cssText); + int curIndex = 0; + int commentStartIndex; + while ((commentStartIndex = cssBuilder.indexOf(CSS_COMMENT_START, curIndex)) >= 0) { + int afterCommentEndIndex = cssBuilder.indexOf(CSS_COMMENT_END, commentStartIndex + CSS_COMMENT_START.length()); + if (afterCommentEndIndex >= 0) { + afterCommentEndIndex += CSS_COMMENT_END.length(); + } else { + afterCommentEndIndex = cssBuilder.length(); + } + cssBuilder.replace(commentStartIndex, afterCommentEndIndex, " "); //$NON-NLS-1$ + curIndex = commentStartIndex + 1; + } + return cssBuilder.toString(); } } \ No newline at end of file