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 495 - Broken rich compare operators if they use an object-type as parameter
: Broken rich compare operators if they use an object-type as parameter
Status: CLOSED FIXED
Product: PySide
Classification: Unclassified
Component: Shiboken
: HEAD
: All All
: P2 normal
Assigned To: Hugo Parente Lima
:
:
: 497 569
  Show dependency treegraph
 
Reported: 2010-11-24 10:57 EET by Luciano Wolf
Modified: 2011-01-06 16:19 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 Luciano Wolf 2010-11-24 10:57:58 EET
I was generating bindings for QtMobility-Location and at some point it uses
rich compare operators. They get references as parameters but shiboken tries to
create a new copy to pass to them. The parameter is an object-type, leading to
a compilation error.

The generated code:

// operator<(const QtMobility::QGeoMapObject & other) const
QtMobility::QGeoMapObject cppOther = Shiboken::Converter<
::QtMobility::QGeoMapObject >::toCpp(other);
result = (cppSelf < cppOther);
Comment 1 Hugo Parente Lima 2010-11-24 15:45:08 EET
Changed priority to P1, It's a blocker for QtMobility bindings.
Comment 2 Matti Airas 2010-11-24 15:49:11 EET
Reverting. P1 and P2 are only to be set by the EM.
Comment 3 Matti Airas 2010-11-24 15:51:21 EET
I'm not comfortable to see further delays to the PySide release, if at all
possible. If this bug is not blocking essential functionality, I'd rather just
reject the offending functions/classes and file a bugs against them so that
this issue can be solved after the next PySide release.
Comment 4 Hugo Parente Lima 2010-11-24 16:29:16 EET
Ok, back to the bug queue...
Comment 5 Matti Airas 2010-12-21 14:23:05 EET
OK, since this appears to cause problem to external binding developers as well,
I'm increasing priority to P2.
Comment 6 Hugo Parente Lima 2010-12-29 10:17:13 EET
Fixed in commit:

shiboken/670b96db4dd58dc4e265a5e49439f705bb6193c1
Comment 7 renato filho 2011-01-06 16:19:28 EET
released on beta3