LocalGenerator: Add a string overload for AppendFlags

This commit is contained in:
Ben Boeckel 2014-05-01 16:07:26 -04:00
parent 01b79c6385
commit 6fa6bedf78
8 changed files with 38 additions and 20 deletions

View File

@ -399,7 +399,7 @@ cmExtraSublimeTextGenerator::ComputeFlagsForObject(cmSourceFile* source,
lg->GetIncludeDirectories(includes, gtgt, language, config); lg->GetIncludeDirectories(includes, gtgt, language, config);
std::string includeFlags = std::string includeFlags =
lg->GetIncludeFlags(includes, gtgt, language, true); // full include paths lg->GetIncludeFlags(includes, gtgt, language, true); // full include paths
lg->AppendFlags(flags, includeFlags.c_str()); lg->AppendFlags(flags, includeFlags);
} }
// Append old-style preprocessor definition flags. // Append old-style preprocessor definition flags.

View File

@ -2200,7 +2200,7 @@ static void AddVisibilityCompileOption(std::string &flags, cmTarget* target,
return; return;
} }
std::string option = std::string(opt) + prop; std::string option = std::string(opt) + prop;
lg->AppendFlags(flags, option.c_str()); lg->AppendFlags(flags, option);
} }
static void AddInlineVisibilityCompileOption(std::string &flags, static void AddInlineVisibilityCompileOption(std::string &flags,
@ -2384,11 +2384,10 @@ void cmLocalGenerator::AddConfigVariableFlags(std::string& flags,
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void cmLocalGenerator::AppendFlags(std::string& flags, void cmLocalGenerator::AppendFlags(std::string& flags,
const char* newFlags) const std::string& newFlags)
{ {
if(newFlags && *newFlags) if(!newFlags.empty())
{ {
std::string newf = newFlags;
if(flags.size()) if(flags.size())
{ {
flags += " "; flags += " ";
@ -2397,11 +2396,21 @@ void cmLocalGenerator::AppendFlags(std::string& flags,
} }
} }
//----------------------------------------------------------------------------
void cmLocalGenerator::AppendFlags(std::string& flags,
const char* newFlags)
{
if(newFlags && *newFlags)
{
this->AppendFlags(flags, std::string(newFlags));
}
}
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void cmLocalGenerator::AppendFlagEscape(std::string& flags, void cmLocalGenerator::AppendFlagEscape(std::string& flags,
const std::string& rawFlag) const std::string& rawFlag)
{ {
this->AppendFlags(flags, this->EscapeForShell(rawFlag).c_str()); this->AppendFlags(flags, this->EscapeForShell(rawFlag));
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------

View File

@ -152,6 +152,7 @@ public:
void AddCompilerRequirementFlag(std::string &flags, cmTarget* target, void AddCompilerRequirementFlag(std::string &flags, cmTarget* target,
const std::string& lang); const std::string& lang);
///! Append flags to a string. ///! Append flags to a string.
virtual void AppendFlags(std::string& flags, const std::string& newFlags);
virtual void AppendFlags(std::string& flags, const char* newFlags); virtual void AppendFlags(std::string& flags, const char* newFlags);
virtual void AppendFlagEscape(std::string& flags, virtual void AppendFlagEscape(std::string& flags,
const std::string& rawFlag); const std::string& rawFlag);

View File

@ -956,21 +956,28 @@ cmLocalUnixMakefileGenerator3
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void cmLocalUnixMakefileGenerator3::AppendFlags(std::string& flags, void cmLocalUnixMakefileGenerator3::AppendFlags(std::string& flags,
const char* newFlags) const std::string& newFlags)
{ {
if(this->WatcomWMake && newFlags && *newFlags) if(this->WatcomWMake && !newFlags.empty())
{ {
std::string newf = newFlags; std::string newf = newFlags;
if(newf.find("\\\"") != newf.npos) if(newf.find("\\\"") != newf.npos)
{ {
cmSystemTools::ReplaceString(newf, "\\\"", "\""); cmSystemTools::ReplaceString(newf, "\\\"", "\"");
this->cmLocalGenerator::AppendFlags(flags, newf.c_str()); this->cmLocalGenerator::AppendFlags(flags, newf);
return; return;
} }
} }
this->cmLocalGenerator::AppendFlags(flags, newFlags); this->cmLocalGenerator::AppendFlags(flags, newFlags);
} }
//----------------------------------------------------------------------------
void cmLocalUnixMakefileGenerator3::AppendFlags(std::string& flags,
const char* newFlags)
{
this->cmLocalGenerator::AppendFlags(flags, newFlags);
}
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void void
cmLocalUnixMakefileGenerator3 cmLocalUnixMakefileGenerator3

View File

@ -168,6 +168,7 @@ public:
const std::string& tgt); const std::string& tgt);
// append flags to a string // append flags to a string
virtual void AppendFlags(std::string& flags, const std::string& newFlags);
virtual void AppendFlags(std::string& flags, const char* newFlags); virtual void AppendFlags(std::string& flags, const char* newFlags);
// append an echo command // append an echo command

View File

@ -249,7 +249,7 @@ void cmMakefileLibraryTargetGenerator::WriteLibraryRules
// Create set of linking flags. // Create set of linking flags.
std::string linkFlags; std::string linkFlags;
this->LocalGenerator->AppendFlags(linkFlags, extraFlags.c_str()); this->LocalGenerator->AppendFlags(linkFlags, extraFlags);
// Add OSX version flags, if any. // Add OSX version flags, if any.
if(this->Target->GetType() == cmTarget::SHARED_LIBRARY || if(this->Target->GetType() == cmTarget::SHARED_LIBRARY ||
@ -810,6 +810,6 @@ cmMakefileLibraryTargetGenerator
// Append the flag since a non-zero version is specified. // Append the flag since a non-zero version is specified.
cmOStringStream vflag; cmOStringStream vflag;
vflag << flag << major << "." << minor << "." << patch; vflag << flag << major << "." << minor << "." << patch;
this->LocalGenerator->AppendFlags(flags, vflag.str().c_str()); this->LocalGenerator->AppendFlags(flags, vflag.str());
} }
} }

View File

@ -303,7 +303,7 @@ std::string cmMakefileTargetGenerator::GetFlags(const std::string &l)
// Add include directory flags. // Add include directory flags.
this->LocalGenerator-> this->LocalGenerator->
AppendFlags(flags,this->GetFrameworkFlags(l).c_str()); AppendFlags(flags,this->GetFrameworkFlags(l));
// Add target-specific flags. // Add target-specific flags.
this->LocalGenerator->AddCompileOptions(flags, this->Target, this->LocalGenerator->AddCompileOptions(flags, this->Target,
@ -551,7 +551,7 @@ cmMakefileTargetGenerator
std::string langFlags = "$("; std::string langFlags = "$(";
langFlags += lang; langFlags += lang;
langFlags += "_FLAGS)"; langFlags += "_FLAGS)";
this->LocalGenerator->AppendFlags(flags, langFlags.c_str()); this->LocalGenerator->AppendFlags(flags, langFlags);
std::string configUpper = std::string configUpper =
cmSystemTools::UpperCase(this->LocalGenerator->ConfigurationName); cmSystemTools::UpperCase(this->LocalGenerator->ConfigurationName);
@ -1968,11 +1968,11 @@ void cmMakefileTargetGenerator::AddIncludeFlags(std::string& flags,
std::string arg = "@" + std::string arg = "@" +
this->CreateResponseFile(name.c_str(), includeFlags, this->CreateResponseFile(name.c_str(), includeFlags,
this->FlagFileDepends[lang]); this->FlagFileDepends[lang]);
this->LocalGenerator->AppendFlags(flags, arg.c_str()); this->LocalGenerator->AppendFlags(flags, arg);
} }
else else
{ {
this->LocalGenerator->AppendFlags(flags, includeFlags.c_str()); this->LocalGenerator->AppendFlags(flags, includeFlags);
} }
} }
@ -2044,7 +2044,7 @@ void cmMakefileTargetGenerator::AddFortranFlags(std::string& flags)
modflag += this->Convert(mod_dir, modflag += this->Convert(mod_dir,
cmLocalGenerator::START_OUTPUT, cmLocalGenerator::START_OUTPUT,
cmLocalGenerator::SHELL); cmLocalGenerator::SHELL);
this->LocalGenerator->AppendFlags(flags, modflag.c_str()); this->LocalGenerator->AppendFlags(flags, modflag);
} }
// If there is a separate module path flag then duplicate the // If there is a separate module path flag then duplicate the
@ -2066,7 +2066,7 @@ void cmMakefileTargetGenerator::AddFortranFlags(std::string& flags)
flg += this->Convert(*idi, flg += this->Convert(*idi,
cmLocalGenerator::NONE, cmLocalGenerator::NONE,
cmLocalGenerator::SHELL); cmLocalGenerator::SHELL);
this->LocalGenerator->AppendFlags(flags, flg.c_str()); this->LocalGenerator->AppendFlags(flags, flg);
} }
} }
} }
@ -2093,7 +2093,7 @@ void cmMakefileTargetGenerator::AddModuleDefinitionFlag(std::string& flags)
// vs6's "cl -link" pass it to the linker. // vs6's "cl -link" pass it to the linker.
std::string flag = defFileFlag; std::string flag = defFileFlag;
flag += (this->LocalGenerator->ConvertToLinkReference(def)); flag += (this->LocalGenerator->ConvertToLinkReference(def));
this->LocalGenerator->AppendFlags(flags, flag.c_str()); this->LocalGenerator->AppendFlags(flags, flag);
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------

View File

@ -169,7 +169,7 @@ cmNinjaTargetGenerator::ComputeFlagsForObject(cmSourceFile const* source,
if(cmGlobalNinjaGenerator::IsMinGW()) if(cmGlobalNinjaGenerator::IsMinGW())
cmSystemTools::ReplaceString(includeFlags, "\\", "/"); cmSystemTools::ReplaceString(includeFlags, "\\", "/");
this->LocalGenerator->AppendFlags(languageFlags, includeFlags.c_str()); this->LocalGenerator->AppendFlags(languageFlags, includeFlags);
// Append old-style preprocessor definition flags. // Append old-style preprocessor definition flags.
this->LocalGenerator->AppendFlags(languageFlags, this->LocalGenerator->AppendFlags(languageFlags,
@ -698,7 +698,7 @@ cmNinjaTargetGenerator
std::string flag = defFileFlag; std::string flag = defFileFlag;
flag += (this->LocalGenerator->ConvertToLinkReference( flag += (this->LocalGenerator->ConvertToLinkReference(
this->ModuleDefinitionFile)); this->ModuleDefinitionFile));
this->LocalGenerator->AppendFlags(flags, flag.c_str()); this->LocalGenerator->AppendFlags(flags, flag);
} }
void void