From 1bb321c41ff50d0e19207a7edf641a0760cf4a35 Mon Sep 17 00:00:00 2001
From: Connor McLaughlin <stenzek@gmail.com>
Date: Tue, 27 Dec 2022 22:16:17 +1000
Subject: [PATCH] MDEC: Don't round when texture replacements are enabled

This changes texture hashes, and people got annoyed :/
---
 src/core/mdec.cpp | 53 ++++++++++++++++++++++++++++++++++-------------
 1 file changed, 39 insertions(+), 14 deletions(-)

diff --git a/src/core/mdec.cpp b/src/core/mdec.cpp
index b89f21e853..38b398225d 100644
--- a/src/core/mdec.cpp
+++ b/src/core/mdec.cpp
@@ -634,24 +634,49 @@ void MDEC::CopyOutBlock(void* param, TickCount ticks, TickCount ticks_late)
 
     case DataOutputDepth_15Bit:
     {
-      const u32 a = ZeroExtend32(s_status.data_output_bit15.GetValue()) << 15;
-      for (u32 i = 0; i < static_cast<u32>(s_block_rgb.size());)
+      // people have made texture packs using the old conversion routines.. best to just leave them be.
+      if (g_settings.texture_replacements.enable_vram_write_replacements ||
+          g_settings.texture_replacements.dump_vram_writes)
       {
+        const u16 a = ZeroExtend16(s_status.data_output_bit15.GetValue()) << 15;
+        for (u32 i = 0; i < static_cast<u32>(s_block_rgb.size());)
+        {
+          u32 color = s_block_rgb[i++];
+          u16 r = Truncate16((color >> 3) & 0x1Fu);
+          u16 g = Truncate16((color >> 11) & 0x1Fu);
+          u16 b = Truncate16((color >> 19) & 0x1Fu);
+          const u16 color15a = r | (g << 5) | (b << 10) | (a << 15);
+
+          color = s_block_rgb[i++];
+          r = Truncate16((color >> 3) & 0x1Fu);
+          g = Truncate16((color >> 11) & 0x1Fu);
+          b = Truncate16((color >> 19) & 0x1Fu);
+          const u16 color15b = r | (g << 5) | (b << 10) | (a << 15);
+
+          s_data_out_fifo.Push(ZeroExtend32(color15a) | (ZeroExtend32(color15b) << 16));
+        }
+      }
+      else
+      {
+        const u32 a = ZeroExtend32(s_status.data_output_bit15.GetValue()) << 15;
+        for (u32 i = 0; i < static_cast<u32>(s_block_rgb.size());)
+        {
 #define E8TO5(color) (std::min<u32>((((color) + 4) >> 3), 0x1F))
-        u32 color = s_block_rgb[i++];
-        u32 r = E8TO5(color & 0xFFu);
-        u32 g = E8TO5((color >> 8) & 0xFFu);
-        u32 b = E8TO5((color >> 16) & 0xFFu);
-        const u32 color15a = r | (g << 5) | (b << 10) | a;
-
-        color = s_block_rgb[i++];
-        r = E8TO5(color & 0xFFu);
-        g = E8TO5((color >> 8) & 0xFFu);
-        b = E8TO5((color >> 16) & 0xFFu);
-        const u32 color15b = r | (g << 5) | (b << 10) | a;
+          u32 color = s_block_rgb[i++];
+          u32 r = E8TO5(color & 0xFFu);
+          u32 g = E8TO5((color >> 8) & 0xFFu);
+          u32 b = E8TO5((color >> 16) & 0xFFu);
+          const u32 color15a = r | (g << 5) | (b << 10) | a;
+
+          color = s_block_rgb[i++];
+          r = E8TO5(color & 0xFFu);
+          g = E8TO5((color >> 8) & 0xFFu);
+          b = E8TO5((color >> 16) & 0xFFu);
+          const u32 color15b = r | (g << 5) | (b << 10) | a;
 #undef E8TO5
 
-        s_data_out_fifo.Push(color15a | (color15b << 16));
+          s_data_out_fifo.Push(color15a | (color15b << 16));
+        }
       }
     }
     break;