From 881a6ff7d071b4736ad3a565e280ed94dc36fe4c Mon Sep 17 00:00:00 2001
From: "jon.chambers@gmail.com"
 <jon.chambers@gmail.com@b68fe964-5755-0410-a06c-9fee2ea08261>
Date: Sat, 10 Aug 2013 16:08:51 +0000
Subject: [PATCH] Avoided passing temporary strings when serializing a
 collection for a modest performance gain.

---
 src/main/java/org/json/simple/JSONArray.java | 30 ++++++--------------
 1 file changed, 9 insertions(+), 21 deletions(-)

diff --git a/src/main/java/org/json/simple/JSONArray.java b/src/main/java/org/json/simple/JSONArray.java
index aa8e00b..38a791a 100644
--- a/src/main/java/org/json/simple/JSONArray.java
+++ b/src/main/java/org/json/simple/JSONArray.java
@@ -5,6 +5,7 @@
 package org.json.simple;
 
 import java.io.IOException;
+import java.io.StringWriter;
 import java.io.Writer;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -85,29 +86,16 @@ public class JSONArray extends ArrayList implements JSONAware, JSONStreamAware {
 	 * @return JSON text, or "null" if list is null.
 	 */
 	public static String toJSONString(Collection collection){
-		if(collection == null)
-			return "null";
+		final StringWriter writer = new StringWriter();
 		
-        boolean first = true;
-        StringBuffer sb = new StringBuffer();
-		Iterator iter=collection.iterator();
-        
-        sb.append('[');
-		while(iter.hasNext()){
-            if(first)
-                first = false;
-            else
-                sb.append(',');
-            
-			Object value=iter.next();
-			if(value == null){
-				sb.append("null");
-				continue;
-			}
-			sb.append(JSONValue.toJSONString(value));
+		try {
+			writeJSONString(collection, writer);
+		} catch (IOException e) {
+			// This should never happen for a StringWriter
+			throw new RuntimeException(e);
 		}
-        sb.append(']');
-		return sb.toString();
+		
+		return writer.toString();
 	}
 
 	public static void writeJSONString(byte[] array, Writer out) throws IOException{