From 45f41be8f0f4346e7587011cbd57a0ac7d6ae0d7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Henrik=20Rydg=C3=A5rd?= <hrydgard@gmail.com>
Date: Fri, 5 Jun 2026 17:55:28 +0200
Subject: [PATCH] Add a debug window to check active compat flags

---
 Common/UI/PopupScreens.h |  2 +-
 Core/Compatibility.cpp   | 21 ++++++++++++++++++++-
 Core/Compatibility.h     |  4 ++++
 UI/DevScreens.cpp        | 22 ++++++++++++++++++++++
 4 files changed, 47 insertions(+), 2 deletions(-)

diff --git a/Common/UI/PopupScreens.h b/Common/UI/PopupScreens.h
index ec86de344d..b0703acbbe 100644
--- a/Common/UI/PopupScreens.h
+++ b/Common/UI/PopupScreens.h
@@ -130,7 +130,7 @@ private:
 
 class MessagePopupScreen : public PopupScreen {
 public:
-	MessagePopupScreen(std::string_view title, std::string_view message, std::string_view button1, std::string_view button2, std::function<void(bool)> callback)
+	MessagePopupScreen(std::string_view title, std::string_view message, std::string_view button1, std::string_view button2, std::function<void(bool)> callback = nullptr)
 		: PopupScreen(title, button1, button2), message_(message), callback_(callback) {}
 
 	const char *tag() const override { return "MessagePopupScreen"; }
diff --git a/Core/Compatibility.cpp b/Core/Compatibility.cpp
index 5571e2e3d9..9116968a41 100644
--- a/Core/Compatibility.cpp
+++ b/Core/Compatibility.cpp
@@ -44,6 +44,7 @@ void Compatibility::Load(const std::string &gameID) {
 		// This loads from assets.
 		if (compat.LoadFromVFS(g_VFS, "compat.ini")) {
 			CheckSettings(compat, gameID);
+			filesLoaded_.push_back("assets/compat.ini");
 		} else {
 			auto e = GetI18NCategory(I18NCat::ERRORS);
 			std::string msg = ApplySafeSubstitutions(e->T("File not found: %1"), "compat.ini");
@@ -57,6 +58,7 @@ void Compatibility::Load(const std::string &gameID) {
 		Path path = GetSysDirectory(DIRECTORY_SYSTEM) / "compat.ini";
 		if (compat2.Load(path)) {
 			CheckSettings(compat2, gameID);
+			filesLoaded_.push_back(path.ToString());
 		}
 	}
 
@@ -67,6 +69,7 @@ void Compatibility::Load(const std::string &gameID) {
 			// This loads from assets.
 			if (compat.LoadFromVFS(g_VFS, "compatvr.ini")) {
 				CheckVRSettings(compat, gameID);
+				filesLoaded_.push_back("assets/compatvr.ini");
 			}
 		}
 
@@ -76,6 +79,7 @@ void Compatibility::Load(const std::string &gameID) {
 			Path path = GetSysDirectory(DIRECTORY_SYSTEM) / "compatvr.ini";
 			if (compat2.Load(path)) {
 				CheckVRSettings(compat2, gameID);
+				filesLoaded_.push_back(path.ToString());
 			}
 		}
 	}
@@ -85,6 +89,7 @@ void Compatibility::Clear() {
 	memset(&flags_, 0, sizeof(flags_));
 	memset(&vrCompat_, 0, sizeof(vrCompat_));
 	activeList_.clear();
+	filesLoaded_.clear();
 }
 
 void Compatibility::CheckSettings(IniFile &iniFile, const std::string &gameID) {
@@ -191,10 +196,14 @@ void Compatibility::CheckSetting(IniFile &iniFile, const std::string &gameID, co
 		section->Get("ALL", &all);
 		if (all) {
 			*flag = true;
+		}
+
+		if (*flag) {
 			if (!activeList_.empty()) {
 				activeList_ += "\n";
 			}
 			activeList_ += option;
+			activeList_ += ": True";
 		}
 	}
 }
@@ -204,6 +213,11 @@ void Compatibility::CheckSetting(IniFile &iniFile, const std::string &gameID, co
 	Section *section = iniFile.GetSection(option);
 	if (section && section->Get(gameID.c_str(), &value)) {
 		*flag = stof(value);
+
+		if (!activeList_.empty()) {
+			activeList_ += "\n";
+		}
+		activeList_ += std::string(option) + ": " + std::to_string(*flag);
 	}
 }
 
@@ -211,6 +225,11 @@ void Compatibility::CheckSetting(IniFile &iniFile, const std::string &gameID, co
 	std::string value;
 	Section *section = iniFile.GetSection(option);
 	if (section && section->Get(gameID.c_str(), &value)) {
-		*flag = stof(value);
+		*flag = stoi(value);
+
+		if (!activeList_.empty()) {
+			activeList_ += ":" + std::to_string(*flag) + "\n";
+		}
+		activeList_ += std::string(option) + ": " + std::to_string(*flag);
 	}
 }
diff --git a/Core/Compatibility.h b/Core/Compatibility.h
index 3098f74bc5..dbee85f97c 100644
--- a/Core/Compatibility.h
+++ b/Core/Compatibility.h
@@ -18,6 +18,7 @@
 #pragma once
 
 #include <string>
+#include <vector>
 #include <cstdint>
 #include <set>
 
@@ -152,6 +153,8 @@ public:
 	const std::string &GetActiveFlagsString() const {
 		return activeList_;
 	}
+	const std::vector<std::string> &filesLoaded() const { return filesLoaded_; }
+
 
 private:
 	void Clear();
@@ -165,4 +168,5 @@ private:
 	VRCompat vrCompat_{};
 	std::set<std::string> ignored_;
 	std::string activeList_;
+	std::vector<std::string> filesLoaded_;
 };
diff --git a/UI/DevScreens.cpp b/UI/DevScreens.cpp
index 82400ac8a2..44a0cae30e 100644
--- a/UI/DevScreens.cpp
+++ b/UI/DevScreens.cpp
@@ -198,6 +198,28 @@ void DevMenuScreen::CreatePopupContents(UI::ViewGroup *parent) {
 		});
 	}
 
+	items->Add(new Choice(dev->T("Compatibility flags")))->OnClick.Add([this](UI::EventParams &e) {
+		auto dev = GetI18NCategory(I18NCat::DEVELOPER);
+		auto di = GetI18NCategory(I18NCat::DIALOG);
+		std::string activeFlags = PSP_CoreParameter().compat.GetActiveFlagsString();
+		std::string compatText = join(di->T("Enabled"), "\n");
+		compatText += activeFlags.empty() ? "None" : activeFlags;
+		compatText += "\n\n";
+		std::string ignored = g_Config.sIgnoreCompatSettings;
+		compatText += join(di->T("Ignored"), ":");
+		if (ignored.empty()) {
+			compatText += " None";
+		} else {
+			compatText += "\n" + ignored;
+		}
+		compatText += "\n\n";
+		compatText += "Loaded files: ";
+		for (const auto &file : PSP_CoreParameter().compat.filesLoaded()) {
+			compatText += "\n" + file;
+		}
+		screenManager()->push(new MessagePopupScreen(dev->T("Compatibility flags"), compatText, di->T("OK"), ""));
+	});
+
 	scroll->Add(items);
 	parent->Add(scroll);