Remove allocation of enumerator

yupliner · yupliner · commit a02a0b18b8e0 · 2018-09-20T12:36:56.000+05:00
diff --git a/projects/client/RabbitMQ.Client/src/client/impl/SocketFrameHandler.cs b/projects/client/RabbitMQ.Client/src/client/impl/SocketFrameHandler.cs
@@ -238,7 +238,10 @@ public void WriteFrameSet(IList<OutboundFrame> frames)
         {
             var ms = new MemoryStream();
             var nbw = new NetworkBinaryWriter(ms);
-            foreach (var f in frames) f.WriteTo(nbw);
+            for (var i = 0; i < frames.Count; ++i)
+            {
+                frames[i].WriteTo(nbw);
+            }
             m_socket.Client.Poll(m_writeableStateTimeout, SelectMode.SelectWrite);
             Write(ms.GetBufferSegment());
         }

Original file line number	Diff line number	Diff line change
`@@ -238,7 +238,10 @@ public void WriteFrameSet(IList<OutboundFrame> frames)`
`238`	`238`	`{`
`239`	`239`	`var ms = new MemoryStream();`
`240`	`240`	`var nbw = new NetworkBinaryWriter(ms);`
`241`		`- foreach (var f in frames) f.WriteTo(nbw);`
	`241`	`+ for (var i = 0; i < frames.Count; ++i)`
	`242`	`+ {`
	`243`	`+ frames[i].WriteTo(nbw);`
	`244`	`+ }`
`242`	`245`	`m_socket.Client.Poll(m_writeableStateTimeout, SelectMode.SelectWrite);`
`243`	`246`	`Write(ms.GetBufferSegment());`
`244`	`247`	`}`