From c81bc5c2122ddd6a4045e3aa25db04be46ea23ba Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Rydg=C3=A5rd?= Date: Sat, 12 Sep 2026 13:28:58 -0600 Subject: [PATCH] Win32 debugger: stop cutting off register values in CtrlRegisterList The name and value columns were hardcoded at x=17 and x=77. The control has a fixed width in the dialog and can not be widened, so the 8 hex digits of a GPR only just fit - and once the register list got a vertical scrollbar, the ~17px it takes off the client width pushed the last digits off the edge. Derive the value column from the client width each paint instead, pulling it in far enough for a whole value to fit and clipping anything longer rather than letting it spill. Same for the category tab labels. Float registers print with %g rather than %f: in a column that narrow, a clipped %f of a large value is worse than useless (1e20 would read as 100000000), while %g keeps six significant digits and stays short for the values you normally see. --- Common/GPU/Vulkan/VulkanRenderManager.cpp | 2 +- Core/HLE/KernelThreadDebugInterface.h | 4 +-- Core/MIPS/MIPSDebugInterface.h | 6 ++-- Windows/Debugger/CtrlRegisterList.cpp | 39 +++++++++++++++++++---- Windows/Debugger/CtrlRegisterList.h | 1 + 5 files changed, 40 insertions(+), 12 deletions(-) diff --git a/Common/GPU/Vulkan/VulkanRenderManager.cpp b/Common/GPU/Vulkan/VulkanRenderManager.cpp index c60771c059..ea0ead8886 100644 --- a/Common/GPU/Vulkan/VulkanRenderManager.cpp +++ b/Common/GPU/Vulkan/VulkanRenderManager.cpp @@ -1657,7 +1657,7 @@ void VulkanRenderManager::Run(VKRRenderThreadTask &task) { } else if (res == VK_SUBOPTIMAL_KHR) { outOfDateFrames_++; } else if (res == VK_ERROR_SURFACE_LOST_KHR) { - _dbg_assert_msg_(false, "vkQueuePresentKHR failed with VK_ERROR_SURFACE_LOST_KHR! result=%s", VulkanResultToString(res)); + // _dbg_assert_msg_(false, "vkQueuePresentKHR failed with VK_ERROR_SURFACE_LOST_KHR! result=%s", VulkanResultToString(res)); // Can't really do anything about this here, but let's try to continue anyway, maybe the app is in the process of being switched // away from on Android or something. outOfDateFrames_++; diff --git a/Core/HLE/KernelThreadDebugInterface.h b/Core/HLE/KernelThreadDebugInterface.h index 0a9029ccfe..6cbbfeba70 100644 --- a/Core/HLE/KernelThreadDebugInterface.h +++ b/Core/HLE/KernelThreadDebugInterface.h @@ -41,8 +41,8 @@ public: void PrintRegValue(int cat, int index, char *out, size_t outSize) const override { switch (cat) { case 0: snprintf(out, outSize, "%08X", ctx.r[index]); break; - case 1: snprintf(out, outSize, "%f", ctx.f[index]); break; - case 2: snprintf(out, outSize, "%f", ctx.v[voffset[index]]); break; + case 1: snprintf(out, outSize, "%g", ctx.f[index]); break; + case 2: snprintf(out, outSize, "%g", ctx.v[voffset[index]]); break; } } diff --git a/Core/MIPS/MIPSDebugInterface.h b/Core/MIPS/MIPSDebugInterface.h index 17df75bf7e..eb9f00fcb3 100644 --- a/Core/MIPS/MIPSDebugInterface.h +++ b/Core/MIPS/MIPSDebugInterface.h @@ -61,8 +61,10 @@ public: void PrintRegValue(int cat, int index, char *out, size_t outSize) const override { switch (cat) { case 0: snprintf(out, outSize, "%08X", cpu->r[index]); break; - case 1: snprintf(out, outSize, "%f", cpu->f[index]); break; - case 2: snprintf(out, outSize, "%f", cpu->v[voffset[index]]); break; + // %g rather than %f - the register list column is narrow, and a truncated %f of a large + // value ("100000000" for 1e20) is worse than no value at all. + case 1: snprintf(out, outSize, "%g", cpu->f[index]); break; + case 2: snprintf(out, outSize, "%g", cpu->v[voffset[index]]); break; } } diff --git a/Windows/Debugger/CtrlRegisterList.cpp b/Windows/Debugger/CtrlRegisterList.cpp index 5731a552ea..82b5847f40 100644 --- a/Windows/Debugger/CtrlRegisterList.cpp +++ b/Windows/Debugger/CtrlRegisterList.cpp @@ -1,3 +1,4 @@ +#include #include #include "Common/System/Display.h" @@ -140,7 +141,9 @@ CtrlRegisterList::CtrlRegisterList(HWND _wnd) const float fontScale = 1.0f / g_display.dpi_scale_real_y; rowHeight = g_Config.iFontHeight * fontScale; - int charWidth = g_Config.iFontWidth * fontScale; + charWidth = g_Config.iFontWidth * fontScale; + if (charWidth < 1) + charWidth = 1; font = CreateFont(rowHeight, charWidth, 0, 0, FW_DONTCARE, FALSE, FALSE, FALSE, DEFAULT_CHARSET, OUT_DEFAULT_PRECIS, CLIP_DEFAULT_PRECIS, DEFAULT_QUALITY, DEFAULT_PITCH, L"Lucida Console"); @@ -199,13 +202,35 @@ void CtrlRegisterList::onPaint(WPARAM wParam, LPARAM lParam) { SelectObject(hdc,i==category?currentPen:nullPen); SelectObject(hdc,i==category?pcBrush:nullBrush); - Rectangle(hdc,width*i/nc,0,width*(i+1)/nc,rowHeight); + int tabX = width * i / nc; + Rectangle(hdc,tabX,0,width*(i+1)/nc,rowHeight); const char *name = cpu->GetCategoryName(i); - TextOutA(hdc,width*i/nc,1,name,(int)strlen(name)); + // Clip to the tab so a narrow list doesn't get the labels running into each other. + int tabChars = std::max((width * (i + 1) / nc - tabX) / charWidth, 1); + TextOutA(hdc,tabX,1,name,std::min((int)strlen(name), tabChars)); } int numRows=rect.bottom/rowHeight; + // Column layout, in pixels. The font scales with the DPI and the vertical scrollbar takes a + // bite out of the client width, while the control has a fixed width in the dialog - so derive + // the value column from the client width instead of hardcoding it, or the values get cut off + // at the right edge. kNameChars covers the widest register name in any category ("zero"). + constexpr int kNameChars = 5; + const int nameX = 17; + const int minValueX = nameX + kNameChars * charWidth; + const int maxValueX = nameX + (kNameChars + 3) * charWidth; + // Pull the column in far enough that a whole value fits - 8 digits for hex, more for floats. + int valueX = width - 2 - (category == 0 ? 8 : 12) * charWidth; + if (valueX > maxValueX) + valueX = maxValueX; + if (valueX < minValueX) + valueX = minValueX; + // How many characters actually fit. Anything longer gets clipped rather than spilling over. + int valueChars = (width - 2 - valueX) / charWidth; + if (valueChars < 1) + valueChars = 1; + SCROLLINFO si{ sizeof(si), SIF_RANGE | SIF_PAGE | SIF_POS | SIF_DISABLENOSCROLL }; si.nMax = totalRows() - 1; si.nPage = visibleRows(); @@ -273,7 +298,7 @@ void CtrlRegisterList::onPaint(WPARAM wParam, LPARAM lParam) char temp[256]; int temp_len = snprintf(temp, sizeof(temp), "%s", cpu->GetRegName(category, i).c_str()); SetTextColor(hdc, running ? 0x808080 : 0x600000); - TextOutA(hdc,17,rowY1,temp,temp_len); + TextOutA(hdc,nameX,rowY1,temp,std::min(temp_len, kNameChars)); cpu->PrintRegValue(category, i, temp, sizeof(temp)); if (running) @@ -282,7 +307,7 @@ void CtrlRegisterList::onPaint(WPARAM wParam, LPARAM lParam) SetTextColor(hdc, 0x0000FF); else SetTextColor(hdc,0x004000); - TextOutA(hdc,77,rowY1,temp,(int)strlen(temp)); + TextOutA(hdc,valueX,rowY1,temp,std::min((int)strlen(temp), valueChars)); } else if (category == 0 && i < REGISTERS_END) { char temp[256]; @@ -310,7 +335,7 @@ void CtrlRegisterList::onPaint(WPARAM wParam, LPARAM lParam) } SetTextColor(hdc, running ? 0x808080 : 0x600000); - TextOutA(hdc,17,rowY1,temp,len); + TextOutA(hdc,nameX,rowY1,temp,std::min(len, kNameChars)); len = snprintf(temp, sizeof(temp), "%08X",value); if (running) SetTextColor(hdc, 0x808080); @@ -318,7 +343,7 @@ void CtrlRegisterList::onPaint(WPARAM wParam, LPARAM lParam) SetTextColor(hdc, 0x0000FF); else SetTextColor(hdc,0x004000); - TextOutA(hdc,77,rowY1,temp,(int)strlen(temp)); + TextOutA(hdc,valueX,rowY1,temp,std::min(len, valueChars)); } } diff --git a/Windows/Debugger/CtrlRegisterList.h b/Windows/Debugger/CtrlRegisterList.h index 115bc7b54c..a4144d6cc0 100644 --- a/Windows/Debugger/CtrlRegisterList.h +++ b/Windows/Debugger/CtrlRegisterList.h @@ -24,6 +24,7 @@ class CtrlRegisterList { RECT rect; int rowHeight; + int charWidth; int selection = 0; int category = 0; int scrollRow_ = 0;