mirror of
https://github.com/Kitware/CMake.git
synced 2025-10-15 20:46:37 +08:00
cmCTestMultiProcessHandler: Explain testRun ownership in comments
The ownership semantics of the 'testRun' variable are subtle and may fool static analysers. Add comments explaining them for now. Later some refactoring could be done to clarify the code. Issue: #19610
This commit is contained in:
@@ -189,10 +189,13 @@ bool cmCTestMultiProcessHandler::StartTestProcess(int test)
|
||||
std::strerror(workdir.GetLastResult()));
|
||||
} else {
|
||||
if (testRun->StartTest(this->Completed, this->Total)) {
|
||||
// Ownership of 'testRun' has moved to another structure.
|
||||
// When the test finishes, FinishTestProcess will be called.
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
// Pass ownership of 'testRun'.
|
||||
this->FinishTestProcess(testRun, false);
|
||||
return false;
|
||||
}
|
||||
|
Reference in New Issue
Block a user