From 582377f5199ea66856ff4d673430dc8467db00f9 Mon Sep 17 00:00:00 2001
From: Eratosthenes <ray.barker@hughes.net>
Date: Wed, 8 Sep 2010 20:35:59 -0400
Subject: [PATCH] Catch up WinTables.py to recent changes in TableWindow.py.

---
 pyfpdb/TableWindow.py |  2 +-
 pyfpdb/WinTables.py   | 21 ++++++---------------
 pyfpdb/XTables.py     |  4 +++-
 3 files changed, 10 insertions(+), 17 deletions(-)

diff --git a/pyfpdb/TableWindow.py b/pyfpdb/TableWindow.py
index a77a9aaa..637f2e01 100644
--- a/pyfpdb/TableWindow.py
+++ b/pyfpdb/TableWindow.py
@@ -57,7 +57,7 @@ limit_game_names = { #fpdb name      Stars Name   FTP Name
 
 #    A window title might have our table name + one of these words/
 #    phrases. If it has this word in the title, it is not a table.
-bad_words = ('History for table:', 'HUD:', 'Chat:')
+bad_words = ('History for table:', 'HUD:', 'Chat:', 'FPDBHUD')
 
 #    Here are the custom signals we define for allowing the 'client watcher'
 #    thread to communicate with the gui thread. Any time a poker client is
diff --git a/pyfpdb/WinTables.py b/pyfpdb/WinTables.py
index 42bb8b41..638cb5b5 100644
--- a/pyfpdb/WinTables.py
+++ b/pyfpdb/WinTables.py
@@ -1,9 +1,7 @@
 #!/usr/bin/env python
-"""WinTables.py
-
-Routines for detecting and handling poker client windows for MS Windows.
+"""Routines for detecting and handling poker client windows for MS Windows.
 """
-#    Copyright 2008 - 2009, Ray E. Barker
+#    Copyright 2008 - 2010, Ray E. Barker
 
 #    This program is free software; you can redistribute it and/or modify
 #    it under the terms of the GNU General Public License as published by
@@ -49,24 +47,19 @@ tb_height = 29
 
 class Table(Table_Window):
 
-    def find_table_parameters(self, search_string):
+    def find_table_parameters(self):
         """Finds poker client window with the given table name."""
         titles = {}
         win32gui.EnumWindows(win_enum_handler, titles)
         for hwnd in titles:
             if titles[hwnd] == "": continue
-            # print "searching ", search_string, " in ", titles[hwnd]
-            if re.search(search_string, titles[hwnd]):
-                if 'History for table:' in titles[hwnd]: continue # Everleaf Network HH viewer window
-                if 'HUD:' in titles[hwnd]: continue # FPDB HUD window
-                if 'Chat:' in titles[hwnd]: continue # Some sites (FTP? PS? Others?) have seperable or seperately constructed chat windows
-                if 'FPDBHUD' in titles[hwnd]: continue # can't attach to ourselves!
-                self.window = hwnd
+            if re.search(self.search_string, titles[hwnd]):
+                if self.check_bad_words(titles[hwnd]): continue
                 break
 
         try:
             if self.window == None:
-                log.error( "Window %s not found. Skipping." % search_string )
+                log.error( "Window %s not found. Skipping." % self.search_string )
                 return None
         except AttributeError:
             log.error( "self.window doesn't exist? why?" )
@@ -79,8 +72,6 @@ class Table(Table_Window):
         self.width  = width - x
         self.height = height - y
         log.debug("x = %s y = %s width = %s height = %s" % (self.x, self.y, self.width, self.height))
-        #self.height = int(height) - b_width - tb_height
-        #self.width  = int(width) - 2*b_width
 
         self.exe    = self.get_nt_exe(hwnd)
         self.title  = titles[hwnd]
diff --git a/pyfpdb/XTables.py b/pyfpdb/XTables.py
index 10626950..722547e5 100644
--- a/pyfpdb/XTables.py
+++ b/pyfpdb/XTables.py
@@ -45,8 +45,10 @@ class Table(Table_Window):
         for listing in os.popen('xwininfo -root -tree').readlines():
             if re.search(self.search_string, listing):
                 mo = re.match('\s+([\dxabcdef]+) (.+):\s\(\"([a-zA-Z0-9\-.]+)\".+  (\d+)x(\d+)\+\d+\+\d+  \+(\d+)\+(\d+)', listing)
+                title  = re.sub('\"', '', mo.group(2))
+                if self.check_bad_words(title): continue
                 self.number = int( mo.group(1), 0)
-                self.title  = re.sub('\"', '', mo.group(2))
+                self.title = title
                 self.exe    = ""     # not used?
                 self.hud    = None   # specified later
                 break