Change-Id: I169dcc02f02a647b0a2ae6478227a852be922de2
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

* Const correctness
* Better variable names
* Sort order of includes and forward declarations
* Comment fixes
* Cosmetic/Whitespace changes

Change-Id: Ieb137266ac7eddedb00c37f8b2cc8677d67e4bdd
Reviewed-by: David Schulz <david.schulz@digia.com>

E.g.:
enum E { x = 1 + 2 + 1 };

constantValue for enumeratorDeclaration currently is equal: '1+2+1'
(spaces between are removed).
Fixed version produces "1 + 2 + 1".

Change-Id: I38f7703bd5c856c28aadeecd7f7923aa4a7da422
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>

Change-Id: Ic3f5db354cce4fd992008fa89aa4748254ea290e
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I182d5b557f5770816059897e746709d054ecac51
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Use directly the NodeInstanceView

Change-Id: I8e4da4fa9e130fe62db178df6a0b60e9bda01a86
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I8e88bd6f3a97e09f2325a28be975352ccd48a909
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I5c3f05beec22f056daa3419fac40ef837a655a8d
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I8753eae6c4f79967822003709e7cd35dbea4eeba
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I087c4d6652d4f57e3c7682176e79253c0e378809
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

It has nothing to do with a ModelNode

Change-Id: I5fd6afd42be8db9e6ac17de9df8777c8f286a9c2
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I770a15c653c6b25d10a919f6c6dbefdb79c17d89
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

Change-Id: Iba9d32aacbfe11d9542f629fca0571998b4fadfb
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

Change-Id: Iffa82f4ab06162ab57e77301e77fb37ed7230bd5
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

Change-Id: If95aead7d922511e4f1b3abdee38f4c403da37a5
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: Ieee0fc5fc95832d6976f0c1397e0261164f317c3
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: I421686cdc9faf2a9dadad3162cfbfc6793ab3caf
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

Change-Id: If8b98e008e59de1eae87108a3256005fd85981b9
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

It is much smarter to use the view functions instead of generating the whole
list and pick the first item or ask if the list is empty.

Change-Id: Ifcd40784ddab75c505b5f7f2dfc7b11cbd11c69d
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

To improve the readability of the source methods should be used.

Change-Id: I839e448369a03dc42b3649baed033daa20c4d49a
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

QML Profiler does not make use of the run configuration aspect.
Return the run configuration aspect for only Valgrind.

Regression introduced in Ic8279755f0188ab53253a62322fcccf1c17b6aaf

Change-Id: I568e309e72f7e7f0107bd720acc9dbbda78acfc1
Reviewed-by: hjk <hjk121@nokiamail.com>

Change-Id: I67b27bbe541054feec6642b6906360f99cdb5297
Reviewed-by: Christiaan Janssen <christiaan.janssen@digia.com>

Change-Id: I597f214f3c7c912acc137f2fb453836ee712720b
Reviewed-by: Christiaan Janssen <christiaan.janssen@digia.com>

The UI freeze happens when an user cancels 'Add BlackBerry Device' wizard
while SSH-keys are being generated.

Removing a need for terminate() and wait() calls in
BlackBerryDeviceConfigurationWizardSshKeyPage dtor by invoking a new
thread each time a Generate button is pressed. Such a thread can delete
itself later when the calculation is finished.
The only drawback is that the calculation is still happening even when the
wizard is closed. Just the calculation results are not used anywhere.

Task-number: QTCREATORBUG-9888

Change-Id: Ibc5e98d08e129d7f76620a5dea9ed5190932243b
Reviewed-by: Mehdi Fekari <mfekari@blackberry.com>
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

In recent internal NDK versions (10.2) some new
environment variables are introduced (they are evaluated
when soucring the bbnk-script) that QNX_TARGET and QNX_HOST
are depending on.

This patch is a workaroud, in the future the parser
should be deprecated/replaced to make the plugin less depending on
the bbndk-scrit changes.

Change-Id: I8d2dc7f0a87cbada6f8807df2bafdfd14476e01d
Reviewed-by: Tobias Nätterlund <tobias.naetterlund@kdab.com>
Reviewed-by: Nicolas Arnaud-Cormos <nicolas@kdab.com>

Change-Id: I58b0d7582b2c0d13061067e89b3662a5a82f2a8b
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

For example, C++ source files will be compiled but the build
stops before linking.

Task-number: QBS-283

Change-Id: If0573ea58b9a047980aab0fd8e4828f3d0c315b8
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

In most cases, there is no value in writing out the index explicitly,
and it is downright annoying when adding steps conditionally.

Change-Id: I2a3947d3dbc11364046c0917c661497d0720725a
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

Change-Id: Id8d0a702941453326e9e17ce4cfcdbd09a5bd2b0
Reviewed-by: Nicolas Arnaud-Cormos <nicolas@kdab.com>

Remove unused code and cancel any running indexing.

Change-Id: I1afbd6e4e041224e36c715242b8be31991259e86
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

Change-Id: Ie63ef0b1e2989c58de7d9bf92fa3cbc100608512
Reviewed-by: Christiaan Janssen <christiaan.janssen@digia.com>

Change-Id: Idcc8459660526f972949a6f6908acc19bd52a4f7
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

Task-number: QTCREATORBUG-9789
Change-Id: If3626e244b9d85ad670d586438710e17bba8e927
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

Task-number: QTCREATORBUG-9789

Change-Id: Id501ade0ac2ce7093fa38368f13b2bfab31bd35f
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

We should not duplicate all wizard resources for every supported
build system -- this will not scale. Instead, have one top-level
directory for each type of wizard with build system specific data
in respective subdirectories and common data in another.
To support this approach, CustomWizard now traverses the template
directory recursively as long as no wizard specification has been found.

Change-Id: I8aad8cf36c5bf24d062f2d2c17fdb87ad12fd450
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>

Rationale:
    a) For people not interested in CMake, the wizards unnecessarily
clutter the UI. If unhandled, this problem would get worse the more
build systems we support.
    b) If such a wizard is actually used in the absence of the plugin,
nothing visible will happen after it has finished, which is guaranteed
to confuse users.

Change-Id: I070409d3185b11d2e04e33927a97843e60c657ff
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>

Change-Id: Ibd2d6f86cd82e98bf9116bc9d5dc14dec1c6d058
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

We should change that QmlModelView architecture anyway. It is not really
fitting. Maybe attach one to the model.

Change-Id: Iee041d6e1972bd2cd9040f932404feffbe5728ef
Reviewed-by: Marco Bubke <marco.bubke@digia.com>

Change-Id: I7a6e3705807651a81b1408ce4dfaea2cdda731df
Reviewed-by: Tim Jenssen <tim.jenssen@digia.com>

Change-Id: Idedc365c1cdf73f6728597b87c7a807b25581a1f
Reviewed-by: Tim Jenssen <tim.jenssen@digia.com>