From a6317ead69079a9145c662315bcb88b1e9dd68cf Mon Sep 17 00:00:00 2001 From: fallenoak Date: Mon, 17 Nov 2025 16:31:01 -0600 Subject: [PATCH] chore(console): improve comment clarity in ConsoleDetectGetResolutions --- src/console/Detect.cpp | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/console/Detect.cpp b/src/console/Detect.cpp index 9b409ce..441b0e1 100644 --- a/src/console/Detect.cpp +++ b/src/console/Detect.cpp @@ -7,7 +7,11 @@ void AddResolution(TSGrowableArray& resolutions, const C2iVector& res // TODO sometimes 640x480 is included, sometimes its ignored -- more than one function? void ConsoleDetectGetResolutions(TSGrowableArray& resolutions, int32_t widescreen) { - // Widescreen resolutions + // "Widescreen" resolutions + // + // This branch is just dynamically enumerating resolutions above a minimum size and aspect + // ratio as opposed to using a fixed list of common 1.33:1 resolutions. The name widescreen + // comes from the CVar that feeds the argument (see: SetupResolutions). if (widescreen) { TSGrowableArray monitorModes; @@ -19,12 +23,12 @@ void ConsoleDetectGetResolutions(TSGrowableArray& resolutions, int32_ auto& monitorMode = monitorModes[i]; auto& resolution = monitorMode.size; - // "Widescreen" aspect ratio + // Minimum aspect ratio if (static_cast(resolution.x) / static_cast(resolution.y) < 1.248f) { continue; } - // At least 640x480 + // Minimum resolution if (resolution.x < 640 || resolution.y < 480) { continue; }