Bug 695 – QWidget.winId() returns a deprecated PyCObject

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 695 - QWidget.winId() returns a deprecated PyCObject

Summary:

QWidget.winId() returns a deprecated PyCObject

Status:	CLOSED WONTFIX

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

Importance:	P3 normal
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-02-22 06:15 EET by Daniele Esposti
Modified:	2011-03-04 15:10 EET (History)
CC List:	9 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 Daniele Esposti 2011-02-22 06:15:58 EET

QWidget.winId() method returns a PyCObject which is deprecated since Python
2.7.
I can see two possible solution for that:

- return a PyCObject until Python 2.6 and PyCapsule if Python2.7+
- return always a PyObject

Comment 1 Hugo Parente Lima 2011-02-22 11:09:15 EET

Python 2.7 *is* deprecated in favor of Python 3 or at least can be considered
deprecated and there will have no Python 2.8, so the fact that PyCObject was
deprecated in Python 2.7 is a bit pointless =].

At the moment we need to support python 2.5 (yes, Maemo still using it), 2.6
and 2.7, PyCapsule wasn't backported to python 2.5 neither 2.6 so we can't use
it.

PyCObject is already an PyObject, so I didn't get the point of the second
solution.

We already tought about create our own class to store void pointers like
sip.voidptr but in the end we took the standard Python type instead of create
our own.

Comment 2 Daniele Esposti 2011-02-22 12:00:11 EET

Ok, this make sense, so I agree for the current solution. I wasn't aware about
the Maemo requirements.

Comment 3 Hugo Parente Lima 2011-03-04 15:10:41 EET

Closing bug due to 1.0.0 release.