switch order of method/object in Future::then to match std::bind
authorHans Fugal <fugalh@fb.com>
Tue, 10 Feb 2015 20:32:04 +0000 (12:32 -0800)
committerSara Golemon <sgolemon@fb.com>
Wed, 11 Feb 2015 02:02:00 +0000 (18:02 -0800)
commitd8bc4210d2feacbe01a24cd4fe2ad38a9c966dc8
tree85406e55d6c29c79e03e3fea4e83685afa9fbe75
parente1c576b407c95030c8d1a063bebd27dd6dc89980
switch order of method/object in Future::then to match std::bind

Summary:
I have half a mind to just rip this out and let people use std::bind if they need this. But I won't be so cruel.

Why isn't this just implemented as `then(std::bind(method, object))` anyway? Is the template soup we have now faster?

Test Plan:
Fixed the unit tests to use the new format.
Will look to contbuild to catch all the things this might break (if anyone is using it at all?), and will fix them.

Reviewed By: hannesr@fb.com

Subscribers: trunkagent, exa, folly-diffs@, yfeldblum, jsedgwick, davejwatson

FB internal diff: D1831118

Signature: t1:1831118:1423243771:65db9a89daf14d8bd88331c503ba1ea7ab03b679
folly/futures/Future-inl.h
folly/futures/Future.h
folly/futures/test/FutureTest.cpp
folly/futures/test/Thens.cpp
folly/futures/test/ViaTest.cpp
folly/futures/test/thens.rb