From: Christian Heller <c.heller@plomlompom.de>
Date: Sat, 7 Nov 2015 20:54:31 +0000 (+0100)
Subject: New client: Reorganize code.
X-Git-Tag: tce~255
X-Git-Url: https://plomlompom.com/repos/%7B%7Bprefix%7D%7D/%22https:/validator.w3.org/%7B%7Btodo.comment%7D%7D?a=commitdiff_plain;h=8bd3d9463906cacf18fc8e91b302f87d6d01c59d;p=plomrogue

New client: Reorganize code.
---

diff --git a/client_prototype.py b/client_prototype.py
index c828f7d..2dd666f 100644
--- a/client_prototype.py
+++ b/client_prototype.py
@@ -171,9 +171,9 @@ def draw_screen():
 
 def cursed_main(stdscr):
 
-    def ping_test(server_answered):
+    def ping_test():
         half_wait_time = 5
-        if server_answered:
+        if len(new_data_from_server) > 0:
             ping_test.sent = False
         elif ping_test.wait_start + half_wait_time < time.time():
             if not ping_test.sent:
@@ -185,13 +185,13 @@ def cursed_main(stdscr):
                 raise SystemExit("Server not answering anymore.")
     ping_test.wait_start = 0
 
-    def read_into_message_queue(string):
-        if string == "":
+    def read_into_message_queue():
+        if new_data_from_server == "":
             return
         new_open_end = False
-        if string[-1] is not "\n":
+        if new_data_from_server[-1] is not "\n":
             new_open_end = True
-        new_messages = string.splitlines()
+        new_messages = new_data_from_server.splitlines()
         if message_queue["open_end"]:
             message_queue["messages"][-1] += new_messages[0]
             del new_messages[0]
@@ -218,8 +218,8 @@ def cursed_main(stdscr):
             commands[chr(char)]()
             cursed_main.redraw = True
         new_data_from_server = io["file_in"].read()
-        ping_test(len(new_data_from_server) > 0)
-        read_into_message_queue(new_data_from_server)
+        ping_test()
+        read_into_message_queue()
 
 
 def foo():