From 6d1f5785f94832a659be23860c1f76eeaa345f8a Mon Sep 17 00:00:00 2001 From: Vito Caputo Date: Sat, 14 Jun 2025 19:34:38 -0700 Subject: modules/droste: comment re: droste fragmenter bug --- src/modules/droste/droste.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/modules/droste/droste.c b/src/modules/droste/droste.c index 9cbfbac..717e513 100644 --- a/src/modules/droste/droste.c +++ b/src/modules/droste/droste.c @@ -112,6 +112,10 @@ static int droste_fragmenter(til_module_context_t *context, const til_fb_fragmen assert(fragment); assert(res_fragment); + /* This should be using half of the frame_{width,height}, but must be clipped + * to the fragment dimensions. As-is this is quite broken in scenarios like droste in a checkers filler + * when the perimeter checkers get clipped. FIXME FIXME FIXME + */ inset.width = fragment->width >> 1; inset.height = fragment->height >> 1; inset.frame_width = inset.width; -- cgit v1.2.3