reduce cost of initializing 'batch' in ProcessInternal by 8% when isSignleRpc is true

Austin Harris · Austin Harris · commit 87d61b7cf021 · 2015-01-30T15:11:05.000-07:00
diff --git a/Json-Rpc/JsonRpcProcessor.cs b/Json-Rpc/JsonRpcProcessor.cs
@@ -7,6 +7,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
+using Newtonsoft.Json;
 
 namespace AustinHarris.JsonRpc
 {
@@ -60,18 +61,18 @@ private static string ProcessInternal(string sessionId, string jsonRpc, object j
 
             try
             {
+                Tuple<JsonRequest, JsonResponse>[] batch = null;
                 if (isSingleRpc(jsonRpc))
                 {
-                    var sbAddBrackets = new StringBuilder("[", jsonRpc.Length + 2);
-                    sbAddBrackets.Append(jsonRpc);
-                    sbAddBrackets.Append("]");
-                    jsonRpc = sbAddBrackets.ToString();
+                    batch = new [] { Tuple.Create(JsonConvert.DeserializeObject<JsonRequest>(jsonRpc), new JsonResponse()) };
                 }
-
-                var batch =
-                    Newtonsoft.Json.JsonConvert.DeserializeObject<JsonRequest[]>(jsonRpc)
-                        .Select(request => new Tuple<JsonRequest, JsonResponse>(request, new JsonResponse())).ToArray();
-
+                else
+                {
+                    batch = JsonConvert.DeserializeObject<JsonRequest[]>(jsonRpc)
+                            .Select(request => new Tuple<JsonRequest, JsonResponse>(request, new JsonResponse()))
+                            .ToArray();
+                }
+            
                 if (batch.Length == 0)
                 {
                     return Newtonsoft.Json.JsonConvert.SerializeObject(new JsonResponse
