Uploaded image for project: 'Qt for Python'
  1. Qt for Python
  2. PYSIDE-2646

[REG: 6.2->6.5] Connections to lambda or nested function prevent destruction of objects they refer to

    XMLWordPrintable

Details

    • 47d6abbd7 (dev), 9529690b4 (dev), 16bedacb1 (6.7), 1d429a7b5 (6.7), 24f39fa50 (6.6), eaff347f5 (6.6), eec4d27a4 (tqtc/lts-6.5), c777151d2 (tqtc/lts-6.5), c174421fa (dev), 279658b85 (dev), 24cca3617 (6.7), 66a58efd8 (6.7), 3cafacbbf (6.6), ef3c3697b (dev)

    Description

      Seems similar to PYSIDE-2371, but the fix there didn't fix this case.

      In attached example, QSlider and QSpinBox are not destroyed until when the whole process exits.

      Attachments

        1. pyside2646.py
          6 kB
          Friedemann Kleint
        2. pyside2646_pyqt.py
          5 kB
          Friedemann Kleint
        3. pyside2646_log.txt
          2 kB
          Friedemann Kleint
        4. bug.py
          2 kB
          Joni Poikelin

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. PYSIDE-2141 Crash when connecting/disconnecting inherited slot

          • Not Evaluated - Not yet evaluated/prioritized
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. PYSIDE-2299 REG->6.5: Memory/Reference Leak with connecting signals to lambda functions referencing "self"

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. PYSIDE-2371 REG->6.5: Memory leak connecting signal to local function

          • P4: Low - Would be nice to fix, but it's not very important.
          • Closed
          For Gerrit Dashboard: PYSIDE-2646
          # Subject Branch Project Status CR V
          550926,4 libpyside: Clean up lambdas connected to signals earlier in shutdown dev pyside/pyside-setup Status: MERGED +2 0
          550981,2 libpyside: Fix spelling error in function name dev pyside/pyside-setup Status: MERGED +2 0
          551018,2 libpyside: Fix spelling error in function name 6.7 pyside/pyside-setup Status: MERGED +2 0
          551020,2 libpyside: Clean up lambdas connected to signals earlier in shutdown 6.7 pyside/pyside-setup Status: MERGED +2 0
          551078,2 libpyside: Fix spelling error in function name 6.6 pyside/pyside-setup Status: MERGED +2 0
          551079,2 libpyside: Clean up lambdas connected to signals earlier in shutdown 6.6 pyside/pyside-setup Status: MERGED +2 0
          551144,2 libpyside: Fix spelling error in function name tqtc/lts-6.5 pyside/tqtc-pyside-setup Status: MERGED +2 0
          551145,2 libpyside: Clean up lambdas connected to signals earlier in shutdown tqtc/lts-6.5 pyside/tqtc-pyside-setup Status: MERGED +2 0
          551912,5 libpyside: Refactor the signal-lambda-test dev pyside/pyside-setup Status: MERGED +2 0
          551913,7 libpyside/Signal manager: Ensure cleanup of main thread targets dev pyside/pyside-setup Status: MERGED +2 0
          552275,2 libpyside: Refactor the signal-lambda-test 6.7 pyside/pyside-setup Status: MERGED +2 0
          552276,2 libpyside/Signal manager: Ensure cleanup of main thread targets 6.7 pyside/pyside-setup Status: MERGED +2 0
          552278,2 libpyside: Refactor the signal-lambda-test 6.6 pyside/pyside-setup Status: MERGED +2 0
          552345,2 libpyside: Refactor the signal-lambda-test tqtc/lts-6.5 pyside/tqtc-pyside-setup Status: MERGED +2 0
          552436,4 Fix flake8-warnings in signals tests dev pyside/pyside-setup Status: MERGED +2 0
          552691,6 Port signals tests to modern syntax dev pyside/pyside-setup Status: MERGED +2 0
          552850,2 Fix flake8-warnings in signals tests 6.7 pyside/pyside-setup Status: MERGED +2 0
          552902,3 Port the QtCore signals tests to modern syntax dev pyside/pyside-setup Status: MERGED +2 0
          553859,2 Port the QtCore signals tests to modern syntax 6.7 pyside/pyside-setup Status: MERGED +2 0
          553860,2 Port signals tests to modern syntax 6.7 pyside/pyside-setup Status: MERGED +2 0
          563468,3 libpyside/Signal manager: Ensure cleanup of main thread targets tqtc/lts-6.5 pyside/tqtc-pyside-setup Status: MERGED +2 +1

          Activity

            People

              kleint Friedemann Kleint
              poikelin Joni Poikelin
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews