From 7ef2c2fac38f983566267ebf36a981680d9868f0 Mon Sep 17 00:00:00 2001
From: Tyler Wilding <xTVaser@users.noreply.github.com>
Date: Thu, 25 Jan 2024 21:28:47 -0500
Subject: [PATCH] decomp: add a new config flag to disable var-name casts
 (#3338)

This is handy when you want to try comparing the decompiler's output
across games (ie. for checking if only naming differs)
---
 decompiler/config.cpp | 49 ++++++++++++++++++++++++-------------------
 decompiler/config.h   |  1 +
 2 files changed, 28 insertions(+), 22 deletions(-)

diff --git a/decompiler/config.cpp b/decompiler/config.cpp
index a77cf4709..e84df0571 100644
--- a/decompiler/config.cpp
+++ b/decompiler/config.cpp
@@ -97,6 +97,9 @@ Config make_config_via_json(nlohmann::json& json) {
   if (json.contains("read_spools")) {
     config.read_spools = json.at("read_spools").get<bool>();
   }
+  if (json.contains("ignore_var_name_casts")) {
+    config.ignore_var_name_casts = json.at("ignore_var_name_casts").get<bool>();
+  }
   if (json.contains("old_all_types_file")) {
     config.old_all_types_file = json.at("old_all_types_file").get<std::string>();
   }
@@ -156,29 +159,31 @@ Config make_config_via_json(nlohmann::json& json) {
       config.anon_function_types_by_obj_by_id[obj_file_name][id] = type_name;
     }
   }
-  auto var_names_json = read_json_file_from_config(json, "var_names_file");
-  for (auto& kv : var_names_json.items()) {
-    auto& function_name = kv.key();
-    auto arg = kv.value().find("args");
-    if (arg != kv.value().end()) {
-      for (auto& x : arg.value()) {
-        config.function_arg_names[function_name].push_back(x);
-      }
-    }
-
-    auto var = kv.value().find("vars");
-    if (var != kv.value().end()) {
-      for (auto& vkv : var->get<std::unordered_map<std::string, nlohmann::json>>()) {
-        LocalVarOverride override;
-        if (vkv.second.is_string()) {
-          override.name = vkv.second.get<std::string>();
-        } else if (vkv.second.is_array()) {
-          override.name = vkv.second[0].get<std::string>();
-          override.type = vkv.second[1].get<std::string>();
-        } else {
-          throw std::runtime_error("Invalid function var override.");
+  if (!config.ignore_var_name_casts) {
+    auto var_names_json = read_json_file_from_config(json, "var_names_file");
+    for (auto& kv : var_names_json.items()) {
+      auto& function_name = kv.key();
+      auto arg = kv.value().find("args");
+      if (arg != kv.value().end()) {
+        for (auto& x : arg.value()) {
+          config.function_arg_names[function_name].push_back(x);
+        }
+      }
+
+      auto var = kv.value().find("vars");
+      if (var != kv.value().end()) {
+        for (auto& vkv : var->get<std::unordered_map<std::string, nlohmann::json>>()) {
+          LocalVarOverride override;
+          if (vkv.second.is_string()) {
+            override.name = vkv.second.get<std::string>();
+          } else if (vkv.second.is_array()) {
+            override.name = vkv.second[0].get<std::string>();
+            override.type = vkv.second[1].get<std::string>();
+          } else {
+            throw std::runtime_error("Invalid function var override.");
+          }
+          config.function_var_overrides[function_name][vkv.first] = override;
         }
-        config.function_var_overrides[function_name][vkv.first] = override;
       }
     }
   }
diff --git a/decompiler/config.h b/decompiler/config.h
index 9b2ac66f1..a5f87cb78 100644
--- a/decompiler/config.h
+++ b/decompiler/config.h
@@ -121,6 +121,7 @@ struct Config {
   bool extract_collision = false;
   bool find_functions = false;
   bool read_spools = false;
+  bool ignore_var_name_casts = false;
 
   bool write_hex_near_instructions = false;
   bool hexdump_code = false;