kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

VS: Remove always-true condition on compile options block 

The cmVisualStudio10TargetGenerator::ComputeClOptions method is
only called when the target type compiles, so do not duplicate
that check in the implementation.
This commit is contained in:
Brad King 2014-06-05 11:50:10 -04:00
parent eaa9f2f8ee
commit 8f4bdcc6cb
1 changed files with 32 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Source/cmVisualStudio10TargetGenerator.cxx
View File

@ -1367,9 +1367,6 @@ bool cmVisualStudio10TargetGenerator::ComputeClOptions(
Options& clOptions = *pOptions;
std::string flags;
// collect up flags for
if(this->Target->GetType() < cmTarget::UTILITY)
{
const std::string& linkLanguage =
this->Target->GetLinkerLanguage(configName.c_str());
if(linkLanguage.empty())
@ -1404,7 +1401,6 @@ bool cmVisualStudio10TargetGenerator::ComputeClOptions(
}
this->LocalGenerator->AddCompileOptions(flags, this->Target,
linkLanguage, configName.c_str());
}
// Get preprocessor definitions for this directory.
std::string defineFlags = this->Target->GetMakefile()->GetDefineFlags();