From b17c732e88e5681fd1ec90d587d796c44329028b Mon Sep 17 00:00:00 2001 From: Brad King Date: Thu, 2 Nov 2023 15:04:49 -0400 Subject: cmCTestMultiProcessHandler: Clarify role of StartTestProcess Focus its role on actually running the test process. Move administrative tasks to the call site. --- Source/CTest/cmCTestMultiProcessHandler.cxx | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Source/CTest/cmCTestMultiProcessHandler.cxx b/Source/CTest/cmCTestMultiProcessHandler.cxx index 3f1e019..d910367 100644 --- a/Source/CTest/cmCTestMultiProcessHandler.cxx +++ b/Source/CTest/cmCTestMultiProcessHandler.cxx @@ -164,12 +164,8 @@ void cmCTestMultiProcessHandler::RunTests() void cmCTestMultiProcessHandler::StartTestProcess(int test) { - this->LockResources(test); - cmCTestOptionalLog(this->CTest, HANDLER_VERBOSE_OUTPUT, "test " << test << "\n", this->Quiet); - // now remove the test itself - this->ErasePendingTest(test); auto testRun = cm::make_unique(*this, test); @@ -591,8 +587,12 @@ void cmCTestMultiProcessHandler::StartNextTests() continue; } + // Lock resources needed by this test. + this->LockResources(test); + // The test is ready to run. numToStart -= processors; + this->ErasePendingTest(test); this->StartTest(test); } -- cgit v0.12