You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by mc...@apache.org on 2011/11/10 17:18:28 UTC
svn commit: r1200405 -
/struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java
Author: mcucchiara
Date: Thu Nov 10 16:18:27 2011
New Revision: 1200405
URL: http://svn.apache.org/viewvc?rev=1200405&view=rev
Log:
Removed redundant cast and unnecessary boxing
Modified:
struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java
Modified: struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java
URL: http://svn.apache.org/viewvc/struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java?rev=1200405&r1=1200404&r2=1200405&view=diff
==============================================================================
--- struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java (original)
+++ struts/struts2/trunk/plugins/json/src/main/java/org/apache/struts2/json/JSONReader.java Thu Nov 10 16:18:27 2011
@@ -40,14 +40,14 @@ class JSONReader {
private static Map<Character, Character> escapes = new HashMap<Character, Character>();
static {
- escapes.put(new Character('"'), new Character('"'));
- escapes.put(new Character('\\'), new Character('\\'));
- escapes.put(new Character('/'), new Character('/'));
- escapes.put(new Character('b'), new Character('\b'));
- escapes.put(new Character('f'), new Character('\f'));
- escapes.put(new Character('n'), new Character('\n'));
- escapes.put(new Character('r'), new Character('\r'));
- escapes.put(new Character('t'), new Character('\t'));
+ escapes.put('"', '"');
+ escapes.put('\\', '\\');
+ escapes.put('/', '/');
+ escapes.put('b', '\b');
+ escapes.put('f', '\f');
+ escapes.put('n', '\n');
+ escapes.put('r', '\r');
+ escapes.put('t', '\t');
}
private CharacterIterator it;
@@ -75,7 +75,7 @@ class JSONReader {
}
private Object read() throws JSONException {
- Object ret = null;
+ Object ret;
this.skipWhiteSpace();
@@ -198,8 +198,8 @@ class JSONReader {
this.addDigits();
}
- return (this.buf.indexOf(".") >= 0) ? (Object) Double.parseDouble(this.buf.toString())
- : (Object) Long.parseLong(this.buf.toString());
+ return (this.buf.indexOf(".") >= 0) ? Double.parseDouble(this.buf.toString()) :
+ Long.parseLong(this.buf.toString());
}
private Object string(char quote) {
@@ -212,10 +212,10 @@ class JSONReader {
if (this.c == 'u') {
this.add(this.unicode());
} else {
- Object value = escapes.get(new Character(this.c));
+ Object value = escapes.get(this.c);
if (value != null) {
- this.add(((Character) value).charValue());
+ this.add((Character) value);
}
}
} else {