Bug 33080

Summary:	Move qHash(Tp::ProtocolParameter) in namespace Tp
Product:	Telepathy	Reporter:	George Kiagiadakis <gkiagia>
Component:	tp-qt	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	blocker
Priority:	medium
Version:	git master
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description George Kiagiadakis 2011-01-13 14:10:11 UTC

Hi,

I mentioned this on irc, but I think it may get lost and since it's highly important to be fixed before the next release (because the change is not ABI-compatible), here is a bug :)

Commit 936d6da2c553d5de7d1d592b09f0777aac8f1962 introduced a qHash(Tp::ProtocolParameter) function. This function needs to be inside namespace Tp, otherwise the code that is using QHash<Tp::ProtocolParameter, T> will need to include this tp-qt4 header *before* <QtCore/QHash> and this is a nightmare...

Regards,
George

Comment 1 Andre Moreira Magalhaes 2011-01-17 05:54:07 UTC

Fixed on release 0.5.3.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.