Use Synchronized in RequestContext
authorYedidya Feldblum <yfeldblum@fb.com>
Mon, 29 Aug 2016 21:01:15 +0000 (14:01 -0700)
committerFacebook Github Bot 1 <facebook-github-bot-1-bot@fb.com>
Mon, 29 Aug 2016 21:08:32 +0000 (14:08 -0700)
commit214d5f7cbe007028ae50d91615ec7ae4ffcdb04a
tree3ce6abf3df0c22e51f46e0ec7bff57401f9cb0f3
parentb90a2ba56027995d17ed3a1d37f895e42ad255cd
Use Synchronized in RequestContext

Summary:
[Folly] Use `Synchronized` in `RequestContext`.

Because we can. And it makes the code a tad simpler and also enforces access correctness a tad.

Also use `folly::make_unique` in `RequestContextTest` to keep balance between the explicit `new` and `delete` ops.

Reviewed By: markisaa

Differential Revision: D3781115

fbshipit-source-id: 63b41ddd8009e9546e3be5f89bdd23a4d791105c
folly/io/async/Request.cpp
folly/io/async/Request.h
folly/io/async/test/RequestContextTest.cpp