Merge pull request #37 from jochenberger/patch-1

UrielCh · web-flow · commit 29dd02d6509f · 2017-03-25T09:32:47.000-07:00
Avoid costly call to String#trim()
diff --git a/json-smart/src/main/java/net/minidev/json/parser/JSONParserString.java b/json-smart/src/main/java/net/minidev/json/parser/JSONParserString.java
@@ -63,8 +63,13 @@ protected void extractString(int beginIndex, int endIndex) {
 	}
 
 	protected void extractStringTrim(int start, int stop) {
+		while (start < stop-1 && Character.isWhitespace(in.charAt(start))) {
+			start++;
+		}
+		while (stop-1 > start && Character.isWhitespace(in.charAt(stop-1))) {
+			stop--;
+		}
 		extractString(start, stop);
-		xs = xs.trim();
 	}
 
 	protected int indexOf(char c, int pos) {

Original file line number	Diff line number	Diff line change
`@@ -63,8 +63,13 @@ protected void extractString(int beginIndex, int endIndex) {`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`protected void extractStringTrim(int start, int stop) {`
	`66`	`+ while (start < stop-1 && Character.isWhitespace(in.charAt(start))) {`
	`67`	`+ start++;`
	`68`	`+ }`
	`69`	`+ while (stop-1 > start && Character.isWhitespace(in.charAt(stop-1))) {`
	`70`	`+ stop--;`
	`71`	`+ }`
`66`	`72`	`extractString(start, stop);`
`67`		`- xs = xs.trim();`
`68`	`73`	`}`
`69`	`74`
`70`	`75`	`protected int indexOf(char c, int pos) {`