Another stab at waitWithSemaphore -> Future<T>::wait()
authorJames Sedgwick <jsedgwick@fb.com>
Wed, 28 Jan 2015 16:05:30 +0000 (08:05 -0800)
committerwoo <woo@fb.com>
Mon, 2 Feb 2015 21:12:59 +0000 (13:12 -0800)
commitc4a221ce754b643ce89ce853f1bb3edbc603a735
tree5bee821f14544d91c28e38afa5b86a4cec320782
parentc182b25f363bd80a8043d8616dec9dff8cf8736b
Another stab at waitWithSemaphore -> Future<T>::wait()

Summary:
See D1785572. Check out the new Future test and the commented portion of wait(Duration) for the fix

The test only fails a few times out of a hundred before the fix, but hasn't failed yet after

Test Plan: futures unit, wait for contbuild

Reviewed By: hans@fb.com

Subscribers: trunkagent, rushix, fbcode-common-diffs@, hero-diffs@, cold-storage-diffs@, adamsyta, zhuohuang, darshan, micha, folly-diffs@, lins, tingy, hannesr, jsedgwick

FB internal diff: D1803526

Tasks: 59400086059995

Signature: t1:1803526:1422309486:3613c59a708ecac312d241723828763feb2a57aa
folly/futures/Future-inl.h
folly/futures/Future.h
folly/futures/test/FutureTest.cpp