Bug 887 – Bad return values in QGraphicsLayoutItem.sizeHint may lead to strange program behaviour

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 887 - Bad return values in QGraphicsLayoutItem.sizeHint may lead to strange program behaviour

Summary:

Bad return values in QGraphicsLayoutItem.sizeHint may lead to strange program...

Status:	CLOSED WORKSFORME

Product:	PySide
Classification:	Unclassified
Component:	PySide
Version:	1.0.3
Platform:	PC MS Windows XP/Vista/7

Importance:	P5 normal
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-06-15 21:42 EEST by Christian
Modified:	2011-06-22 20:25 EEST (History)
CC List:	8 users (show)

See Also:

Attachments
Minimal Example to reproduce the error described above. (1.61 KB, text/plain) 2011-06-15 21:42 EEST, Christian	Details
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 Christian 2011-06-15 21:42:29 EEST

Created attachment 345 [details]
Minimal Example to reproduce the error described above.

Invalid return value in TestRect.sizeHint leads to event handling Problems.

Run the script as it is and you will observe the following issues:

- Frame cannot be closed kindly only by killing the process

- Resizing the Program leads to strange error messages:
     QPainter::begin: Paint device returned engine == 0, type: 0

  Since the problem is related to sizeHint this is a secondary error to the 
  initial problem.

Comment 1 Hugo Parente Lima 2011-06-15 21:55:56 EEST

For me it works as expected, a warning is show telling about the issue:


[hugo@thinkpadugo ~  4.7] $ python /tmp/test.py 
sys:1: RuntimeWarning: Invalid return value in function
QGraphicsLayoutItem.sizeHint, expected PySide.QtCore.QSizeF, got
PySide.QtCore.QRectF
-------------------------------
WrapperMap: 0x1765b90 (size: 0)
-------------------------------
[hugo@thinkpadugo ~  4.7] $

Comment 2 Hugo Parente Lima 2011-06-15 21:58:15 EEST

Just a note: I'm using the version from git, i.e. pos 1.0.3, what will be
1.0.4.

Comment 3 renato filho 2011-06-22 20:25:21 EEST

release 1.0.4