X-Git-Url: http://git.hungrycats.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=hacks%2Fxspirograph.c;h=3c241fc36fdc7ee36051931a3bcf4b3f379b3017;hb=4ade52359b6eba3621566dac79793a33aa4c915f;hp=a0e9792a8b7c833dea0aff5f76747b435f5818ca;hpb=c1b9b55ad8d59dc05ef55e316aebf5863e7dfa56;p=xscreensaver

diff --git a/hacks/xspirograph.c b/hacks/xspirograph.c
index a0e9792a..3c241fc3 100644
--- a/hacks/xspirograph.c
+++ b/hacks/xspirograph.c
@@ -45,8 +45,7 @@ struct state {
   int radius1, radius2;
   double divisor;
 
-  int new_layer;
-  int erasing;
+  enum curstate { NEW_LAYER, DRAW, ERASE1, ERASE2 } drawstate;
   eraser_state *eraser;
 };
 
@@ -193,7 +192,7 @@ xspirograph_init (Display *dpy, Window window)
 
   init_tsg (st);
   st->theta = 1;
-  st->new_layer = 1;
+  st->drawstate = NEW_LAYER;
 
   return st;
 }
@@ -223,62 +222,66 @@ xspirograph_draw (Display *dpy, Window window, void *closure)
 {
   struct state *st = (struct state *) closure;
   Bool free_color = False;
-  Bool flip_p;
-
-  /* 5 sec delay before starting the erase */
-  if (st->erasing == 2) {
-    st->erasing--;
-    return (st->long_delay == 0 ? 0 : 5000000);
-  }
-
-  /* erase, delaying 1/50th sec between frames */
-  if (st->erasing || st->eraser) {
-    st->erasing = 0;
-    st->eraser = erase_window (st->dpy, st->window, st->eraser);
-    if (st->eraser)
-      return 20000;
-    else
+  Bool flip_p = (st->counter & 1);
+  int i;
+
+  switch (st->drawstate) {
+    case ERASE1:
+      /* 5 sec delay before starting the erase */
+      st->drawstate = ERASE2;
+      /* shouldn't this use the configured long_delay value??? */
+      return (st->long_delay == 0 ? 0 : 5000000);
+
+    case ERASE2:
+      /* erase, delaying 1/50th sec between frames */
+      st->eraser = erase_window(st->dpy, st->window, st->eraser);
+      if (st->eraser)
+	/* shouldn't this be a configured pause??? */
+	return 20000;
+      st->drawstate = NEW_LAYER;
       /* just finished erasing -- leave screen black for 1 sec */
       return (st->long_delay == 0 ? 0 : 1000000);
-  }
-
-  flip_p = (st->counter & 1);
-
-  if (st->new_layer) {
-
-    st->new_layer = 0;
-    st->counter++;
-
-    if (st->counter > (2 * st->num_layers))
-      {
-        st->counter = 0;
 
-        if (free_color)
-          XFreeColors (st->dpy, st->xgwa.colormap, &st->color.pixel, 1, 0);
-
-        st->erasing = 2;
+    case DRAW:
+      /* most common case put in front */
+      for (i = 0; i < 1000; i++) {
+        if (go(st, st->radius1, (flip_p ? st->radius2 : -st->radius2),
+	       st->distance)) {
+	  st->drawstate = NEW_LAYER;
+	  break;
+	}
       }
-
-    if (! flip_p)
-      pick_new (st);
-
-    new_colors (st);
-    st->new_layer = 0;
-  }
-
-  {
-    int i;
-    for (i = 0; i < 1000; i++) {
-      if (go (st, st->radius1, 
-              (flip_p ? st->radius2 : -st->radius2),
-              st->distance)) {
-        st->new_layer = 1;
-        break;
+      /* Next draw is delayed sleep_time (initialized value)*/
+      return st->sub_sleep_time;
+
+    case NEW_LAYER:
+      /* Increment counter */
+      st->counter++;
+      if (st->counter > (2 * st->num_layers)) {
+	/* reset to zero, free, and erase next time through */
+	st->counter = 0;
+	if (free_color)
+	  XFreeColors (st->dpy, st->xgwa.colormap, &st->color.pixel, 1, 0);
+	st->drawstate = ERASE1;
+      } else {
+	/* first, third, fifth, ... time through */
+	if (!flip_p)
+	  pick_new (st);
+
+	new_colors (st);
+	st->drawstate = DRAW;
       }
-    }
+      /* No delay to the next draw */
+      return 0;
+
+    default:
+      /* OOPS!! */
+      fprintf(stderr, "%s: invalid state\n", progname);
+      exit(1);
   }
 
   return st->sub_sleep_time;
+  /* notreached */
 }
 
 
@@ -312,6 +315,9 @@ static const char *xspirograph_defaults [] = {
   "*subdelay:   	20000",
   "*layers:     	2",
   "*alwaysfinish:	false",
+#ifdef USE_IPHONE
+  "*ignoreRotation:     True",
+#endif
   0
 };