sapling/eden/fs/takeover
Lee Howes 5b4a9d3362 Replace Future::then(e, f) with Future::then(V/T)Inline(makeAsyncTask(e, f))
Summary:
Remove calls to Future::then(executor, callback).

This form of Future::then is ambiguous, and does not yet implement the stronger
typing of thenValue and thenTry. It is also tempting to use instead of via,
where it is not obvious that it has the behaviour of wrapping a via call in a
push and pop of the current executor:
.pushCurrentExecutor().via(executor).then(callback).popCurrentExecutor().

With the addition of inline continuations, we can instead make the nesting
explicit at low cost by making it an inline continuation that launches an
asynchronous task on the passed executor.

Reviewed By: yfeldblum

Differential Revision: D15487604

fbshipit-source-id: 24b83f56c4f3e8843a6ce8339c70ff4794500d5b
2019-05-26 19:44:34 -07:00
..
test remove SerializedFileHandleMap 2018-12-13 12:29:13 -08:00
CMakeLists.txt eden: handlemap.thrift was removed, but not from cmake build 2018-12-15 13:43:31 -08:00
takeover.thrift Add missing copyright notices 2019-03-07 19:32:39 -08:00
TakeoverClient.cpp Future::onError replaced with Future::thenError 2019-01-30 09:56:15 -08:00
TakeoverClient.h add version handshake to takeover protocol 2018-01-30 14:21:18 -08:00
TakeoverData.cpp remove SerializedFileHandleMap 2018-12-13 12:29:13 -08:00
TakeoverData.h remove SerializedFileHandleMap 2018-12-13 12:29:13 -08:00
TakeoverHandler.h initial code to listen for graceful takeover attempts 2017-11-19 15:47:20 -08:00
TakeoverServer.cpp Replace Future::then(e, f) with Future::then(V/T)Inline(makeAsyncTask(e, f)) 2019-05-26 19:44:34 -07:00
TakeoverServer.h AsyncServerSocket::AcceptCallback::connectionAccepted to NetworkSocket (attempt #2) 2019-04-10 15:01:32 -07:00