[PySide] Release 1.1.1 bug 1041 regression

[Hugo Parente Lima]

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