On Friday, April 20, 2012 02:39:39 PM Nathan Smith wrote: > First off, congratulations on getting the first release out the door after > such a major change! This version so far seems very stable. > > Release 1.1.1 has bug 1041 in its list of fixed bugs, but unfortunately the > fix for that seems to have been lost in the travels between the old and new > systems. The fix was to change the return policy on the > QAbstractItemView.selectionModel() method to mirror that of > QAbstractItemView.model()'s return policy. Below is a patch for this bug > against the latest version in the repository:
Thanks very much for point this issue! The fix was landed. > > diff --git a/PySide/QtGui/typesystem_gui_common.xml > b/PySide/QtGui/typesystem_gui_common.xml > index 552636a..ab97ee1 100644 > --- a/PySide/QtGui/typesystem_gui_common.xml > +++ b/PySide/QtGui/typesystem_gui_common.xml > @@ -1890,6 +1890,13 @@ > <define-ownership class="target" owner="default"/> > </modify-argument> > </modify-function> > + <modify-function signature="selectionModel() const"> > + <modify-argument index="return"> > + <!-- Defining ownership as "default" avoids the object to be > automatically > + set as parent of the returned pointer. --> > + <define-ownership class="target" owner="default"/> > + </modify-argument> > + </modify-function> > <modify-function signature="setIndexWidget(const QModelIndex &, > QWidget *)"> > <modify-argument index="2"> > <parent index="this" action="add"/> > > > Nathan _______________________________________________ PySide mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/pyside
