From 44cda56d350efd0bd997db144e61b060cd0df0da Mon Sep 17 00:00:00 2001
From: Triang3l <triang3l@yandex.ru>
Date: Sun, 8 May 2022 19:41:11 +0300
Subject: [PATCH] [GPU] Handle kRegisters and kGammaRamp in the trace viewer

---
 src/xenia/gpu/trace_reader.h  | 30 ------------------------------
 src/xenia/gpu/trace_viewer.cc | 12 ++++++++++++
 2 files changed, 12 insertions(+), 30 deletions(-)

diff --git a/src/xenia/gpu/trace_reader.h b/src/xenia/gpu/trace_reader.h
index d1b51e4cd..7ee65a283 100644
--- a/src/xenia/gpu/trace_reader.h
+++ b/src/xenia/gpu/trace_reader.h
@@ -20,36 +20,6 @@
 namespace xe {
 namespace gpu {
 
-// void Foo() {
-//  auto trace_ptr = trace_data;
-//  while (trace_ptr < trace_data + trace_size) {
-//    auto cmd_type = *reinterpret_cast<const TraceCommandType*>(trace_ptr);
-//    switch (cmd_type) {
-//      case TraceCommandType::kPrimaryBufferStart:
-//        break;
-//      case TraceCommandType::kPrimaryBufferEnd:
-//        break;
-//      case TraceCommandType::kIndirectBufferStart:
-//        break;
-//      case TraceCommandType::kIndirectBufferEnd:
-//        break;
-//      case TraceCommandType::kPacketStart:
-//        break;
-//      case TraceCommandType::kPacketEnd:
-//        break;
-//      case TraceCommandType::kMemoryRead:
-//        break;
-//      case TraceCommandType::kMemoryWrite:
-//        break;
-//      case TraceCommandType::kEvent:
-//        break;
-//    }
-//    /*trace_ptr = graphics_system->PlayTrace(
-//    trace_ptr, trace_size - (trace_ptr - trace_data),
-//    GraphicsSystem::TracePlaybackMode::kBreakOnSwap);*/
-//  }
-//}
-
 class TraceReader {
  public:
   struct CommandBuffer {
diff --git a/src/xenia/gpu/trace_viewer.cc b/src/xenia/gpu/trace_viewer.cc
index 660f24805..a4733ecbf 100644
--- a/src/xenia/gpu/trace_viewer.cc
+++ b/src/xenia/gpu/trace_viewer.cc
@@ -408,6 +408,18 @@ void TraceViewer::DrawPacketDisassemblerUI() {
         }
         break;
       }
+      case TraceCommandType::kRegisters: {
+        auto cmd = reinterpret_cast<const RegistersCommand*>(trace_ptr);
+        trace_ptr += sizeof(*cmd) + cmd->encoded_length;
+        // ImGui::BulletText("Registers");
+        break;
+      }
+      case TraceCommandType::kGammaRamp: {
+        auto cmd = reinterpret_cast<const GammaRampCommand*>(trace_ptr);
+        trace_ptr += sizeof(*cmd) + cmd->encoded_length;
+        // ImGui::BulletText("GammaRamp");
+        break;
+      }
     }
   }
   ImGui::EndChild();