Bug 397 – Signal signature confusion when using typedef

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 397 - Signal signature confusion when using typedef

Summary:

Signal signature confusion when using typedef

Status:	CLOSED FIXED

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

Importance:	P3 critical
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-10-01 19:33 EEST by renato filho
Modified:	2010-10-13 13:49 EEST (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 renato filho 2010-10-01 19:33:51 EEST

In QtMobility::QMessageManager  class:

has a signal with this signature:
messageUpdated(const QMessageId & id, const,
MessageManager::NotificationFilterIdSet & matchingFilterIds )

the type "MessageManager::NotificationFilterIdSet" is a typedef to a
"QSet<QtMobility::QMessageManager::NotificationFilterId>" during the shiboken
generation this type is resolved and then shiboke use type found after to
declare the signal and use this new type in typeresolve:

Shiboken::TypeResolver::createValueTypeResolver<QSet<QtMobility::QMessageManager::NotificationFilterId>
>("QSet<QtMobility::QMessageManager::NotificationFilterId>");

signal_item = PySide::signalNew("messageUpdated", "const
QtMobility::QMessageId&,const
QSet<QtMobility::QMessageManager::NotificationFilterId>&", NULL);


This cause 2 problems, the first is: When connect this signal in the new SIGNAL
api the signal is not called because the signature does not match with
QtSignature.

The other problem is: when use the old way to connect the signal, the type used
in the signature does not have a register on typeresolver what cause a segfault
during the signal emission.

Comment 1 renato filho 2010-10-01 19:34:27 EEST

link to class documentation:

http://doc.qt.nokia.com/qtmobility-1.0/qmessagemanager.html

Comment 2 renato filho 2010-10-07 14:11:43 EEST

fixed on shiboken commit:

commit ed8481cd7accabe1b9a88aa27101bfe13bb87a62
Author: renatofilho <renato.filho@openbossa.org>
Date:   Mon Oct 4 19:16:57 2010 -0300

Comment 3 renato filho 2010-10-13 13:49:10 EEST

released on PySide 0.4.2