PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 677 - generated code has setParent twice
: generated code has setParent twice
Status: CLOSED FIXED
Product: PySide
Classification: Unclassified
Component: Shiboken
: HEAD
: All All
: P2 normal
Assigned To: Marcelo Lira
:
:
:
  Show dependency treegraph
 
Reported: 2011-02-09 19:26 EET by renato filho
Modified: 2011-02-17 19:18 EET (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description renato filho 2011-02-09 19:26:10 EET
The code generated by shiboken has setParent twice in some cases.

ex: QToolbar.addAction

          case 3: // addAction(QString)
            {
                QString cppArg0 = Shiboken::Converter<QString
>::toCpp(pyargs[0]);

                if(!PyErr_Occurred()) {
                    // addAction(QString)
                    PyThreadState* _save = PyEval_SaveThread(); //
Py_BEGIN_ALLOW_THREADS
                    QAction * cppResult = cppSelf->addAction(cppArg0);
                    PyEval_RestoreThread(_save); // Py_END_ALLOW_THREADS
                    pyResult = Shiboken::Converter< ::QAction *
>::toPython(cppResult);
                    Shiboken::Object::setParent(self, pyResult);
                    Shiboken::Object::setParent(self, pyResult);
                }
                break;
            }
Comment 1 Hugo Parente Lima 2011-02-09 20:11:35 EET
Prioritizing as P2, it's not a crash nor a regression but I think those basic
things should be working without known issues before the 1.0 release.
Comment 2 Marcelo Lira 2011-02-11 16:51:14 EET
Fixed in Shiboken/7901c69c
Comment 3 Hugo Parente Lima 2011-02-17 19:18:23 EET
Released in 1.0.0~rc1