code cleanup: IPShellQt and IPShellQt4

IPython/Shell.py

@@ -4,7 +4,7 b''
 All the matplotlib support code was co-developed with John Hunter,
 matplotlib's author.
 
-$Id: Shell.py 2755 2007-09-09 20:10:30Z darren.dale $"""
+$Id: Shell.py 2756 2007-09-10 18:25:30Z darren.dale $"""
 
 #*****************************************************************************
 #       Copyright (C) 2001-2006 Fernando Perez <[e-mail]>
@@ -885,25 +885,37 b' class IPShellQt(IPThread):'
         
         import qt
 
-        class newQApplication:
+        self.qApp = qt.qApp
+
+        class DummyQApp:
+            def __init__(self, *args):
+                self.qApp = qt.qApp
+
+            def exec_loop(*args, **kwargs):
+                pass
+
+            def __getattr__(self, name):
+                return getattr(self.qApp, name)
+
+        class DummyQApplication:
             def __init__( self ):
                 self.QApplication = qt.QApplication
                 
             def __call__( *args, **kwargs ):
+                # mask until module NoneType errors can be fixed
+#                return DummyQApp()
                 return qt.qApp
 
-            def exec_loop( *args, **kwargs ):
-                pass
-
             def __getattr__( self, name ):
                 return getattr( self.QApplication, name )
           
-        qt.QApplication = newQApplication()
+        qt.QApplication = DummyQApplication()
 
         # Allows us to use both Tk and QT.
         self.tk = get_tk()
 
-        self.IP = make_IPython(argv,user_ns=user_ns,
+        self.IP = make_IPython(argv,
+                               user_ns=user_ns,
                                user_global_ns=user_global_ns,
                                debug=debug,
                                shell_class=shell_class,
@@ -923,15 +935,20 b' class IPShellQt(IPThread):'
         self._banner = banner
 
         if qt.QApplication.startingUp():
-          a = qt.QApplication.QApplication(sys.argv)
+            self.qApp = qt.QApplication.QApplication(sys.argv)
+            # mask until module NoneType errors can be fixed
+#            qt.qApp = qt.QApplication()
+
         self.timer = qt.QTimer()
-        qt.QObject.connect( self.timer, qt.SIGNAL( 'timeout()' ), self.on_timer )
+        qt.QObject.connect(self.timer,
+                           qt.SIGNAL( 'timeout()' ),
+                           self.on_timer)
 
         self.start()
         self.timer.start( self.TIMEOUT, True )
         while True:
             if self.IP._kill: break
-            qt.qApp.exec_loop()
+            self.qApp.exec_loop()
         self.join()
 
     def on_timer(self):
@@ -955,25 +972,37 b' class IPShellQt4(IPThread):'
 
         from PyQt4 import QtCore, QtGui
 
-        class newQApplication:
-            def __init__( self ):
+        self.qApp = QtGui.qApp
+
+        class DummyQApp:
+            def __init__(self, *args):
+                self.qApp = QtGui.qApp
+
+            def exec_(*args, **kwargs):
+                pass
+
+            def __getattr__(self, name):
+                return getattr(self.qApp, name)
+
+        class DummyQApplication:
+            def __init__(self, *args):
                 self.QApplication = QtGui.QApplication
 
             def __call__( *args, **kwargs ):
+                # mask until module NoneType errors can be fixed
+#                return DummyQApp()
                 return QtGui.qApp
 
-            def exec_loop( *args, **kwargs ):
-                pass
-
             def __getattr__( self, name ):
                 return getattr( self.QApplication, name )
 
-        QtGui.QApplication = newQApplication()
+        QtGui.QApplication = DummyQApplication()
 
         # Allows us to use both Tk and QT.
         self.tk = get_tk()
 
-        self.IP = make_IPython(argv,user_ns=user_ns,
+        self.IP = make_IPython(argv,
+                               user_ns=user_ns,
                                user_global_ns=user_global_ns,
                                debug=debug,
                                shell_class=shell_class,
@@ -993,15 +1022,20 b' class IPShellQt4(IPThread):'
         self._banner = banner
 
         if QtGui.QApplication.startingUp():
-          a = QtGui.QApplication.QApplication(sys.argv)
+            self.qApp = QtGui.QApplication.QApplication(sys.argv)
+            # mask until module NoneType errors can be fixed
+#            QtGui.qApp = QtGui.QApplication()
+
         self.timer = QtCore.QTimer()
-        QtCore.QObject.connect( self.timer, QtCore.SIGNAL( 'timeout()' ), self.on_timer )
+        QtCore.QObject.connect(self.timer,
+                               QtCore.SIGNAL('timeout()'),
+                               self.on_timer)
 
         self.start()
         self.timer.start( self.TIMEOUT )
         while True:
+            self.qApp.exec_()
             if self.IP._kill: break
-            QtGui.qApp.exec_()
         self.join()
 
     def on_timer(self):

doc/ChangeLog

@@ -1,3 +1,7 b''
[phone]  Darren Dale  <[e-mail]>
+
+        * Cleanup of IPShellQt and IPShellQt4
+
 2007-09-09  Fernando Perez  <[e-mail]>
 
 	* IPython/FakeModule.py (FakeModule.__init__): further fixes for