svn commit: r1803589 - /felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java

[dleangen]

Author: dleangen
Date: Mon Jul 31 22:14:01 2017
New Revision: 1803589

URL: http://svn.apache.org/viewvc?rev=1803589&view=rev
Log:
Added small optimization to Serializer

Modified:
    
felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java

Modified: 
felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java?rev=1803589&r1=1803588&r2=1803589&view=diff
==============================================================================
--- 
felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java
 (original)
+++ 
felix/trunk/converter/serializer/src/main/java/org/apache/felix/serializer/impl/json/JsonSerializingImpl.java
 Mon Jul 31 22:14:01 2017
@@ -81,7 +81,14 @@ public class JsonSerializingImpl impleme
             return ignoreNull ? "" : "null";
         }
 
-        if (obj instanceof Map) {
+        if (obj instanceof String) {
+            // Optimization for when the value is already a String
+            // David B.: is this ok? Or does the Converter do something else
+            //           other than just returning a String??
+            //           I noticed that a lot of calculations were going on, 
just
+            //           to return a String anyway.
+            return "\"" + (String)obj + "\"";
+        } else if (obj instanceof Map) {
             return encodeMap((Map) obj);
         } else if (obj instanceof Collection) {
             return encodeCollection((Collection) obj);