This ammends e19bbdf8

.

Change-Id: Idebb29cdbb6fb30ac446701ebc415a7944f48421
Reviewed-by: Jan Arve Sæther <jan-arve.saether@theqtcompany.com>

Fix release mode compiler warnings:

In member function 'virtual void QPatternistSDK::DebugExpressionFactory::processTemplateRule(const Ptr&, const Ptr&, const QXmlName&, QPatternist::ExpressionFactory::TemplateCompilationStage)':
warning: 'title' may be used uninitialized in this function [-Wmaybe-uninitialized]
In member function 'virtual void QPatternistSDK::DebugExpressionFactory::processNamedTemplate(const QXmlName&, const Ptr&, QPatternist::ExpressionFactory::TemplateCompilationStage)':
warning: 'title' may be used uninitialized in this function [-Wmaybe-uninitialized]

by rewriting the code to directly add to a QString.

Change-Id: I23781b624e6b3fbc385d8b041a1923d49c291f51
Reviewed-by: Mitch Curtis <mitch.curtis@theqtcompany.com>

Change-Id: I41beab20e4de378725e3e2efd929ddd83460d9d8

e.g.
  export XMLPATTERNSXQTS_TESTRANGE=42,49

will execute the tests from number 42 to number 49. The tests are
numbered sequentially.

This will allow us to not having to run through all the tests when
testing for a specific testcase.

Change-Id: I4c21158a4eada1b48eb809a887216a7160d73220
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@theqtcompany.com>

Change-Id: I45ad981fa4d6d7302cf2e06429ea1134957034f3
Reviewed-by: Erik Verbruggen <erik.verbruggen@theqtcompany.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@theqtcompany.com>

The examplesinstallpath variable in .qdocconf files defines the path
under QT_INSTALL_EXAMPLES where examples are found.

To match the way examples are packaged in Qt 5.6, prefix the
install path with the repository name.

Task-number: QTBUG-48736
Change-Id: Ib00fdd4029f62f9e75eddf5cf58eec8cd1c405bb
Reviewed-by: Venugopal Shivashankar <venugopal.shivashankar@digia.com>

Change-Id: I9a379d34bf80344de9b6e9465e6f35fca7fd1a12

d17eba9d

 added literalsequence.xq, but
forgot to add the baseline file.

This probably passed in 5.5 because the test was run twice (because of
flakyness):
 1. The first pass it would fail (but generate the baseline).
 2. The second pass it would succeed because the baseline was already
 there

Change-Id: If40ee6c57e97cb4cffc959e0f6bb768e610a76d4
Reviewed-by: Liang Qi <liang.qi@theqtcompany.com>

Appararently, some new tests fail and other new tests started pass.
I haven't digged into the reasons why, but the fact that the pass
percentage is at 96%, this is better than not testing it at all.

Change-Id: Id5e2710ebd3c17b65a2792245a073c676d116208
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@theqtcompany.com>

QXmlName has overloads for its functions, marked \internal, and
QDoc fails to link to the public ones because it sees the
internal ones first.

As a workaround, exclude parentheses from the link - QDoc then
uses a different code path which resolves the functions
correctly.

Change-Id: I0470cfd67c5d089ca586fd2debcb7b71c606a716
Reviewed-by: Nico Vertriest <nico.vertriest@digia.com>

A \target whose purpose is to link to the top of a
page (and not to a section within a page) works better
as a \keyword, because \target generates a
new html anchor which, in this case, is not tied to
any title element on the page.
A \keyword links to the page itself, as expected.

Task-number: QTBUG-48482
Change-Id: I0d55924f66c26f3fb23a07be1f1d57f2105c879d
Reviewed-by: Topi Reiniö <topi.reinio@digia.com>

Change-Id: Ied9eefcaff7c7a991cde483dc08c6905ba9ca421
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@theqtcompany.com>

Fix warning:
expr\qorderby.cpp:79:7: warning: 'template<class T> class qLess' is deprecated [-Wdeprecated-declarations]
 class qLess<Item::List>
       ^
In file included from qtbase\include\QtCore/qalgorithms.h:1:0,
                 from qtbase\include\QtCore/QtAlgorithms:1,
                 from expr\qorderby.cpp:34:
qtbase/src/corelib/tools/qalgorithms.h:150:40: note: declared here
 class QT_DEPRECATED_X("Use std::less") qLess
                                        ^
expr\qorderby.cpp: In member function 'virtual QAbstractXmlForwardIterator<QPatternist::Item>::Ptr QPatternist::OrderBy::evaluateSequence(const Ptr&) const':
expr\qorderby.cpp:172:11: warning: 'template<class T> class qLess' is deprecated [-Wdeprecated-declarations]
     const qLess<Item::List> sorter(m_orderSpecs, context);
           ^
In file included from qtbase\include\QtCore/qalgorithms.h:1:0,
                 from qtbase\include\QtCore/QtAlgorithms:1,
                 from expr\qorderby.cpp:34:

Change-Id: Ieb626eb0de96aea670433bb46a52ed780240e8d7
Reviewed-by: Björn Breitmeyer <bjoern.breitmeyer@kdab.com>

Introduce a message when starting fails or timeouts occur and flip
the order to check for timeouts first and then check the exit code.

Change-Id: I293aa55888186f66c12cd86bcb8010802f521032
Reviewed-by: Mitch Curtis <mitch.curtis@theqtcompany.com>

Conflicts:
	.qmake.conf

Change-Id: If7c5b9674646fcd1cf8f2a6c43e4e1e0f126950f

LGPLv3 refers to it but does not include it in its body.

Change-Id: Ib056b47dde3341ef9a52ffff13eed18cf3504738
Reviewed-by: Lars Knoll <lars.knoll@theqtcompany.com>

Change-Id: Iea043221b965bbbc05f9c98b8f58aa7870acadfe
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@theqtcompany.com>

Change-Id: I9dda1a76e730dfe1790fe6771a6f9ccd63c5d13b
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@theqtcompany.com>

It's deprecated in C++14. Found by Clang:

error: 'register' storage class specifier is deprecated [-Werror,-Wdeprecated-register]

Change-Id: Ib306f8f647014b399b87ffff13f23d75e8255950
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>

Change-Id: I54541c588dfa76180ffdf99ac0ed2f6cdc8cfdc4
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@theqtcompany.com>

At least on OS X, the test would use the baseline of xmlpatternsxqts
since that's where tst_suitetest.h is. QFINDTESTDATA would then resolve
Baseline.xml relative to that.

Change-Id: I860d7e3143365b33901f721caed813d4d594f28d
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

The current code iterates over the (empty) list of operands of a Literal
Sequence, ignoring any items in it, so Literal Sequences are always
ignored in the ExpressionSequence.

Task-number: QTBUG-35897
Change-Id: I60341ac21c8e3b77bf6d8dfeebdbafe010844406
Reviewed-by: Erik Verbruggen <erik.verbruggen@theqtcompany.com>

Conflicts:
	.qmake.conf

Change-Id: I56232f2e3996efa9804cfabae61db765629e060a

Change-Id: I2d7bf0419d63ec8dde51162a4c3a7d191a8d2089

The elements on the parser stack are of a complex type, which bison
generated parsers don't know how to reallocate when they need to grow
the stack. This patch implements yyoverflow, which is called whenever
the parser runs out of stack space.

The size of the elements is quite large (152 bytes on x86_64), so the
initial stack (which is allocated on the C stack) is set to 1 element.
Any subsequent reallocations are done by using QVector for reallocation,
copying the elements, and handling the deallocation. Because of the size
of the elements, the stack (vectors) are grown linearly.

The upper limit of the stack size if left at 10000 elements, which
should really be enough for an lalr(1) grammar.

Task-number: QTBUG-4470
Change-Id: Ic1ef08655b388c38ef452d03a425cbd31b91825b
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

Identifier names like "IN" and "AS" have a high probability of being
defined by certain platforms as macros, esp. with Visual Studio.

Change-Id: I1fbc0ae33f94387b83377181b6012683a418459a
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

Change-Id: I40f55a4eabaf2d0d6bcda0a4b6111b27130f24f3

It's much faster

Change-Id: I959559a442ab6acf424d7798b9bcd01ab29e65e3
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>

No need to initialize it before main.

Change-Id: I2ebc9a2b04a58275570e967c17ace9028f11d5ba
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>

Change-Id: I86391bf0ede4fb32037b4e98889650e36ef5d279
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>

We have a QSet::contains() since 4.6.

Change-Id: Idbafeec6e58aadff4a07e7ed50ac54dc8738623a
Reviewed-by: Konstantin Ritt <ritt.ks@gmail.com>
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>

Change-Id: I6e7fa03e23796db90db44a944544002e851cffd9

These tests crashed, and thus prevented us from getting the results from
the ~19000 other test cases.

Change-Id: I8fd2abcc68b545177f4baa912e9e0d5dbe931201
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

It seems to require a larger refactoring in order to solve this, so
we skip this for now in order for us to be able to run the remaining
tests in CI.

Change-Id: I2ae4860a92cdb7fb88ca89c4dd1f047bacbaaf9e
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

Change-Id: I5e588fe388d93483b5d533cdd539d749e4fb4ce8

Avoid conflicts with the newly introduced qdebug_p.h
of QtCore.

Change-Id: I04b83e159900d9d36fa96ae5112eb4cdf75c0551
Reviewed-by: Liang Qi <liang.qi@theqtcompany.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@theqtcompany.com>

Change-Id: Ic5fa3d4a4bc9e1a9c96c78869c2f8b5c4cd19a07

I currently have many failures (I assume through bad testsuite set up),
they go by silently on the second run, instead now there will always
be output:
PASS   : tst_XmlPatternsSchemaTS::runTestSuite()
SUMMARY:
        Total:                39193
        Failures:             39193
        Passes:               0
        Not tested:           0
        Pass percentage(%):   0
        Unexpected failures:  0
        Unexpected passes:    0
Result was identical to the baseline, baseline was not updated.
PASS   : tst_XmlPatternsSchemaTS::checkTestSuiteResult()

Change-Id: I4e11832ea711b8cb9feff088a09434122968d080
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

We have configure -headersclean now

Change-Id: Iaa084ece95cefd5264a34726d85d13ea7a65016d
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>

It was used to disable code/variables referenced in Q_ASSERT_X.
Since the macro is now expanded in release mode as well, this
causes compilation errors.

Change-Id: I5b90515c042f9d3e76be84cd5385eae081a037c8
Reviewed-by: Louai Al-Khanji <louai.al-khanji@theqtcompany.com>