Bug 304 – C++ virtual wrapper should be aware of ownership modifications

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 304 - C++ virtual wrapper should be aware of ownership modifications

Summary:

C++ virtual wrapper should be aware of ownership modifications

Status:	CLOSED FIXED

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

Importance:	P5 major
Assigned To:	Marcelo Lira

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-08-18 16:02 EEST by Lauro Moura
Modified:	2010-09-10 19:21 EEST (History)
CC List:	7 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 Lauro Moura 2010-08-18 16:02:54 EEST

Currently, ownership modifications are implemented only in the CPython method
wrapper, and not in the C++ virtual wrapper. So, when a virtual is called from
C++, none of these modifications are applied.

This behavior results in potential segfaults, specially in factory functions as
in MeeGo.Touch.MCellCreator.createCell. It returns a new MWidget that is
collected right after the virtual call, thus making the resulting pointer
invalid.

Comment 1 renato filho 2010-09-02 18:26:02 EEST

fixed on Shiboken commit:
commit 515f5edc0f3f3d8ecbd9b583b55f294d607abdf9
Author: Renato Filho <renato.filho@openbossa.org>
Date:   Thu Sep 2 17:07:07 2010 -0300


PySide commit:
commit 94a402db7b479c9f33e6501aa670c5ed0d5f10e4
Author: Renato Filho <renato.filho@openbossa.org>
Date:   Thu Sep 2 17:14:09 2010 -0300