Skip to content

Commit 54b97a1

Browse files
generatedunixname89002005287564facebook-github-bot
authored andcommitted
Fix CQS signal. Id] 84479791 -- modernize-concat-nested-namespaces in fbcode/thrift/lib/cpp/concurrency [B]
Reviewed By: dtolnay Differential Revision: D74924218 fbshipit-source-id: 7cffbc748c4f9871dd2fafbdb876fd32fe37a57d
1 parent ca71c64 commit 54b97a1

File tree

4 files changed

+8
-26
lines changed

4 files changed

+8
-26
lines changed

thrift/lib/cpp/concurrency/SFQThreadManager.cpp

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,7 @@
2626
#include <thrift/lib/cpp/concurrency/SFQThreadManager.h>
2727
#include <thrift/lib/cpp/concurrency/ThreadManager.h>
2828

29-
namespace apache {
30-
namespace thrift {
31-
namespace concurrency {
29+
namespace apache::thrift::concurrency {
3230

3331
static constexpr uint64_t kDefaultTenantId{0};
3432

@@ -97,6 +95,4 @@ size_t SFQThreadManager::pendingUpstreamTaskCount() const {
9795
return count;
9896
}
9997

100-
} // namespace concurrency
101-
} // namespace thrift
102-
} // namespace apache
98+
} // namespace apache::thrift::concurrency

thrift/lib/cpp/concurrency/ThreadManager.cpp

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,7 @@
5151
FOLLY_GFLAGS_DEFINE_bool(
5252
codel_enabled, false, "Enable codel queue timeout algorithm");
5353

54-
namespace apache {
55-
namespace thrift {
56-
namespace concurrency {
54+
namespace apache::thrift::concurrency {
5755

5856
namespace {
5957
/* Translates from wangle priorities (normal at 0, higher is higher)
@@ -1745,6 +1743,4 @@ PriorityThreadManager::newPriorityThreadManager(size_t normalThreadsCount) {
17451743
return newPriorityThreadManager({{2, 2, 2, normalThreadsCount, 2}});
17461744
}
17471745

1748-
} // namespace concurrency
1749-
} // namespace thrift
1750-
} // namespace apache
1746+
} // namespace apache::thrift::concurrency

thrift/lib/cpp/concurrency/ThreadManager.h

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,7 @@
4141

4242
FOLLY_GFLAGS_DECLARE_bool(codel_enabled);
4343

44-
namespace apache {
45-
namespace thrift {
46-
namespace concurrency {
44+
namespace apache::thrift::concurrency {
4745

4846
class Runnable;
4947
class ThreadFactory;
@@ -553,6 +551,4 @@ inline std::shared_ptr<ThreadFactory> Factory(
553551
return std::make_shared<PosixThreadFactory>(PosixThreadFactory::OTHER, prio);
554552
}
555553

556-
} // namespace concurrency
557-
} // namespace thrift
558-
} // namespace apache
554+
} // namespace apache::thrift::concurrency

thrift/lib/cpp/concurrency/test/ThreadFactoryTests.h

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,7 @@
3333
#include <thrift/lib/cpp/concurrency/Thread.h>
3434
#include <thrift/lib/cpp/thrift_config.h>
3535

36-
namespace apache {
37-
namespace thrift {
38-
namespace concurrency {
39-
namespace test {
36+
namespace apache::thrift::concurrency::test {
4037

4138
using std::shared_ptr;
4239
using namespace apache::thrift::concurrency;
@@ -427,7 +424,4 @@ class ThreadFactoryTests {
427424

428425
const double ThreadFactoryTests::ERROR = .20;
429426

430-
} // namespace test
431-
} // namespace concurrency
432-
} // namespace thrift
433-
} // namespace apache
427+
} // namespace apache::thrift::concurrency::test

0 commit comments

Comments
 (0)