home · contact · privacy
In curses client, don't flash outside of refresh loop.
authorChristian Heller <c.heller@plomlompom.de>
Sun, 15 Nov 2020 23:47:00 +0000 (00:47 +0100)
committerChristian Heller <c.heller@plomlompom.de>
Sun, 15 Nov 2020 23:47:00 +0000 (00:47 +0100)
rogue_chat_curses.py

index 2dfbdd1b6b76772877ac47b9db7c04b4e199e919..18977d2cd4b13e3867f1d52d55fff745143398d1 100755 (executable)
@@ -139,7 +139,7 @@ cmd_PORTAL.argtypes = 'yx_tuple:nonneg string'
 
 def cmd_PLAY_ERROR(game, msg):
     game.tui.log_msg('? ' + msg)
 
 def cmd_PLAY_ERROR(game, msg):
     game.tui.log_msg('? ' + msg)
-    game.tui.flash()
+    game.tui.flash = True
     game.tui.do_refresh = True
 cmd_PLAY_ERROR.argtypes = 'string'
 
     game.tui.do_refresh = True
 cmd_PLAY_ERROR.argtypes = 'string'
 
@@ -291,11 +291,9 @@ class TUI:
         self.force_instant_connect = True
         self.input_lines = []
         self.fov = ''
         self.force_instant_connect = True
         self.input_lines = []
         self.fov = ''
+        self.flash = False
         curses.wrapper(self.loop)
 
         curses.wrapper(self.loop)
 
-    def flash(self):
-        curses.flash()
-
     def connect(self):
 
         def handle_recv(msg):
     def connect(self):
 
         def handle_recv(msg):
@@ -431,7 +429,7 @@ class TUI:
                 self.explorer = target
                 self.query_info()
             else:
                 self.explorer = target
                 self.query_info()
             else:
-                self.flash()
+                self.flash = True
 
         def draw_history():
             lines = []
 
         def draw_history():
             lines = []
@@ -636,6 +634,9 @@ class TUI:
                 else:
                     self.send('PING')
                 last_ping = now
                 else:
                     self.send('PING')
                 last_ping = now
+            if self.flash:
+                curses.flash()
+                self.flash = False
             if self.do_refresh:
                 draw_screen()
                 self.do_refresh = False
             if self.do_refresh:
                 draw_screen()
                 self.do_refresh = False
@@ -748,7 +749,7 @@ class TUI:
                         self.host = self.game.portals[player.position]
                         self.reconnect()
                     else:
                         self.host = self.game.portals[player.position]
                         self.reconnect()
                     else:
-                        self.flash()
+                        self.flash = True
                         self.log_msg('? not standing on portal')
                 elif key in self.movement_keys and 'MOVE' in self.game.tasks:
                     self.send('TASK:MOVE ' + self.movement_keys[key])
                         self.log_msg('? not standing on portal')
                 elif key in self.movement_keys and 'MOVE' in self.game.tasks:
                     self.send('TASK:MOVE ' + self.movement_keys[key])