Merge branch 'master' of git://git.assembla.com/fpdb-eric

pyfpdb/HUD_main.py

@@ -159,11 +159,15 @@ class HUD_main(object):
 #    function idle_func() to be run by the gui thread, at its leisure.
         def idle_func():
             gtk.gdk.threads_enter()
-#            try:
             self.hud_dict[table_name].update(new_hand_id, config)
-            [aw.update_gui(new_hand_id) for aw in self.hud_dict[table_name].aux_windows]
-#            finally:
-            gtk.gdk.threads_leave()
+            # The HUD could get destroyed in the above call ^^, which leaves us with a KeyError here vv
+            # if we ever get an error we need to expect ^^ then we need to handle it vv - Eric
+            try:
+                [aw.update_gui(new_hand_id) for aw in self.hud_dict[table_name].aux_windows]
+            except KeyError:
+                pass
+            finally:
+                gtk.gdk.threads_leave()
             return False
 
         gobject.idle_add(idle_func)
@@ -198,7 +202,7 @@ class HUD_main(object):
             try:
                 (table_name, max, poker_game, type, site_id, site_name, tour_number, tab_number) = \
                                 self.db_connection.get_table_info(new_hand_id)
-            except Exception, err:
+            except Exception, err: # TODO: we need to make this a much less generic Exception lulz
                 print "db error: skipping %s" % new_hand_id
                 sys.stderr.write("Database error: could not find hand %s.\n" % new_hand_id)
                 continue

pyfpdb/Hud.py

@@ -530,7 +530,8 @@ class Hud:
     def update(self, hand, config):
         self.hand = hand   # this is the last hand, so it is available later
         if os.name == 'nt':
-            self.update_table_position()
+            if self.update_table_position() == False: # we got killed by finding our table was gone
+                return
 
         for s in self.stat_dict:
             try:

pyfpdb/WinTables.py

@@ -57,8 +57,12 @@ class Table(Table_Window):
                 self.window = hwnd
                 break
 
-        if self.window == None:
-            print "Window %s not found. Skipping." % search_string
+        try:
+            if self.window == None:
+                print "Window %s not found. Skipping." % search_string
+                return None
+        except AttributeError:
+            print "self.window doesn't exist? why?"
             return None
 
         (x, y, width, height) = win32gui.GetWindowRect(hwnd)
@@ -149,4 +153,3 @@ class Table(Table_Window):
 
 def win_enum_handler(hwnd, titles):
     titles[hwnd] = win32gui.GetWindowText(hwnd)
-

pyfpdb/fpdb_import.py

@@ -548,7 +548,7 @@ class Importer:
                         if self.callHud:
                             #print "call to HUD here. handsId:",handsId
                             #pipe the Hands.id out to the HUD
-                            print "fpdb_import: sending hand to hud", handsId, "pipe =", self.caller.pipe_to_hud
+                            # print "fpdb_import: sending hand to hud", handsId, "pipe =", self.caller.pipe_to_hud
                             self.caller.pipe_to_hud.stdin.write("%s" % (handsId) + os.linesep)
                     except Exceptions.DuplicateError:
                         duplicates += 1