The-Homebrew-Cloud/dolphin
2
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-01-10 08:09:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #12112 from JosJuice/hires-texture-encoding

Browse Source 
VideoCommon: Fix std::filesystem::path encoding conversion
This commit is contained in:
Admiral H. Curtiss 2023-08-16 18:39:18 +02:00 committed by GitHub
commit 23ae8c439c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
View File

@ -103,18 +103,18 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadPixelShader(const
} }
const auto approx_mem_size = metadata_size + shader_size; const auto approx_mem_size = metadata_size + shader_size;
if (!File::ReadFileToString(shader->second.string(), data->m_shader_source)) if (!File::ReadFileToString(PathToString(shader->second), data->m_shader_source))
{ {
ERROR_LOG_FMT(VIDEO, "Asset '{}' error - failed to load the shader file '{}',", asset_id, ERROR_LOG_FMT(VIDEO, "Asset '{}' error - failed to load the shader file '{}',", asset_id,
shader->second.string()); PathToString(shader->second));
return {}; return {};
} }
std::string json_data; std::string json_data;
if (!File::ReadFileToString(metadata->second.string(), json_data)) if (!File::ReadFileToString(PathToString(metadata->second), json_data))
{ {
ERROR_LOG_FMT(VIDEO, "Asset '{}' error - failed to load the json file '{}',", asset_id, ERROR_LOG_FMT(VIDEO, "Asset '{}' error - failed to load the json file '{}',", asset_id,
metadata->second.string()); PathToString(metadata->second));
return {}; return {};
} }
@ -125,7 +125,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadPixelShader(const
{ {
ERROR_LOG_FMT(VIDEO, ERROR_LOG_FMT(VIDEO,
"Asset '{}' error - failed to load the json file '{}', due to parse error: {}", "Asset '{}' error - failed to load the json file '{}', due to parse error: {}",
asset_id, metadata->second.string(), error); asset_id, PathToString(metadata->second), error);
return {}; return {};
} }
if (!root.is<picojson::object>()) if (!root.is<picojson::object>())
@ -133,7 +133,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadPixelShader(const
ERROR_LOG_FMT( ERROR_LOG_FMT(
VIDEO, VIDEO,
"Asset '{}' error - failed to load the json file '{}', due to root not being an object!", "Asset '{}' error - failed to load the json file '{}', due to root not being an object!",
asset_id, metadata->second.string()); asset_id, PathToString(metadata->second));
return {}; return {};
} }
@ -159,10 +159,10 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadMaterial(const As
const auto& asset_path = asset_map.begin()->second; const auto& asset_path = asset_map.begin()->second;
std::string json_data; std::string json_data;
if (!File::ReadFileToString(asset_path.string(), json_data)) if (!File::ReadFileToString(PathToString(asset_path), json_data))
{ {
ERROR_LOG_FMT(VIDEO, "Asset '{}' error - material failed to load the json file '{}',", ERROR_LOG_FMT(VIDEO, "Asset '{}' error - material failed to load the json file '{}',",
asset_id, asset_path.string()); asset_id, PathToString(asset_path));
return {}; return {};
} }
@ -174,7 +174,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadMaterial(const As
ERROR_LOG_FMT( ERROR_LOG_FMT(
VIDEO, VIDEO,
"Asset '{}' error - material failed to load the json file '{}', due to parse error: {}", "Asset '{}' error - material failed to load the json file '{}', due to parse error: {}",
asset_id, asset_path.string(), error); asset_id, PathToString(asset_path), error);
return {}; return {};
} }
if (!root.is<picojson::object>()) if (!root.is<picojson::object>())
@ -182,7 +182,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadMaterial(const As
ERROR_LOG_FMT(VIDEO, ERROR_LOG_FMT(VIDEO,
"Asset '{}' error - material failed to load the json file '{}', due to root not " "Asset '{}' error - material failed to load the json file '{}', due to root not "
"being an object!", "being an object!",
asset_id, asset_path.string()); asset_id, PathToString(asset_path));
return {}; return {};
} }
@ -193,7 +193,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadMaterial(const As
ERROR_LOG_FMT(VIDEO, ERROR_LOG_FMT(VIDEO,
"Asset '{}' error - material failed to load the json file '{}', as material " "Asset '{}' error - material failed to load the json file '{}', as material "
"json could not be parsed!", "json could not be parsed!",
asset_id, asset_path.string()); asset_id, PathToString(asset_path));
return {}; return {};
} }
@ -222,11 +222,11 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
asset_id, ec); asset_id, ec);
return {}; return {};
} }
auto ext = asset_path.extension().string(); auto ext = PathToString(asset_path.extension());
Common::ToLower(&ext); Common::ToLower(&ext);
if (ext == ".dds") if (ext == ".dds")
{ {
if (!LoadDDSTexture(data, asset_path.string())) if (!LoadDDSTexture(data, PathToString(asset_path)))
{ {
ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load dds texture!", asset_id); ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load dds texture!", asset_id);
return {}; return {};
@ -243,7 +243,7 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
if (data->m_levels.empty()) if (data->m_levels.empty())
data->m_levels.push_back({}); data->m_levels.push_back({});
if (!LoadPNGTexture(&data->m_levels[0], asset_path.string())) if (!LoadPNGTexture(&data->m_levels[0], PathToString(asset_path)))
{ {
ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load png texture!", asset_id); ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load png texture!", asset_id);
return {}; return {};
@ -275,7 +275,7 @@ bool DirectFilesystemAssetLibrary::LoadMips(const std::filesystem::path& asset_p
std::string path; std::string path;
std::string filename; std::string filename;
std::string extension; std::string extension;
SplitPath(asset_path.string(), &path, &filename, &extension); SplitPath(PathToString(asset_path), &path, &filename, &extension);
std::string extension_lower = extension; std::string extension_lower = extension;
Common::ToLower(&extension_lower); Common::ToLower(&extension_lower);

2
Source/Core/VideoCommon/HiresTextures.cpp
View File

@ -131,7 +131,7 @@ void HiresTexture::Update()
// Since this is just a texture (single file) the mapper doesn't really matter // Since this is just a texture (single file) the mapper doesn't really matter
// just provide a string // just provide a string
s_file_library->SetAssetIDMapData( s_file_library->SetAssetIDMapData(
filename, std::map<std::string, std::filesystem::path>{{"", path}}); filename, std::map<std::string, std::filesystem::path>{{"", StringToPath(path)}});
if (g_ActiveConfig.bCacheHiresTextures) if (g_ActiveConfig.bCacheHiresTextures)
{ {