summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVito Caputo <vcaputo@pengaru.com>2022-05-25 09:34:06 -0700
committerVito Caputo <vcaputo@pengaru.com>2022-05-25 09:34:06 -0700
commit3e3faa57e22610fa03050089b0e15c9df7fe832e (patch)
tree6a2f55fd8fdcaa6d9b45d931949ac7f2f9af30d6
parent8cab00946cc06cb8e4925ba389f052b40298c751 (diff)
modules/shapes: update TODO comment
A bunch of these have just been done
-rw-r--r--src/modules/shapes/shapes.c14
1 files changed, 4 insertions, 10 deletions
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.
*/
© All Rights Reserved