artemist/yuzu
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

file_sys/registered_cache: Eliminate variable shadowing

Browse source 
Also inverts if statements where applicable to allow unindenting code a
little bit.
This commit is contained in:
Lioncash 2018-12-01 23:50:10 -05:00
parent e88cdcc912
commit efbcff0af0
1 changed files with 33 additions and 34 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
src/core/file_sys/registered_cache.cpp
View file

@ -107,18 +107,20 @@ static ContentRecordType GetCRTypeFromNCAType(NCAContentType type) {
VirtualFile RegisteredCache::OpenFileOrDirectoryConcat(const VirtualDir& dir, VirtualFile RegisteredCache::OpenFileOrDirectoryConcat(const VirtualDir& dir,
std::string_view path) const { std::string_view path) const {
const auto file = dir->GetFileRelative(path); const auto file = dir->GetFileRelative(path);
if (file != nullptr) if (file != nullptr) {
return file; return file;
}
const auto nca_dir = dir->GetDirectoryRelative(path); const auto nca_dir = dir->GetDirectoryRelative(path);
if (nca_dir != nullptr) { if (nca_dir == nullptr) {
const auto nca_dir = dir->GetDirectoryRelative(path); return nullptr;
VirtualFile file = nullptr; }
const auto files = nca_dir->GetFiles(); const auto files = nca_dir->GetFiles();
if (files.size() == 1 && files[0]->GetName() == "00") { if (files.size() == 1 && files[0]->GetName() == "00") {
file = files[0]; return files[0];
} else { }
std::vector<VirtualFile> concat; std::vector<VirtualFile> concat;
// Since the files are a two-digit hex number, max is FF. // Since the files are a two-digit hex number, max is FF.
for (std::size_t i = 0; i < 0x100; ++i) { for (std::size_t i = 0; i < 0x100; ++i) {
@ -127,24 +129,21 @@ VirtualFile RegisteredCache::OpenFileOrDirectoryConcat(const VirtualDir& dir,
concat.push_back(std::move(next)); concat.push_back(std::move(next));
} else { } else {
next = nca_dir->GetFile(fmt::format("{:02x}", i)); next = nca_dir->GetFile(fmt::format("{:02x}", i));
if (next != nullptr) if (next != nullptr) {
concat.push_back(std::move(next)); concat.push_back(std::move(next));
else } else {
break; break;
} }
} }
if (concat.empty())
return nullptr;
file = ConcatenatedVfsFile::MakeConcatenatedFile(concat, concat.front()->GetName());
} }
return file; if (concat.empty()) {
}
return nullptr; return nullptr;
} }
return ConcatenatedVfsFile::MakeConcatenatedFile(concat, concat.front()->GetName());
}
VirtualFile RegisteredCache::GetFileAtID(NcaID id) const { VirtualFile RegisteredCache::GetFileAtID(NcaID id) const {
VirtualFile file; VirtualFile file;
// Try all four modes of file storage: // Try all four modes of file storage: