diff options
Diffstat (limited to 'src/modules/droste')
| -rw-r--r-- | src/modules/droste/droste.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/src/modules/droste/droste.c b/src/modules/droste/droste.c index 9475df7..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; @@ -202,16 +206,8 @@ static void droste_prepare_frame(til_module_context_t *context, til_stream_t *st /* discard the snapshot which will prevent doing anything this frame, * since it doesn't match the incoming fragment (like a resize situation) */ - ctxt->snapshot = til_fb_fragment_reclaim(ctxt->snapshot); - - return; + ctxt->snapshot = til_fb_fragment_reclaim(snapshot); } - - /* TODO: if we're not used as an overlay, here'd be a good place to generate something or - * just use another module as a base layer... until we do something sensible here, we should - * keep this as an experimental module so it doesn't get used by automation as a base layer.. - * it also needs something to show in montage. - */ } } |