Thrift: No using namespace.
authorDavid Reiss <dreiss@apache.org>
Thu, 23 Aug 2007 02:37:19 +0000 (02:37 +0000)
committerDavid Reiss <dreiss@apache.org>
Thu, 23 Aug 2007 02:37:19 +0000 (02:37 +0000)
commitd4a269cdb5ecf62e91384e33cdb6bf887f69e9a9
tree013cc9892421f3738e9cda3be933ee12c1f08d53
parent402ee28f87dd855d8825182b79617891a7ac4b93
Thrift: No using namespace.

Summary:
using namespace XXX; is tricky because it imports a lot of crap into your namespace.
Also, if we ever decide to remove boost as a dependency, "using namespace boost"
makes it harder.

This patch removes all using namesapce XXX; with one or more using XXX::YYY;
There are some exceptions, such as test, tutorial, namespace std, and
facebook namespaces.

Reviewed By: mcslee

Test Plan:
Recompiled thrift.
grep --binary-files=without-match -r 'using namespace' . | \
  egrep -v '^[^:]*: *using namespace (std;$|facebook)' | \
  grep -v '<< endl' | \
  grep -v '[^:]*/tutorial/' | \
  grep -v '^./test/'

Revert Plan: ok

Notes: I wouldn't bother, but it is fixathon.

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665218 13f79535-47bb-0310-9956-ffa450edef68
lib/cpp/src/concurrency/PosixThreadFactory.cpp
lib/cpp/src/concurrency/ThreadManager.cpp
lib/cpp/src/transport/TServerSocket.cpp