diff options
author | Ben Boeckel <ben.boeckel@kitware.com> | 2022-11-22 18:58:40 (GMT) |
---|---|---|
committer | Ben Boeckel <ben.boeckel@kitware.com> | 2022-11-29 17:39:29 (GMT) |
commit | b72c45e39fa9dd5870046bb5cf25d33049006bd5 (patch) | |
tree | 5bbeafd0dc942663e48b999e6d41d0446e1d2b1d /Source/cmVisualStudioSlnParser.cxx | |
parent | 07172aa31ec468baea0dc3e52ce6f706f55d6f12 (diff) | |
download | CMake-b72c45e39fa9dd5870046bb5cf25d33049006bd5.zip CMake-b72c45e39fa9dd5870046bb5cf25d33049006bd5.tar.gz CMake-b72c45e39fa9dd5870046bb5cf25d33049006bd5.tar.bz2 |
clang-tidy: fix `readability-else-after-return` lints
Diffstat (limited to 'Source/cmVisualStudioSlnParser.cxx')
-rw-r--r-- | Source/cmVisualStudioSlnParser.cxx | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/Source/cmVisualStudioSlnParser.cxx b/Source/cmVisualStudioSlnParser.cxx index 2b5cfd5..0c92a96 100644 --- a/Source/cmVisualStudioSlnParser.cxx +++ b/Source/cmVisualStudioSlnParser.cxx @@ -82,8 +82,7 @@ std::string cmVisualStudioSlnParser::ParsedLine::GetArgVerbatim() const { if (this->Arg.second) return Quote + this->Arg.first + Quote; - else - return this->Arg.first; + return this->Arg.first; } const std::string& cmVisualStudioSlnParser::ParsedLine::GetValue( @@ -91,8 +90,7 @@ const std::string& cmVisualStudioSlnParser::ParsedLine::GetValue( { if (idxValue < this->Values.size()) return this->Values[idxValue].first; - else - return BadString; + return BadString; } std::string cmVisualStudioSlnParser::ParsedLine::GetValueVerbatim( @@ -102,10 +100,9 @@ std::string cmVisualStudioSlnParser::ParsedLine::GetValueVerbatim( const StringData& data = this->Values[idxValue]; if (data.second) return Quote + data.first + Quote; - else - return data.first; - } else - return BadString; + return data.first; + } + return BadString; } class cmVisualStudioSlnParser::State |