diff options
| author | Lioncash <mathew1800@gmail.com> | 2018-09-19 19:02:04 -0400 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2018-09-19 19:02:06 -0400 |
| commit | 3146502a12125a2511cfda1cd08924d71d378e10 (patch) | |
| tree | c3208927a64a1cb553c107a5675017a8adfdd7c2 /src | |
| parent | 05d49962b69f6597116246776828c2bbc8a98582 (diff) | |
control_metadata: Remove unnecessary else within GetLanguageEntry()
There's no need to indent the code here, given the if case contains a
return statement at the end of it.
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/file_sys/control_metadata.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/core/file_sys/control_metadata.cpp b/src/core/file_sys/control_metadata.cpp index b47c95d11f..f11b91399c 100644 --- a/src/core/file_sys/control_metadata.cpp +++ b/src/core/file_sys/control_metadata.cpp @@ -31,15 +31,15 @@ NACP::NACP(VirtualFile file) : raw(std::make_unique<RawNACP>()) { const LanguageEntry& NACP::GetLanguageEntry(Language language) const { if (language != Language::Default) { return raw->language_entries.at(static_cast<u8>(language)); - } else { - for (const auto& language_entry : raw->language_entries) { - if (!language_entry.GetApplicationName().empty()) - return language_entry; - } - - // Fallback to English - return GetLanguageEntry(Language::AmericanEnglish); } + + for (const auto& language_entry : raw->language_entries) { + if (!language_entry.GetApplicationName().empty()) + return language_entry; + } + + // Fallback to English + return GetLanguageEntry(Language::AmericanEnglish); } std::string NACP::GetApplicationName(Language language) const { |