projects / folly.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: d80b76c) | patch
(folly/futures) Fix get() bug
authorHans Fugal <fugalh@fb.com>
Fri, 20 Feb 2015 21:15:15 +0000 (13:15 -0800)
committerAlecs King <int@fb.com>
Tue, 3 Mar 2015 03:26:38 +0000 (19:26 -0800)
commit68e16148ecfe41a09f6d3e534368a4d4546e071a
tree61130b230cde93cda00adaa406056120a52f9139tree | snapshot
parentd80b76c761f11142527175473aa70625b6413a04commit | diff
(folly/futures) Fix get() bug

Summary: I thought this was a race, but I think now it was something to do with using a value that had been moved out or something. Anyway, this refactor is cleaner and consolidates a few methods so it's all kinds of fuzzy feelings.

Test Plan: unit tests

Reviewed By: hannesr@fb.com

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

FB internal diff: D1861376

Tasks: 6298004

Signature: t1:1861376:1424465861:736353ab3174656fec9e036e0ebd964970da38d0
folly/futures/Future-inl.h diff | blob | history
folly/futures/test/FutureTest.cpp diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.