From: Lee Howes <lwh@fb.com>
Date: Fri, 3 Nov 2017 23:12:29 +0000 (-0700)
Subject: Parameter order
X-Git-Tag: v2017.11.06.00~4
X-Git-Url: http://plrg.eecs.uci.edu/git/?p=folly.git;a=commitdiff_plain;h=4780814a0fc73b5bd186e56de93396a09fa15337

Parameter order

Summary: Fix for function parameter order that shows up in opt build on later diff.

Reviewed By: andriigrynenko

Differential Revision: D6237125

fbshipit-source-id: fbb7be2c70b32203c658fc239cd74164e01fa1ca
---

diff --git a/folly/futures/Future-inl.h b/folly/futures/Future-inl.h
index 1baee146..20cf87bc 100644
--- a/folly/futures/Future-inl.h
+++ b/folly/futures/Future-inl.h
@@ -523,7 +523,7 @@ SemiFuture<T>::defer(F&& func) && {
   // case
   auto sf =
       std::move(*this)
-          .via(defKeepAlive.get())
+          .via(e)
           // Then add the work, with a wrapper function that captures the
           // keepAlive so the executor is destroyed at the right time.
           .then(