Examples on appdev.App no longer call display_drv.show() themselves

appdev.App drives periodic show() for any display whose needs_refresh is
True, and as of pydevices 0d6350a FBDisplay computes that from the
underlying display rather than inheriting False. So on a MicroPython
dotclock panel the App now presents on its own, and 81 manual show() calls
across 35 examples were doing work that is already done.

Only examples that build an App were touched. The seven that call show()
with no App - framebuf_simpletest, logo, pbm_simpletest and friends - keep
theirs: nothing else would ever present for them.

Three files had show() as the only statement in a block, and those were
rewritten rather than left with a bare pass: piano.py loses the 'changed'
flag that existed only to decide whether to present, scroll_touch_test.py
loses a guard that guarded nothing else, and tower_climb.py keeps
_present() as an explicit no-op because three call sites use it as a seam.

Every changed file re-parsed after editing.

Author: bdbarnett

lib/examples/alien/alien.py

@@ -88,7 +88,6 @@ def _tick(_=None):
             st["yd"] = -abs(st["yd"])
         st["col"], st["row"] = col, row
         tft_bitmap.bitmap(tft, alien, col, row)
-        tft.show()
         st["last_col"], st["old_row"] = col, row
 
     app.every(_tick, period=10, async_=app.timer_async)

lib/examples/apollo/apollo.py

@@ -57,12 +57,10 @@ def _pkg_dir(file):
 
 def _init_apollo():
     dsky.init_screen()
-    display_drv.show()
 
     dsky.write_string("42", dsky.prog_pos)
     dsky.write_string("01", dsky.verb_pos)
     dsky.write_string("23", dsky.noun_pos)
-    display_drv.show()
 
 
 def _update_time():
@@ -75,15 +73,13 @@ def _update_time():
         _last_time = (y, mo, d, h, m, s)
         gc.collect()
         dsky.write_string(f"{mem-mem_free():7}", dsky.data3_pos)
-        display_drv.show()
 
 
 def _scroll_step():
     global _scrolling, _scroll_i
     if not _scrolling:
         return
     display_drv.vscsad(_scroll_i)
-    display_drv.show()
     _scroll_i += 1
     if _scroll_i >= _scroll_end:
         _scrolling = False
@@ -105,7 +101,6 @@ def _key_release():
         dsky.set_button(_pending_key, False)
         _pending_key = None
     dsky.set_acty(False)
-    display_drv.show()
     _key_busy = False
     _key_release_at = None
 

lib/examples/appdev_encoder_test.py

@@ -22,7 +22,6 @@ def draw_line():
     color = color_byte << 8 | color_byte
     display_drv.fill_rect(0, 0, x_pos, thickness, color)
     display_drv.fill_rect(x_pos, 0, w - x_pos, thickness, bg_color)
-    display_drv.show()
 
 
 display_drv.vscsad(y_pos)
@@ -73,7 +72,6 @@ def _on_wheel(e):
         delta = steps * steps * direction  # Quadratic acceleration
         y_pos = (y_pos + delta) % h
         display_drv.vscsad(y_pos)
-        display_drv.show()
     steps, _accum_x = _whole_steps(_accum_x, raw_x)
     if steps != 0:
         direction = factor if steps > 0 else -factor

lib/examples/appdev_touch_coords.py

@@ -48,7 +48,6 @@ def _show():
     display_drv.fill_rect(0, 0, W, H, BG)
     x, y = TARGETS[idx]
     _plus(x, y, FG)
-    display_drv.show()
     _out("[%d/%d] tap (%d, %d)" % (idx + 1, len(TARGETS), x, y))
 
 
@@ -71,7 +70,6 @@ def _on_up(e):
         if idx >= len(TARGETS):
             _out("done %s" % (results,))
             display_drv.fill_rect(0, 0, W, H, BG)
-            display_drv.show()
         else:
             _show()
             timer.sleep_ms(400)

lib/examples/appdev_touch_test.py

@@ -62,7 +62,6 @@ def _draw_target(x, y, half_width, half_height):
         y * half_height + ((half_height - 8) // 2),
         BG_COLOR,
     )
-    display_drv.show()
 
 
 def _clear_target(x, y, half_width, half_height):
@@ -73,7 +72,6 @@ def _clear_target(x, y, half_width, half_height):
         half_height - 1,
         BG_COLOR,
     )
-    display_drv.show()
 
 
 def _report(touch_rotation_table):
@@ -91,7 +89,6 @@ def _report(touch_rotation_table):
         (display_drv.height - 8) // 2,
         FG_COLOR,
     )
-    display_drv.show()
 
 
 # Callback-driven state machine (one path for sync and async).

lib/examples/bmp565_scroll.py

@@ -35,14 +35,12 @@ def _tick(_=None):
                 draw_bg(0, j, 0, j, height=1)
                 st["j"] = j + 1
                 if st["j"] >= display_drv.height:
-                    display_drv.show()
                     st["phase"] = "scroll"
                 return
         # scroll phase
         i = st["i"]
         display_drv.vscsad(i % display_drv.height)
         draw_bg(0, i % display_drv.height, 0, i % image.height)
-        display_drv.show()
         st["i"] = i + 1
 
     app.every(_tick, period=1, async_=app.timer_async)

lib/examples/bmp565_scroll_sprite.py

@@ -93,7 +93,6 @@ def _tick(_=None):
         draw_bg(i % display_drv.width, i % image.height, 1)
         st["i"] = i + 1
         if i < display_drv.width:
-            display_drv.show()
             return
 
         draw_x = st["scroll"] + st["char_x"]
@@ -116,7 +115,6 @@ def _tick(_=None):
                     bg,
                 )
                 st["shot_location"] = 0
-        display_drv.show()
 
     app.on(app.events.MOUSEMOTION, _on_motion)
     # ~20 fps once scrolling; first columns also tick so quit is always serviced.

lib/examples/bmp565_sprite.py

@@ -52,7 +52,6 @@ def draw_sprite(
 
 
 display_drv.fill(bg)
-display_drv.show()
 
 point = namedtuple("point", "x y")
 step = 3
@@ -101,7 +100,6 @@ def _tick(_=None):
         location = point(location.x + step, location.y)
     st["location"] = location
     draw_sprite(*location, pos, direction)
-    display_drv.show()
     st["pos_i"] = (st["pos_i"] + 1) % len(positions)
 
 

lib/examples/bouncing_balls.py

@@ -101,7 +101,6 @@ def _tick(_=None):
     for ball in balls:
         step(ball)
         pygraphics.circle(display_drv, int(ball.x), int(ball.y), ball.r, ball.color, True)
-    display_drv.show()
 
 
 app.every(_tick, period=10, async_=app.timer_async)

lib/examples/boxlines.py

@@ -70,7 +70,6 @@ def poll():
             height,
             palette.color565(getrandbits(8), getrandbits(8), getrandbits(8)),
         )
-        tft.show()
         return False
 
     return poll