From 758eb256b986a7ae07108b667c8e09a404a989a3 Mon Sep 17 00:00:00 2001
From: Sam Lantinga <[EMAIL REDACTED]>
Date: Thu, 20 Mar 2025 21:24:39 -0700
Subject: [PATCH] Fixed DebugLogRenderCommands() output
---
src/render/SDL_render.c | 152 ++++++++++++++++++++--------------------
1 file changed, 75 insertions(+), 77 deletions(-)
diff --git a/src/render/SDL_render.c b/src/render/SDL_render.c
index eac513f08cb7b..15949f75d6505 100644
--- a/src/render/SDL_render.c
+++ b/src/render/SDL_render.c
@@ -213,93 +213,91 @@ static SDL_INLINE void DebugLogRenderCommands(const SDL_RenderCommand *cmd)
SDL_Log("Render commands to flush:");
while (cmd) {
switch (cmd->command) {
- case SDL_RENDERCMD_NO_OP:
- SDL_Log(" %u. no-op", i++);
- break;
-
- case SDL_RENDERCMD_SETVIEWPORT:
- SDL_Log(" %u. set viewport (first=%u, rect={(%d, %d), %dx%d})", i++,
- (unsigned int) cmd->data.viewport.first,
- cmd->data.viewport.rect.x, cmd->data.viewport.rect.y,
- cmd->data.viewport.rect.w, cmd->data.viewport.rect.h);
- break;
-
- case SDL_RENDERCMD_SETCLIPRECT:
- SDL_Log(" %u. set cliprect (enabled=%s, rect={(%d, %d), %dx%d})", i++,
- cmd->data.cliprect.enabled ? "true" : "false",
- cmd->data.cliprect.rect.x, cmd->data.cliprect.rect.y,
- cmd->data.cliprect.rect.w, cmd->data.cliprect.rect.h);
- break;
+ case SDL_RENDERCMD_NO_OP:
+ SDL_Log(" %u. no-op", i++);
+ break;
- case SDL_RENDERCMD_SETDRAWCOLOR:
- SDL_Log(" %u. set draw color (first=%u, r=%d, g=%d, b=%d, a=%d, color_scale=%g)", i++,
- (unsigned int) cmd->data.color.first,
- (int) cmd->data.color.color.r, (int) cmd->data.color.color.g,
- (int) cmd->data.color.color.b, (int) cmd->data.color.color.a, cmd->data.color.color_scale);
- break;
+ case SDL_RENDERCMD_SETVIEWPORT:
+ SDL_Log(" %u. set viewport (first=%u, rect={(%d, %d), %dx%d})", i++,
+ (unsigned int)cmd->data.viewport.first,
+ cmd->data.viewport.rect.x, cmd->data.viewport.rect.y,
+ cmd->data.viewport.rect.w, cmd->data.viewport.rect.h);
+ break;
- case SDL_RENDERCMD_CLEAR:
- SDL_Log(" %u. clear (first=%u, r=%d, g=%d, b=%d, a=%d, color_scale=%g)", i++,
- (unsigned int) cmd->data.color.first,
- (int) cmd->data.color.color.r, (int) cmd->data.color.color.g,
- (int) cmd->data.color.color.b, (int) cmd->data.color.color.a, cmd->data.color.color_scale);
- break;
+ case SDL_RENDERCMD_SETCLIPRECT:
+ SDL_Log(" %u. set cliprect (enabled=%s, rect={(%d, %d), %dx%d})", i++,
+ cmd->data.cliprect.enabled ? "true" : "false",
+ cmd->data.cliprect.rect.x, cmd->data.cliprect.rect.y,
+ cmd->data.cliprect.rect.w, cmd->data.cliprect.rect.h);
+ break;
- case SDL_RENDERCMD_DRAW_POINTS:
- SDL_Log(" %u. draw points (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale);
- break;
+ case SDL_RENDERCMD_SETDRAWCOLOR:
+ SDL_Log(" %u. set draw color (first=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, color_scale=%g)", i++,
+ (unsigned int)cmd->data.color.first,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a, cmd->data.color.color_scale);
+ break;
- case SDL_RENDERCMD_DRAW_LINES:
- SDL_Log(" %u. draw lines (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale);
- break;
+ case SDL_RENDERCMD_CLEAR:
+ SDL_Log(" %u. clear (first=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, color_scale=%g)", i++,
+ (unsigned int)cmd->data.color.first,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a, cmd->data.color.color_scale);
+ break;
- case SDL_RENDERCMD_FILL_RECTS:
- SDL_Log(" %u. fill rects (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale);
- break;
+ case SDL_RENDERCMD_DRAW_POINTS:
+ SDL_Log(" %u. draw points (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale);
+ break;
- case SDL_RENDERCMD_COPY:
- SDL_Log(" %u. copy (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g, tex=%p)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
- break;
+ case SDL_RENDERCMD_DRAW_LINES:
+ SDL_Log(" %u. draw lines (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale);
+ break;
+ case SDL_RENDERCMD_FILL_RECTS:
+ SDL_Log(" %u. fill rects (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale);
+ break;
- case SDL_RENDERCMD_COPY_EX:
- SDL_Log(" %u. copyex (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g, tex=%p)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
- break;
+ case SDL_RENDERCMD_COPY:
+ SDL_Log(" %u. copy (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g, tex=%p)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
+ break;
- case SDL_RENDERCMD_GEOMETRY:
- SDL_Log(" %u. geometry (first=%u, count=%u, r=%d, g=%d, b=%d, a=%d, blend=%d, color_scale=%g, tex=%p)", i++,
- (unsigned int) cmd->data.draw.first,
- (unsigned int) cmd->data.draw.count,
- (int) cmd->data.draw.color.r, (int) cmd->data.draw.color.g,
- (int) cmd->data.draw.color.b, (int) cmd->data.draw.color.a,
- (int) cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
- break;
+ case SDL_RENDERCMD_COPY_EX:
+ SDL_Log(" %u. copyex (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g, tex=%p)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
+ break;
+ case SDL_RENDERCMD_GEOMETRY:
+ SDL_Log(" %u. geometry (first=%u, count=%u, r=%.2f, g=%.2f, b=%.2f, a=%.2f, blend=%d, color_scale=%g, tex=%p)", i++,
+ (unsigned int)cmd->data.draw.first,
+ (unsigned int)cmd->data.draw.count,
+ cmd->data.draw.color.r, cmd->data.draw.color.g,
+ cmd->data.draw.color.b, cmd->data.draw.color.a,
+ (int)cmd->data.draw.blend, cmd->data.draw.color_scale, cmd->data.draw.texture);
+ break;
}
cmd = cmd->next;
}