On Friday 14 January 2011 15:55:12 Patrick Hartling wrote:
> A while ago, I submitted a patch to expose
> QGraphicsWidget::getContentsMargins() to Python. During more extensive
> testing, I found that that patch was incorrect and needed to address the
> fact that QGraphicsLayoutItem::getContentsMargins() is virtual.
>
> I have a patch that almost fixes up this case, but I have run into one
> problem. The code that initializes pyargs doesn't seem to know that the
> four qreal* arguments are removed, thus meaning that the Python method
> invocation will have an empty argument tuple. The following is generated
> for QGraphicsLayoutItem::getContentsMargins() and all its overrides:
>
> Shiboken::AutoDecRef pyargs(Py_BuildValue("()",
>
> ));
>
> That stray comma is, of course, a problem. This happens when using Shiboken
> 1.0.0 Beta 3. Is there a way to control that through the XML given to
> Shiboken, or is this a bug in Shiboken?You need to use a conversion rule to handle those argument removals on virtual functions, the better way to learn how to do it is looking at other functions that also make use of conversion-rule tag, see how it was used and how it reflected in the generated source code. P.S.: And file a new bug report :-) > Once this detail is resolved, I will create a bug that includes the patch > attached to this message. > > -Patrick > > > > -- > Patrick L. Hartling > Senior Software Engineer, Priority 5 > http://www.priority5.com/ > > The information transmitted in this communication is intended only for > the person or entity to which it is addressed and contains proprietary > material. Any review, retransmission, dissemination or other use of, or > taking of any action in reliance upon, this information by persons or > entities other than the intended recipient is prohibited. If you > received this in error, please destroy any copies, contact the sender > and delete the material from any computer. -- Hugo Parente Lima INdT - Instituto Nokia de Tecnologia
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ PySide mailing list [email protected] http://lists.openbossa.org/listinfo/pyside
