refactor, moveonly: order lambda move captures first

Followup for #88

Author: ryanofsky

include/mp/proxy-io.h

@@ -312,7 +312,7 @@ class Connection
         // to the EventLoop TaskSet to avoid "Promise callback destroyed itself"
         // error in cases where f deletes this Connection object.
         m_on_disconnect.add(m_network.onDisconnect().then(
-            [this, f = std::move(f)]() mutable { m_loop.m_task_set->add(kj::evalLater(kj::mv(f))); }));
+            [f = std::move(f), this]() mutable { m_loop.m_task_set->add(kj::evalLater(kj::mv(f))); }));
     }
 
     EventLoop& m_loop;

include/mp/proxy-types.h

@@ -112,9 +112,8 @@ auto PassField(Priority<1>, TypeList<>, ServerContext& server_context, const Fn&
     auto& server = server_context.proxy_server;
     int req = server_context.req;
     auto invoke = MakeAsyncCallable(
-        [&server, req, fn, args...,
-         fulfiller = kj::mv(future.fulfiller),
-         call_context = kj::mv(server_context.call_context)]() mutable {
+        [fulfiller = kj::mv(future.fulfiller),
+         call_context = kj::mv(server_context.call_context), &server, req, fn, args...]() mutable {
                 const auto& params = call_context.getParams();
                 Context::Reader context_arg = Accessor::get(params);
                 ServerContext server_context{server, call_context, req};