mirror/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2026-01-09 15:20:56 -06:00
Code Issues Packages Projects Releases Wiki Activity

cmMakefile: Simplify ExpandArguments signature

Browse Source 
The only call sites that pass the explicit file name argument are in
function blocker `ArgumentsMatch` methods for `function` and `macro`.
We already ensure that they are balanced within a file scope, and the
RAII helpers `BuildsystemFileScope` and `ListFileScope` ensure that the
backtrace and execution list file stacks unwind to the matching level.
Therefore we can assume that the file name where we are checking for
matching arguments matches starting file name where those arguments
first appeared, and do not need to pass it explicitly.
This commit is contained in:
Brad King
2020-09-25 10:12:20 -04:00
parent e456dae669
commit 727ed0c403
4 changed files with 12 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
Source/cmFunctionCommand.cxx
View File

@@ -147,8 +147,7 @@ bool cmFunctionFunctionBlocker::ArgumentsMatch(cmListFileFunction const& lff,
cmMakefile& mf) const
{
std::vector<std::string> expandedArguments;
mf.ExpandArguments(lff.Arguments, expandedArguments,
this->GetStartingContext().FilePath.c_str());
mf.ExpandArguments(lff.Arguments, expandedArguments);
return expandedArguments.empty() ||
expandedArguments.front() == this->Args.front();
}

3
Source/cmMacroCommand.cxx
View File

@@ -157,8 +157,7 @@ bool cmMacroFunctionBlocker::ArgumentsMatch(cmListFileFunction const& lff,
cmMakefile& mf) const
{
std::vector<std::string> expandedArguments;
mf.ExpandArguments(lff.Arguments, expandedArguments,
this->GetStartingContext().FilePath.c_str());
mf.ExpandArguments(lff.Arguments, expandedArguments);
return expandedArguments.empty() || expandedArguments[0] == this->Args[0];
}

23
Source/cmMakefile.cxx
View File

@@ -3335,13 +3335,9 @@ std::string const& cmMakefile::GetExecutionFilePath() const
}
bool cmMakefile::ExpandArguments(std::vector<cmListFileArgument> const& inArgs,
std::vector<std::string>& outArgs,
const char* filename) const
std::vector<std::string>& outArgs) const
{
if (!filename) {
auto const& efp = this->GetExecutionFilePath();
filename = efp.c_str();
}
std::string const& filename = this->GetExecutionFilePath();
std::string value;
outArgs.reserve(inArgs.size());
for (cmListFileArgument const& i : inArgs) {
@@ -3352,8 +3348,8 @@ bool cmMakefile::ExpandArguments(std::vector<cmListFileArgument> const& inArgs,
}
// Expand the variables in the argument.
value = i.Value;
this->ExpandVariablesInString(value, false, false, false, filename, i.Line,
false, false);
this->ExpandVariablesInString(value, false, false, false, filename.c_str(),
i.Line, false, false);
// If the argument is quoted, it should be one argument.
// Otherwise, it may be a list of arguments.
@@ -3368,12 +3364,9 @@ bool cmMakefile::ExpandArguments(std::vector<cmListFileArgument> const& inArgs,
bool cmMakefile::ExpandArguments(
std::vector<cmListFileArgument> const& inArgs,
std::vector<cmExpandedCommandArgument>& outArgs, const char* filename) const
std::vector<cmExpandedCommandArgument>& outArgs) const
{
if (!filename) {
auto const& efp = this->GetExecutionFilePath();
filename = efp.c_str();
}
std::string const& filename = this->GetExecutionFilePath();
std::string value;
outArgs.reserve(inArgs.size());
for (cmListFileArgument const& i : inArgs) {
@@ -3384,8 +3377,8 @@ bool cmMakefile::ExpandArguments(
}
// Expand the variables in the argument.
value = i.Value;
this->ExpandVariablesInString(value, false, false, false, filename, i.Line,
false, false);
this->ExpandVariablesInString(value, false, false, false, filename.c_str(),
i.Line, false, false);
// If the argument is quoted, it should be one argument.
// Otherwise, it may be a list of arguments.

7
Source/cmMakefile.h
View File

@@ -732,12 +732,9 @@ public:
* variable replacement and list expansion.
*/
bool ExpandArguments(std::vector<cmListFileArgument> const& inArgs,
std::vector<std::string>& outArgs,
const char* filename = nullptr) const;
std::vector<std::string>& outArgs) const;
bool ExpandArguments(std::vector<cmListFileArgument> const& inArgs,
std::vector<cmExpandedCommandArgument>& outArgs,
const char* filename = nullptr) const;
std::vector<cmExpandedCommandArgument>& outArgs) const;
/**
* Get the instance