GitLab

Once !3574 (merged) is merged, we can remove some boilerplate from all commands. Do the following for each class that derives from cmCommand:

1. To access the cmMakefile, call status.GetMakefile(). Port away from this->Makefile and this->GetMakefile(). Make sure that the cmExecutionStatus& argument is status rather than /*unused*/.

2. To set an error message, call status.SetError(msg). Port away from this->SetError(msg). Make sure that the cmExecutionStatus& argument is status rather than /*unused*/.

3. Turn member functions into free functions in an unnamed namespace in the source file. Alternatively, extract a helper class that does not derive from cmCommand. It might be necessary to pass cmExecutionStatus& status or other parameters down to those commands.

4. Turn member variables into local variables. Rationale: When the command is executed, the Clone() member function is called and then the InitialPass() member function is called on the clone. Subsequently, the clone is discarded. This means that the lifetime of all the member variables that are not copied in the Clone() member function is limited to the InitialPass() call anyway.

5. Turn the class into a function: Delete the Clone() member function for good, turn the InitialPass() member function into a free function with the original name of the class. In cmCommands.cxx, pass that name to state->AddBuiltinCommand() without any decoration.