Merge topic 'noduplicates'

78357e98a4 Avoid duplicate calls to GetDefinition

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !5073
This commit is contained in:
Brad King
2020-07-29 12:06:58 +00:00
committed by Kitware Robot
2 changed files with 3 additions and 5 deletions

View File

@@ -401,15 +401,13 @@ bool cmGlobalGenerator::FindMakeProgram(cmMakefile* mf)
"all generators must specify this->FindMakeProgramFile");
return false;
}
if (!mf->GetDefinition("CMAKE_MAKE_PROGRAM") ||
cmIsOff(mf->GetDefinition("CMAKE_MAKE_PROGRAM"))) {
if (cmIsOff(mf->GetDefinition("CMAKE_MAKE_PROGRAM"))) {
std::string setMakeProgram = mf->GetModulesFile(this->FindMakeProgramFile);
if (!setMakeProgram.empty()) {
mf->ReadListFile(setMakeProgram);
}
}
if (!mf->GetDefinition("CMAKE_MAKE_PROGRAM") ||
cmIsOff(mf->GetDefinition("CMAKE_MAKE_PROGRAM"))) {
if (cmIsOff(mf->GetDefinition("CMAKE_MAKE_PROGRAM"))) {
std::ostringstream err;
err << "CMake was unable to find a build program corresponding to \""
<< this->GetName() << "\". CMAKE_MAKE_PROGRAM is not set. You "

View File

@@ -260,7 +260,7 @@ void cmGraphVizWriter::ReadSettings(
do { \
const char* value = mf.GetDefinition(cmakeDefinition); \
if (value) { \
(var) = mf.IsOn(cmakeDefinition); \
(var) = cmIsOn(value); \
} \
} while (false)