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

None != None; pyside2 internally uses wrong "None".

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Not Evaluated
    • 5.14.1
    • 5.11.0
    • PySide
    • None
    • 8c5b08a74650e5dea7303d6f232c4df0f706cb35 (pyside/pyside-setup/dev)

    Description

      PySide2 seems to create its own object for "None" internally instead of using the none created by the cpython runtime.

      In [1]: import PySide2.QtWidgets                                                                                     
In [2]: PySide2.__version__                                                                                          
Out[2]: '5.6.0~a1'
In [3]: PySide2.QtWidgets.qApp                                                                                       
Out[3]: None
In [4]: id(PySide2.QtWidgets.qApp)                                                                                   
Out[4]: 140515700103744                                                                                                         
In [5]: id(None)                                                                                                     
Out[5]: 94436718787216

      I tested this also with pyside2 5.13.1

      Attachments

        1. pyside1158_diag.diff
          3 kB
        2. pyside1158.py
          0.4 kB

        Issue Links

          relates to

          Task - A task that needs to be done. PYSIDE-1178 Replacing the qApp "macro" by a no-surprizes qApp callable?

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed
          For Gerrit Dashboard: PYSIDE-1158
          # Subject Branch Project Status CR V
          285655,5 Turn the qApp macro into a regular callable 5.14 pyside/pyside-setup Status: ABANDONED 0 0
          286326,9 Let qApp be noApp instead of pretending to be None 5.14 pyside/pyside-setup Status: MERGED +2 0

          Activity

            People

              ctismer Christian Tismer
              thisch thomas hisch
              Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes