ListenerSecondary - reduce closure allocation

src/Microsoft.AspNet.Server.Kestrel/Http/ListenerSecondary.cs

@@ -17,6 +17,8 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
     /// </summary>
     public abstract class ListenerSecondary : ListenerContext, IDisposable
     {
+        private string _pipeName;
+
         protected ListenerSecondary(ServiceContext serviceContext) : base(serviceContext)
         {
         }
@@ -29,14 +31,25 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
             KestrelThread thread,
             RequestDelegate application)
         {
+            _pipeName = pipeName;
             ServerAddress = address;
             Thread = thread;
             Application = application;
 
             DispatchPipe = new UvPipeHandle(Log);
 
-            var tcs = new TaskCompletionSource<int>();
+            var tcs = new TaskCompletionSource<int>(this);
-            Thread.Post(_ =>
+            Thread.Post(tcs2 => StartCallback(tcs2), tcs);
+            return tcs.Task;
+        }
+
+        private static void StartCallback(TaskCompletionSource<int> tcs)
+        {
+            var listener = (ListenerSecondary)tcs.Task.AsyncState;
+            listener.StartedCallback(tcs);
+        }
+
+        private void StartedCallback(TaskCompletionSource<int> tcs)
         {
             try
             {
@@ -45,8 +58,23 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
                 connect.Init(Thread.Loop);
                 connect.Connect(
                     DispatchPipe,
-                        pipeName,
+                    _pipeName,
-                        (connect2, status, error, state) =>
+                    (connect2, status, error, state) => ConnectCallback(connect2, status, error, (TaskCompletionSource<int>)state),
+                    tcs);
+            }
+            catch (Exception ex)
+            {
+                DispatchPipe.Dispose();
+                tcs.SetException(ex);
+            }
+        }
+
+        private static void ConnectCallback(UvConnectRequest connect, int status, Exception error, TaskCompletionSource<int> tcs)
+        {
+            var listener = (ListenerSecondary)tcs.Task.AsyncState;
+            listener.ConnectedCallback(connect, status, error, tcs);
+        }
+        private void ConnectedCallback(UvConnectRequest connect, int status, Exception error, TaskCompletionSource<int> tcs)
         {
             connect.Dispose();
             if (error != null)
@@ -61,15 +89,26 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
                 var buf = Thread.Loop.Libuv.buf_init(ptr, 4);
 
                 DispatchPipe.ReadStart(
-                                    (_1, _2, _3) => buf,
+                    (handle, status2, state) => buf,
-                                    (_1, status2, state2) =>
+                    (handle, status2, state) => ((ListenerSecondary)state).ReadStartCallback(handle, status2, ptr), this);
+
+                tcs.SetResult(0);
+            }
+            catch (Exception ex)
             {
-                                        if (status2 < 0)
+                DispatchPipe.Dispose();
+                tcs.SetException(ex);
+            }
+        }
+
+        private void ReadStartCallback(UvStreamHandle handle, int status, IntPtr ptr)
         {
-                                            if (status2 != Constants.EOF)
+            if (status < 0)
+            {
+                if (status != Constants.EOF)
                 {
                     Exception ex;
-                                                Thread.Loop.Libuv.Check(status2, out ex);
+                    Thread.Loop.Libuv.Check(status, out ex);
                     Log.LogError("DispatchPipe.ReadStart", ex);
                 }
 
@@ -98,26 +137,6 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
 
             var connection = new Connection(this, acceptSocket);
             connection.Start();
-                                    },
-                                    null);
-
-                                tcs.SetResult(0);
-                            }
-                            catch (Exception ex)
-                            {
-                                DispatchPipe.Dispose();
-                                tcs.SetException(ex);
-                            }
-                        },
-                        null);
-                }
-                catch (Exception ex)
-                {
-                    DispatchPipe.Dispose();
-                    tcs.SetException(ex);
-                }
-            }, null);
-            return tcs.Task;
         }
 
         /// <summary>
@@ -133,7 +152,7 @@ namespace Microsoft.AspNet.Server.Kestrel.Http
             // the exception that stopped the event loop will never be surfaced.
             if (Thread.FatalError == null)
             {
-                Thread.Send(_ => DispatchPipe.Dispose(), null);
+                Thread.Send(listener => ((ListenerSecondary)listener).DispatchPipe.Dispose(), this);
             }
         }
     }