Skip to content

Commit b101600

Browse files
yfeldblummeta-codesync[bot]
authored andcommitted
no std::make_pair in ThreadPoolExecutor test
Summary: Just use `std::pair` instead of `std::make_pair`. Reviewed By: ilvokhin, dmm-fb Differential Revision: D93278876 fbshipit-source-id: 0564aaf51f7b37a1d73691ae770fd1593cbd2e59
1 parent 8f7eb9e commit b101600

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

third-party/folly/src/folly/executors/test/ThreadPoolExecutorTest.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -635,7 +635,7 @@ TEST(ThreadPoolExecutorTest, NoThreadPriorityInheritance) {
635635
// If minThreads == maxThreads, no threads should be created on add(), and
636636
// thus they should not inherit the parent thread's priority. Instead, all
637637
// threads should inherit the priority at the time of construction.
638-
CPUThreadPoolExecutor exe{std::make_pair(kNumThreads, kNumThreads)};
638+
CPUThreadPoolExecutor exe{std::pair(kNumThreads, kNumThreads)};
639639

640640
// Nice the current thread.
641641
setpriority(PRIO_PROCESS, 0, 19);
@@ -669,7 +669,7 @@ TEST(ThreadPoolExecutorTest, NoTimeoutInNonDynamicPool) {
669669
// If minThreads == maxThreads, only non-timeout take() should be used on the
670670
// queue.
671671
CPUThreadPoolExecutor exe{
672-
std::make_pair(kNumThreads, kNumThreads),
672+
std::pair(kNumThreads, kNumThreads),
673673
std::make_unique<
674674
NoTimeoutBlockingQueue<CPUThreadPoolExecutor::CPUTask>>()};
675675
exe.add([] {});

0 commit comments

Comments
 (0)