clang-tidy: fix readability-else-after-return lints

This commit is contained in:
Ben Boeckel 2023-07-27 18:22:05 -04:00
parent 4489e9a85c
commit ffa49c23aa
2 changed files with 10 additions and 9 deletions

View File

@ -997,12 +997,12 @@ bool cmCPackWIXGenerator::RequireOption(std::string const& name,
value = *tmp;
return true;
} else {
cmCPackLogger(cmCPackLog::LOG_ERROR,
"Required variable " << name << " not set" << std::endl);
return false;
}
cmCPackLogger(cmCPackLog::LOG_ERROR,
"Required variable " << name << " not set" << std::endl);
return false;
}
std::string cmCPackWIXGenerator::GetArchitecture() const
@ -1012,9 +1012,8 @@ std::string cmCPackWIXGenerator::GetArchitecture() const
if (void_p_size == "8"_s) {
return "x64";
} else {
return "x86";
}
return "x86";
}
std::string cmCPackWIXGenerator::GenerateGUID()
@ -1109,7 +1108,8 @@ std::string cmCPackWIXGenerator::CreateNewIdForPath(std::string const& path)
<< path << '\'' << std::endl);
return std::string();
} else if (ambiguityCount > 1) {
}
if (ambiguityCount > 1) {
result << '_' << ambiguityCount;
}

View File

@ -158,7 +158,8 @@ void cmWIXRichTextFormatWriter::EmitUnicodeCodepoint(int c)
// Do not emit byte order mark (BOM)
if (c == 0xFEFF) {
return;
} else if (c <= 0xFFFF) {
}
if (c <= 0xFFFF) {
EmitUnicodeSurrogate(c);
} else {
c -= 0x10000;