msmith - in flumotion/trunk: . flumotion/manager

[flumotion-commit at lists.fluendo.com]

Author: msmith
Date: Thu Mar  8 16:49:54 2007
New Revision: 4605

Modified:
   flumotion/trunk/ChangeLog
   flumotion/trunk/flumotion/manager/manager.py
Log:
        * flumotion/manager/manager.py:
          Work around a pychecker bug so I can run pychecker again.



Modified: flumotion/trunk/ChangeLog
==============================================================================
--- flumotion/trunk/ChangeLog	(original)
+++ flumotion/trunk/ChangeLog	Thu Mar  8 16:49:54 2007
@@ -1,5 +1,10 @@
 2007-03-08  Michael Smith  <msmith at fluendo.com>
 
+	* flumotion/manager/manager.py:
+	  Work around a pychecker bug so I can run pychecker again.
+	  
+2007-03-08  Michael Smith  <msmith at fluendo.com>
+
 	* flumotion/twisted/fdserver.py:
 	  Backwards compatibility for FDClient; accept messages without our
 	  special magic cookie.

Modified: flumotion/trunk/flumotion/manager/manager.py
==============================================================================
--- flumotion/trunk/flumotion/manager/manager.py	(original)
+++ flumotion/trunk/flumotion/manager/manager.py	Thu Mar  8 16:49:54 2007
@@ -694,9 +694,10 @@
         # in list given by worker
         d = workerAvatar.getComponents()
         def workerAvatarComponentListReceived(workerComponents):
-            lostComponents = [c for c in self.getComponentStates()
+            # list() is called to work around a pychecker bug. FIXME.
+            lostComponents = list([c for c in self.getComponentStates()
                               if c.get('workerRequested') == workerId and \
-                                 c.get('mood') == moods.lost.value]
+                                 c.get('mood') == moods.lost.value])
             for comp in workerComponents:
                 # comp is an avatarId string
                 # components is a list of {ManagerComponentState}