Bug 501 – Shiboken generator doesn't use full qualified name (with :: prefix) on all places.

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 501 - Shiboken generator doesn't use full qualified name (with :: prefix) on all places.

Summary:

Shiboken generator doesn't use full qualified name (with :: prefix) on all pl...

Status:	CLOSED FIXED

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

Importance:	P4 normal
Assigned To:	Hugo Parente Lima

URL:

Depends on:
Blocks:	464
	Show dependency tree / graph

Reported:	2010-11-25 17:04 EET by Hugo Parente Lima
Modified:	2011-06-22 20:14 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 Hugo Parente Lima 2010-11-25 17:04:55 EET

Shiboken generator doesn't use full qualified name on all places, this can
cause compiler errors in some use cases like a enum with the same name of their
namespace.

To fix it the generator should always use the full qualified name of a type
including the "::" prefix to tell the compiler to search the symbol on global
namespace.

To reproduce the error, revert the commit
shiboken/907842acf4baa9e520431157bf1b0293476d65c3 and try to compile it on
Windows using MSVC.

Comment 1 Matti Airas 2010-11-26 10:11:15 EET

I can't tell the significance of this by reading the explanation alone. What
problems does the current behaviour cause?

Comment 2 Matti Airas 2010-12-17 16:44:53 EET

Prioritizing to P4.

Comment 3 Hugo Parente Lima 2010-12-20 17:42:55 EET

This was fixed on 23/Nov in commit:

shiboken/fd81c050afca0ff3190e852062837be05017f00f

Marking it as fixed then closed.

Comment 4 Hugo Parente Lima 2010-12-20 18:00:10 EET

Oops, I tested on Windows and the bug still valid, only part of it were fixed
to implement another feature some weeks ago.

Comment 5 Hugo Parente Lima 2011-06-06 23:54:15 EEST

Fixed in commits:

generatorrunner/75cffde7e51646e6e5561384caacb4a1932005cf
shiboken/122f18828fc1ef52ac40983bd86997bfd22b611c

Comment 6 renato filho 2011-06-22 20:14:09 EEST

release 1.0.4