upstream/ipython Files · IPython/core/tests/test_page.py

Fix memory leak in Qt event loop integration (#14240) (#14251)...

Fix memory leak in Qt event loop integration (#14240) (#14251) The QEventLoop object, `event_loop`, created in `IPython/terminal/pt_intputhooks/qt.py` L58 is not deleted when exiting the scope as passing `app` to the constructor parents the object to `app`. This creates a memory leak as QEventLoop objects are being accumulated. The issue was originally reported by @pag who also suggested the fix. `pytest IPython/terminal/tests/` runs through without errors. I have tested the changes and see no more accumulation of `QEventLoop` objects. This fixes

Matthias BUSSONNIER - - Load All Authors

File last commit:

r5390:c82649ea


                r28517:c0a699d1

Download file

             test_page.py
        
                    20 lines
            
             | 772 B
            
                | text/x-python
            
             |
                PythonLexer
            
             / IPython / core / tests / test_page.py
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      #-----------------------------------------------------------------------------

      #  Copyright (C) 2010-2011 The IPython Development Team.

      #

      #  Distributed under the terms of the BSD License.

      #

      #  The full license is in the file COPYING.txt, distributed with this software.

      #-----------------------------------------------------------------------------

      import io

      # N.B. For the test suite, page.page is overridden (see IPython.testing.globalipapp)

      from IPython.core import page

      def test_detect_screen_size():

          """Simple smoketest for page._detect_screen_size."""

          try:

              page._detect_screen_size(True, 25)

          except (TypeError, io.UnsupportedOperation):

              # This can happen in the test suite, because stdout may not have a

              # fileno.

              pass

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				#-----------------------------------------------------------------------------
				# Copyright (C) 2010-2011 The IPython Development Team.
				#
				# Distributed under the terms of the BSD License.
				#
				# The full license is in the file COPYING.txt, distributed with this software.
				#-----------------------------------------------------------------------------
				import io

				# N.B. For the test suite, page.page is overridden (see IPython.testing.globalipapp)
				from IPython.core import page

				def test_detect_screen_size():
				"""Simple smoketest for page._detect_screen_size."""
				try:
				page._detect_screen_size(True, 25)
				except (TypeError, io.UnsupportedOperation):
				# This can happen in the test suite, because stdout may not have a
				# fileno.
				pass