diff options
| author | Lioncash <mathew1800@gmail.com> | 2018-12-01 22:48:57 -0500 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2018-12-01 22:49:00 -0500 |
| commit | 8cd6b92d3196d11f70c6a3e38356fda9564ea8cb (patch) | |
| tree | 4b9519f21fb4ff2fcb36b3cc52cab61ef589384b /src/yuzu/game_list_worker.cpp | |
| parent | cd9570ce0d29d212475ad8e1dd4d91e884040e2f (diff) | |
yuzu/game_list_worker: Tidy up string handling in FillControlMap()
We don't need to call out to our own file handling functions when we're
going to construct a QFileInfo instance right after it. We also don't
need to convert to a std::string again just to compare the file
extension.
Diffstat (limited to 'src/yuzu/game_list_worker.cpp')
| -rw-r--r-- | src/yuzu/game_list_worker.cpp | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/src/yuzu/game_list_worker.cpp b/src/yuzu/game_list_worker.cpp index 362902e460..3d5f2c0f4b 100644 --- a/src/yuzu/game_list_worker.cpp +++ b/src/yuzu/game_list_worker.cpp @@ -155,14 +155,15 @@ void GameListWorker::AddInstalledTitlesToGameList() { void GameListWorker::FillControlMap(const std::string& dir_path) { const auto nca_control_callback = [this](u64* num_entries_out, const std::string& directory, const std::string& virtual_name) -> bool { - std::string physical_name = directory + DIR_SEP + virtual_name; + const std::string physical_name = directory + DIR_SEP + virtual_name; - if (stop_processing) - return false; // Breaks the callback loop. + if (stop_processing) { + // Breaks the callback loop + return false; + } - bool is_dir = FileUtil::IsDirectory(physical_name); - QFileInfo file_info(physical_name.c_str()); - if (!is_dir && file_info.suffix().toStdString() == "nca") { + const QFileInfo file_info(QString::fromStdString(physical_name)); + if (!file_info.isDir() && file_info.suffix() == QStringLiteral("nca")) { auto nca = std::make_unique<FileSys::NCA>(vfs->OpenFile(physical_name, FileSys::Mode::Read)); if (nca->GetType() == FileSys::NCAContentType::Control) { |
