Bug 495 – Broken rich compare operators if they use an object-type as parameter

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

Summary:

Broken rich compare operators if they use an object-type as parameter

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	Shiboken
Version:	HEAD
Platform:	All All

Importance:	P2 normal
Assigned To:	Hugo Parente Lima

URL:

Depends on:
Blocks:	497 569
	Show dependency tree / graph

Reported:	2010-11-24 10:57 EET by Luciano Wolf
Modified:	2011-01-06 16:19 EET (History)
CC List:	8 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

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