GlobalExecutor.h include Executor.h and IOExecutor.h
authorHans Fugal <fugalh@fb.com>
Tue, 7 Apr 2015 01:20:00 +0000 (18:20 -0700)
committerViswanath Sivakumar <viswanath@fb.com>
Fri, 10 Apr 2015 03:34:11 +0000 (20:34 -0700)
commitec1a66c587f7b3223f154233874bd6db8a9a118f
tree305d1739acddb6f7bab232a74a9ecee883a249f4
parent0978159b6279d9484a6612c5e46193376fd83820
GlobalExecutor.h include Executor.h and IOExecutor.h

Summary: We were forward-declaring `Executor` and `IOExecutor` for great compilation speed. However, this leaves users of this header to include at least one of those headers in their cpp file to actually use it, and that's kind of annoying. `Executor` and `IOExecutor` are not templated and are quite small, so it seems like the compilation speed benefit is minimal compared to the convenience factor, so this diff just includes the two headers. (Probably including `Executor.h` is redundant since `IOExecutor.h` probably includes it, but for completeness and explicitness, and in case `IOExecutor` also just forward-declares `Executor` now or in the future.)

Test Plan: stuff still builds

Reviewed By: davejwatson@fb.com

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

FB internal diff: D1970179

Signature: t1:1970179:1428351076:e9c3700a868688035e7ff50486b8cead751d3c02
folly/wangle/concurrent/GlobalExecutor.h