MakeStep: Rename m_makeArguments -> m_userArguments

These are the arguments provided by the user. Full make argument may
contain also some automatic ones, such as job count and Makefile.

Change-Id: Ief03623daa9c9bbc54ca6ad2dcf1492510fe531b
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
Orgad Shaneh
2019-09-21 23:45:17 +03:00
committed by Orgad Shaneh
parent 471e78a99c
commit 57404c9551
2 changed files with 7 additions and 7 deletions

View File

@@ -246,7 +246,7 @@ bool MakeStep::makeflagsContainsJobCount() const
bool MakeStep::userArgsContainsJobCount() const bool MakeStep::userArgsContainsJobCount() const
{ {
return argsJobCount(m_makeArguments).has_value(); return argsJobCount(m_userArguments).has_value();
} }
Utils::Environment MakeStep::environment(BuildConfiguration *bc) const Utils::Environment MakeStep::environment(BuildConfiguration *bc) const
@@ -275,7 +275,7 @@ QVariantMap MakeStep::toMap() const
QVariantMap map(AbstractProcessStep::toMap()); QVariantMap map(AbstractProcessStep::toMap());
map.insert(id().withSuffix(BUILD_TARGETS_SUFFIX).toString(), m_buildTargets); map.insert(id().withSuffix(BUILD_TARGETS_SUFFIX).toString(), m_buildTargets);
map.insert(id().withSuffix(MAKE_ARGUMENTS_SUFFIX).toString(), m_makeArguments); map.insert(id().withSuffix(MAKE_ARGUMENTS_SUFFIX).toString(), m_userArguments);
map.insert(id().withSuffix(MAKE_COMMAND_SUFFIX).toString(), m_makeCommand.toString()); map.insert(id().withSuffix(MAKE_COMMAND_SUFFIX).toString(), m_makeCommand.toString());
map.insert(id().withSuffix(CLEAN_SUFFIX).toString(), m_clean); map.insert(id().withSuffix(CLEAN_SUFFIX).toString(), m_clean);
const QString jobCountKey = id().withSuffix(JOBCOUNT_SUFFIX).toString(); const QString jobCountKey = id().withSuffix(JOBCOUNT_SUFFIX).toString();
@@ -290,7 +290,7 @@ QVariantMap MakeStep::toMap() const
bool MakeStep::fromMap(const QVariantMap &map) bool MakeStep::fromMap(const QVariantMap &map)
{ {
m_buildTargets = map.value(id().withSuffix(BUILD_TARGETS_SUFFIX).toString()).toStringList(); m_buildTargets = map.value(id().withSuffix(BUILD_TARGETS_SUFFIX).toString()).toStringList();
m_makeArguments = map.value(id().withSuffix(MAKE_ARGUMENTS_SUFFIX).toString()).toString(); m_userArguments = map.value(id().withSuffix(MAKE_ARGUMENTS_SUFFIX).toString()).toString();
m_makeCommand = FilePath::fromString( m_makeCommand = FilePath::fromString(
map.value(id().withSuffix(MAKE_COMMAND_SUFFIX).toString()).toString()); map.value(id().withSuffix(MAKE_COMMAND_SUFFIX).toString()).toString());
m_clean = map.value(id().withSuffix(CLEAN_SUFFIX).toString()).toBool(); m_clean = map.value(id().withSuffix(CLEAN_SUFFIX).toString()).toBool();
@@ -315,12 +315,12 @@ QStringList MakeStep::jobArguments() const
QString MakeStep::userArguments() const QString MakeStep::userArguments() const
{ {
return m_makeArguments; return m_userArguments;
} }
void MakeStep::setUserArguments(const QString &args) void MakeStep::setUserArguments(const QString &args)
{ {
m_makeArguments = args; m_userArguments = args;
} }
FilePath MakeStep::makeCommand() const FilePath MakeStep::makeCommand() const
@@ -332,7 +332,7 @@ CommandLine MakeStep::effectiveMakeCommand() const
{ {
CommandLine cmd(m_makeCommand.isEmpty() ? defaultMakeCommand() : m_makeCommand); CommandLine cmd(m_makeCommand.isEmpty() ? defaultMakeCommand() : m_makeCommand);
cmd.addArgs(m_makeArguments, CommandLine::Raw); cmd.addArgs(m_userArguments, CommandLine::Raw);
cmd.addArgs(jobArguments()); cmd.addArgs(jobArguments());
cmd.addArgs(m_buildTargets); cmd.addArgs(m_buildTargets);

View File

@@ -91,7 +91,7 @@ private:
QStringList m_buildTargets; QStringList m_buildTargets;
QStringList m_availableTargets; QStringList m_availableTargets;
QString m_makeArguments; QString m_userArguments;
Utils::FilePath m_makeCommand; Utils::FilePath m_makeCommand;
int m_userJobCount = 4; int m_userJobCount = 4;
bool m_overrideMakeflags = false; bool m_overrideMakeflags = false;