Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The test msexchange_wsdl is unreliable under Qt5 #14

Closed
frankosterfeld opened this issue Jan 16, 2013 · 1 comment
Closed

The test msexchange_wsdl is unreliable under Qt5 #14

frankosterfeld opened this issue Jan 16, 2013 · 1 comment
Labels
Milestone

Comments

@frankosterfeld
Copy link
Member

The msexchange test sometimes fails in msechange_wsdl.cpp:95 when comparing received against expected XML. The XML differ in the order of the attributes. Without detailed debugging, I suspect this is to QHash usage (in QDom?) for attributes. As iterating over a QHash now leads to undeterministic order in Qt5, one cannot rely on the order of attributes anymore. See also https://bugreports.qt-project.org/browse/QTBUG-25071

@dfaure-kdab
Copy link
Member

On Wednesday 16 January 2013 04:38:43 Frank Osterfeld wrote:

The msexchange test sometimes fails in msechange_wsdl.cpp:95 when comparing
received against expected XML. The XML differ in the order of the
attributes. Without detailed debugging, I suspect this is to QHash usage
(in QDom?) for attributes. As iterating over a QHash now leads to
undeterministic order in Qt5, one cannot rely on the order of attributes
anymore. See also https://bugreports.qt-project.org/browse/QTBUG-25071

Fixed, using the same fix as in qtbase.git
92243bc346aa3b99bb0a9ad356ef5cdb190bfbca : setting qt_qhash_seed.

David Faure | david.faure@kdab.com | Managing Director KDAB France
KDAB (France) S.A.S., a KDAB Group company
Tel. France +33 (0)4 90 84 08 53, Sweden (HQ) +46-563-540090
KDAB - Qt Experts - Platform-independent software solutions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants