Bug 554 – Inner classes don't work and give us a segfault

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 554 - Inner classes don't work and give us a segfault

Summary:

Inner classes don't work and give us a segfault

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	Shiboken
Version:	HEAD
Platform:	All MS Windows XP/Vista/7

Importance:	P1 critical
Assigned To:	Hugo Parente Lima

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-12-17 19:48 EET by Hugo Parente Lima
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 Hugo Parente Lima 2010-12-17 19:48:40 EET

Inner classes doesn't work and give us a segfault.

e.g.:

class Foo:
    def bar(self):
        class Bug (QObject):
            pass

This a regression from beta1 and only affects Windows machines.

Comment 1 Matti Airas 2010-12-17 19:51:21 EET

Thanks for the bug. Since this is a regression in our release, known at the
release time, I'm prioritizing this P1 - this overrides other activities.

Comment 2 Hugo Parente Lima 2010-12-21 17:56:27 EET

News from the front:

- The crash also happens on beta~1.
- The crash is on shiboken, not pyside.

I'll check if there is something wrong with my env. because I remenber Matt
testing the bug with PySide beta1 without crashes.

Comment 3 Matti Airas 2010-12-22 07:29:36 EET

(In reply to comment #2)

> - The crash also happens on beta~1.

I guess this is actually good news: we don't have a regression between beta1
and beta2. I'll make a note on the mailing list that Roman can make the call
whether to wait or to proceed with the packaging.

> I'll check if there is something wrong with my env. because I remenber Matt
> testing the bug with PySide beta1 without crashes.

Yes, I tested on a Windows 7 virtual machine with Python 2.7 downloaded from
python.org and Roman's beta1 package.

Comment 4 Hugo Parente Lima 2010-12-23 16:18:09 EET

Fixed in commit:

shiboken/fc997ef702dfe2d40b8420a3c98ca16d9d276596

Comment 5 renato filho 2011-01-06 16:19:32 EET

released on beta3