Bug 33080 - Move qHash(Tp::ProtocolParameter) in namespace Tp
Summary: Move qHash(Tp::ProtocolParameter) in namespace Tp
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-qt (show other bugs)
Version: git master
Hardware: Other All
: medium blocker
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-13 14:10 UTC by George Kiagiadakis
Modified: 2011-01-17 05:54 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments

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.