Code optimisation

Author: subyssurendran666

org.eclipse.jdt.core.compiler.batch/src/org/eclipse/jdt/internal/compiler/parser/AbstractCommentParser.java

@@ -1394,47 +1394,29 @@ protected boolean parseReference() throws InvalidInputException {
 	}
 
 	// Parses a complete URL reference starting from current position
-	protected boolean parseURLReference() throws InvalidInputException {
-		int pos = 0;
+	protected boolean parseURLReference(int pos, boolean advanceEndPos) throws InvalidInputException {
 		char[]fullURL = null;
 		int firstTokenStartPos;
 		StringBuilder urlBuilder = new StringBuilder();
 		char c;
-		if (this.source[this.index] == '(') {
-			firstTokenStartPos = this.index;
-			while (pos < this.source.length) {
-				c =  readChar();
-				if (c == '[') // invalid syntax for url
-					return false;
-				if (c == '(' || c == ' ')
-					continue;
-
-		        if (c == '\n' || c == '\r' || c == ')' ) {
-		            break;
-		        }
-		        urlBuilder.append(c);
-			}
-			pos = this.index - 1;
-			fullURL = urlBuilder.toString().toCharArray();
-		} else {
-			pos = this.scanner.startPosition - 1;
-			firstTokenStartPos = pos;
-			while (pos < this.source.length) {
-				c = this.source[pos];
-				if (c == '[') // invalid syntax for url
-					return false;
-				if (c == '(' || c == ' ') {
-					pos++;
-					continue;
-				}
-				if (c == '\n' || c == '\r' || c == ')') {
-		            break;
-		        }
-		        urlBuilder.append(c);
-		        pos++;
+		firstTokenStartPos = pos;
+		while (pos < this.source.length) {
+			c = this.source[pos];
+			if (c == '[') // invalid syntax for url
+				return false;
+			if (c == '(' || c == ' ') {
+				pos++;
+				continue;
 			}
-			fullURL = urlBuilder.toString().toCharArray();
-		}
+			if (c == '\n' || c == '\r' || c == ')') {
+	            break;
+	        }
+	        urlBuilder.append(c);
+	        pos++;
+		}
+		if (advanceEndPos)
+			this.index = pos;
+		fullURL = urlBuilder.toString().toCharArray();
 
 		this.identifierPtr = 0;
 		this.identifierStack[this.identifierPtr] =  fullURL;

org.eclipse.jdt.core.compiler.batch/src/org/eclipse/jdt/internal/compiler/parser/JavadocParser.java

@@ -582,7 +582,7 @@ protected boolean parseMarkdownLinks(int previousPosition) throws InvalidInputEx
 						currentChar = readChar();
 						start = this.index;
 					} else if (peekChar() == '(') {
-						valid = parseURLReference();
+						valid = parseURLReference(this.index, true);
 					} else {
 						break loop;
 					}

org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/DocCommentParser.java

@@ -772,7 +772,7 @@ private boolean parseMarkdownLinkTags(boolean refFlag, int start, int previousPo
 		this.tagValue = TAG_LINK_VALUE;
 		int indexBkup = this.index;
 		if (refFlag)
-			valid = parseURLReference();
+			valid = parseURLReference(this.scanner.startPosition - 1, false);
 		else
 			valid = parseReference(true);
 		this.index = indexBkup;