opendev/gerrit
Code Issues Proposed changes

Remove unused parseJSON method

Browse Source 
Gerrit stopped using its custom parseJSON helper and started using
JSONParser.parseStrict instead in v2.6-rc0~778 (Fix JSON parser in
RestApi client, 2012-11-21).

Change-Id: I1be4a481951a87bb3a9d255d238aef638d2ab9b5
This commit is contained in:
Jonathan Nieder 2015-06-03 19:11:00 -07:00 committed by David Pursehouse
parent 3da3b8d05d
commit d2b80ce67d
1 changed files with 0 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
gerrit-gwtui-common/src/main/java/com/google/gerrit/client/rpc/Natives.java
View File

@ -91,34 +91,6 @@ public class Natives {
return arr;
}
@SuppressWarnings("unchecked")
public static <T extends JavaScriptObject> T parseJSON(String json) {
if (json.startsWith("\"")) {
return (T) NativeString.wrap(parseString(parser, json));
}
return Natives.<T> parseObject(parser, json); // javac generics bug
}
private static native <T extends JavaScriptObject>
T parseObject(JavaScriptObject p, String s)
/*-{ return p(s); }-*/;
private static native
String parseString(JavaScriptObject p, String s)
/*-{ return p(s); }-*/;
private static JavaScriptObject parser;
private static native JavaScriptObject bestJsonParser()
/*-{
if ($wnd.JSON && typeof $wnd.JSON.parse === 'function')
return $wnd.JSON.parse;
return function(s) { return eval('(' + s + ')'); };
}-*/;
static {
parser = bestJsonParser();
}
private Natives() {
}
}