Don't typedef types to the same name in different namespaces
authorChristopher Dykes <cdykes@fb.com>
Fri, 22 Jul 2016 17:13:30 +0000 (10:13 -0700)
committerFacebook Github Bot 1 <facebook-github-bot-1-bot@fb.com>
Fri, 22 Jul 2016 17:23:50 +0000 (10:23 -0700)
commit9b2bbca75345932f76311d8d848487ad48f1705d
treeb58ce958df4eb50c61470b9c090853d773df14f8
parent138f44b311c1de18d18eddea8ad7f25b93e975e4
Don't typedef types to the same name in different namespaces

Summary:
This was causing MSVC to complain due to `DSched` already being declared in `BatonTestHelpers.h` as `::folly::DSched`, but it's then typedef'd again in `BatonTest.cpp`, this time as `::DSched`. MSVC complains about the duplicate definitions even though they resolve to the same type, so kill the one in the source file and keep the one in the header.
The same is the case with `GroupVarint32Decoder` and `GroupVarint64Decoder`, which are both declared in `GroupVarint.h` and then again, this time in an anon namespace, in `GroupVarintTest.cpp`.

Reviewed By: yfeldblum

Differential Revision: D3600948

fbshipit-source-id: 401790aa918289fe4a34dd5af683f8c6ca50d847
folly/test/BatonTest.cpp
folly/test/GroupVarintTest.cpp