Don't require folly::Unit in addTaskFinally
authorAndrii Grynenko <andrii@fb.com>
Sat, 30 Apr 2016 00:50:50 +0000 (17:50 -0700)
committerFacebook Github Bot 8 <facebook-github-bot-8-bot@fb.com>
Sat, 30 Apr 2016 01:05:24 +0000 (18:05 -0700)
commitebc7dd0de112dba0d72d297400b031708ba8f970
treeed7f78d7ef08bfa877bf44c8658f16dbf73fedf7
parentb31eb722e444ab0293a73fe9de3f94e657ca6de9
Don't require folly::Unit in addTaskFinally

Summary:
In D3241498 addTaskFuture was fixed to work with function returning void. This however changed addTaskFinally API to not allow finally functor which accepts Try<void>.

folly::fibers generally supports Try<void> so there's no reason to force users to use folly::Unit instead of void for addTaskFinally too.

Reviewed By: yfeldblum

Differential Revision: D3243893

fb-gh-sync-id: d1df54738157d2019120103956f59b3971ba25ff
fbshipit-source-id: d1df54738157d2019120103956f59b3971ba25ff
folly/experimental/fibers/FiberManager-inl.h
folly/futures/Try.h