From 429b2eca8a90f895516203d35b5f9b69f1217151 Mon Sep 17 00:00:00 2001
From: iwubcode <iwubcode@users.noreply.github.com>
Date: Thu, 8 Jun 2023 19:48:45 -0500
Subject: [PATCH 1/2] VideoCommon: add logging for loading texture assets

---
 .../Assets/DirectFilesystemAssetLibrary.cpp      | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp b/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
index 68d63a2466..bbb3a644f5 100644
--- a/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
+++ b/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
@@ -52,7 +52,11 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
 
   // Raw texture is expected to have one asset mapped
   if (asset_map.empty() || asset_map.size() > 1)
+  {
+    ERROR_LOG_FMT(VIDEO, "Asset '{}' error - raw texture expected to have one file mapped!",
+                  asset_id);
     return {};
+  }
   const auto& asset_path = asset_map.begin()->second;
 
   const auto last_loaded_time = std::filesystem::last_write_time(asset_path);
@@ -60,9 +64,12 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
   Common::ToLower(&ext);
   if (ext == ".dds")
   {
-    LoadDDSTexture(data, asset_path.string());
-    if (data->m_levels.empty()) [[unlikely]]
+    if (!LoadDDSTexture(data, asset_path.string()))
+    {
+      ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load dds texture!", asset_id);
       return {};
+    }
+
     if (!LoadMips(asset_path, data))
       return {};
 
@@ -75,13 +82,18 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
       data->m_levels.push_back({});
 
     if (!LoadPNGTexture(&data->m_levels[0], asset_path.string()))
+    {
+      ERROR_LOG_FMT(VIDEO, "Asset '{}' error - could not load png texture!", asset_id);
       return {};
+    }
+
     if (!LoadMips(asset_path, data))
       return {};
 
     return LoadInfo{GetAssetSize(*data), last_loaded_time};
   }
 
+  ERROR_LOG_FMT(VIDEO, "Asset '{}' error - extension '{}' unknown!", asset_id, ext);
   return {};
 }
 

From afa498fa2ff1b88b851cfa53d96ba04d241f2ffe Mon Sep 17 00:00:00 2001
From: iwubcode <iwubcode@users.noreply.github.com>
Date: Thu, 8 Jun 2023 22:20:52 -0500
Subject: [PATCH 2/2] VideoCommon: update DirectFilesystemAssetLibrary to not
 throw exceptions when a file no longer exists

---
 .../Assets/DirectFilesystemAssetLibrary.cpp       | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp b/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
index bbb3a644f5..9c126bf9e1 100644
--- a/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
+++ b/Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
@@ -4,6 +4,7 @@
 #include "VideoCommon/Assets/DirectFilesystemAssetLibrary.h"
 
 #include <algorithm>
+#include <fmt/os.h>
 
 #include "Common/FileUtil.h"
 #include "Common/Logging/Log.h"
@@ -35,7 +36,10 @@ DirectFilesystemAssetLibrary::GetLastAssetWriteTime(const AssetID& asset_id) con
     CustomAssetLibrary::TimeType max_entry;
     for (const auto& [key, value] : asset_map_path)
     {
-      const auto tp = std::filesystem::last_write_time(value);
+      std::error_code ec;
+      const auto tp = std::filesystem::last_write_time(value, ec);
+      if (ec)
+        continue;
       if (tp > max_entry)
         max_entry = tp;
     }
@@ -59,7 +63,14 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
   }
   const auto& asset_path = asset_map.begin()->second;
 
-  const auto last_loaded_time = std::filesystem::last_write_time(asset_path);
+  std::error_code ec;
+  const auto last_loaded_time = std::filesystem::last_write_time(asset_path, ec);
+  if (ec)
+  {
+    ERROR_LOG_FMT(VIDEO, "Asset '{}' error - failed to get last write time with error '{}'!",
+                  asset_id, ec);
+    return {};
+  }
   auto ext = asset_path.extension().string();
   Common::ToLower(&ext);
   if (ext == ".dds")