From 3e3faa57e22610fa03050089b0e15c9df7fe832e Mon Sep 17 00:00:00 2001 From: Vito Caputo Date: Wed, 25 May 2022 09:34:06 -0700 Subject: modules/shapes: update TODO comment A bunch of these have just been done --- src/modules/shapes/shapes.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'src/modules') diff --git a/src/modules/shapes/shapes.c b/src/modules/shapes/shapes.c index a1a3342..8479c71 100644 --- a/src/modules/shapes/shapes.c +++ b/src/modules/shapes/shapes.c @@ -35,11 +35,11 @@ * * - Add more interesting shapes * - * - Expose parameters as settings - * * - Parameterize more things, stuff like twist for the radial shapes - * comes to mind. The simplistic CCW rotation of star/pinwheel should be - * more variable and exposed as settings, etc. + * comes to mind. Twist at glance seems substantially complicated + * actually, since things are no longer just pinched/stretch circles with + * a single radial test to check. It's like the non-convex polygon + * problem... * * - Go threaded, for ease of implementation this is currently simple * non-threaded code. In the checkers use case, the individual checkers @@ -47,12 +47,6 @@ * threaded there. It's just full-frame shapes situations where it * hurts. * - * - The presently static shapes like circle and rhombus could be simply - * rendered once @ context_create() into a dense buffer then copied at - * render_fragment() time. The current implementation is very naive and - * slow procedurally redrawing even these constant shapes. But the - * assumption is as more parameterizing is added, all the shapes will - * become dynamic. So there's no sense adding a cache. */ -- cgit v1.2.3