From efa9704c684ecad98341799245458844e7577010 Mon Sep 17 00:00:00 2001
From: Vito Caputo <vcaputo@pengaru.com>
Date: Sun, 31 Dec 2023 12:28:48 -0800
Subject: modules/swab: trivial optimizations
Just moving some unnecessary computation and derefs out of the
inner render loop... nothing major, good for about +1 FPS here.
---
src/modules/swab/swab.c | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
(limited to 'src/modules')
diff --git a/src/modules/swab/swab.c b/src/modules/swab/swab.c
index 343eb04..c4f1c91 100644
--- a/src/modules/swab/swab.c
+++ b/src/modules/swab/swab.c
@@ -105,6 +105,7 @@ static void swab_render_fragment(til_module_context_t *context, til_stream_t *st
{
swab_context_t *ctxt = (swab_context_t *)context;
til_fb_fragment_t *fragment = *fragment_ptr;
+ unsigned frag_w = fragment->width, frag_h = fragment->height;
float cos_r = cos(ctxt->r);
float sin_r = sin(ctxt->r);
@@ -112,12 +113,13 @@ static void swab_render_fragment(til_module_context_t *context, til_stream_t *st
float z2 = sin_r;
float xscale = 1.f / (float)fragment->frame_width;
float yscale = 1.f / (float)fragment->frame_height;
+ float yscaled;
- for (int y = fragment->y; y < fragment->y + fragment->height; y++) {
- float yscaled = (float)y * yscale;
+ yscaled = (float)fragment->y * yscale;
+ for (unsigned y = 0; y < frag_h; y++, yscaled += yscale) {
+ float xscaled = (float)fragment->x * xscale;
- for (int x = fragment->x; x < fragment->x + fragment->width; x++) {
- float xscaled = (float)x * xscale;
+ for (unsigned x = 0; x < frag_w; x++, xscaled += xscale) {
color_t color;
uint32_t pixel;
float t;
@@ -129,7 +131,7 @@ static void swab_render_fragment(til_module_context_t *context, til_stream_t *st
color.b = din(ctxt->din, &(v3f_t){ .x = xscaled * .81f, .y = yscaled * .81f, .z = z2 }) * t;
pixel = color_to_uint32(color);
- til_fb_fragment_put_pixel_unchecked(fragment, 0, x, y, pixel);
+ til_fb_fragment_put_pixel_unchecked(fragment, 0, fragment->x + x, fragment->y + y, pixel);
}
}
}
--
cgit v1.2.3