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

weird design error in shiboken

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: P2: Important
    • Resolution: Unresolved
    • Affects Version/s: 5.6, dev
    • Fix Version/s: dev
    • Component/s: Shiboken
    • Labels:
    • Environment:
      macOS

      Description

      There is a design bug in shiboken since Qt5 and PySide2 exists.
      I fixed it with an ugly workaround and added a nagging message since
      July 2015, and I'm not sure if anybody ever paid attention to this message?

      Please read this commit message of 70afb33fd3e9e87ac86f635ba88e005af5eabf86:

      fixed a problem with qRegisterMetaType that came up with Qt5.
      
      The problem is that an unqualified name gets registered ("iterator") by some hard to
      understand heuristics of shiboken. This way, the same name is used for objects which have
      different size in Qt5, and therefore things crash.
      
      The quick fix was to skip exactly the name "iterator" and output a warning.
      This needs to be fixed in a better way, but costs too much time at the moment,
      because I don't understand enough about possible side-effects, yet.
      

      You can see my ugly work-around in `cppgenerator.cpp:4817`. Just look for the text "hack(ct)".

      Shiboken finds 'iterator' for some reason and wants to produce wrong code. My hack simply
      intercepts and skips the case, but this is no solution!

      Some brave soul should please

      • comment the patch away and
      • check the wrong behavior by building PySide2.

      This is really broken.

        Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

            Assignee:
            Unassigned
            Reporter:
            ctismer Christian Tismer
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:

                Gerrit Reviews

                There are no open Gerrit changes