GitLab

Change-Id: Iead4267d0919ce4a8d7f9f5fdec61cd9ba8dd0db
Reviewed-by: Christian Stenger <christian.stenger@digia.com>

Conflicts:
	qtcreator.pri
	qtcreator.qbs
	src/shared/qbs

Change-Id: Iba59e41db72e2afdf594f1f7003215d7d8d1e6d3

Task-number: QTCREATORBUG-12054
Change-Id: I994098197514217522fe288f7e2a83e66ffd795e
Sanity-Review: Qt Sanity Bot <qt_sanitybot@qt-project.org>
Reviewed-by: Eike Ziller <eike.ziller@digia.com>

Introduced by 2ad31475



Task-number: QTCREATORBUG-12409
Change-Id: I422d315c5b9d22052251a0869d314adff7741af0
Sanity-Review: Qt Sanity Bot <qt_sanitybot@qt-project.org>
Reviewed-by: Christian Stenger <christian.stenger@digia.com>

Change-Id: I1938ed1bf45523ec62f35bc3324f6553c0dafe22
Sanity-Review: Qt Sanity Bot <qt_sanitybot@qt-project.org>
Reviewed-by: Eike Ziller <eike.ziller@digia.com>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>

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

Change-Id: I47e8948980e3edb361c29b8976c5644dc696fd78
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>

Change-Id: I5c28c7f05b8e4766a5e04e00d9c8cd7e48b8cd95
Reviewed-by: Eike Ziller <eike.ziller@digia.com>

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

Because of the alias we get the cpp type.

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

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

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

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

Task-number: QTCREATORBUG-12374
Change-Id: I9dd8bbf9134bac65fbaa70b0614b27365377b0bf
Reviewed-by: Daniel Teske <daniel.teske@digia.com>

We have to check if the currentNode is actually valid.

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

It was just containing a tree view anyhow

Change-Id: Ied34e370be4dca0a113a861cafc0a8dac5689db7
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

Change-Id: Ib0b67b01604d3abc7d2a30dd3841fa4a46775a4e
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

And increase the priority for those too. Fixes the case:

a.pro
 b.pri
   test
 c.pri

Right click on test and add new file. In that case b.pri should be the
node to which the file is added.

Task-number: QTCREATORBUG-12272
Change-Id: I54d144861bc06545db8e71ed33513a0df971dae2
Reviewed-by: Eike Ziller <eike.ziller@digia.com>

As opt-in functionality. Only works if fetching is synchronous.

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

...by providing the necessary definitions.

Currently this does not has any effect since function-like macros are
not expanded for the semantic document. This might change, therefore we
adapt this test.

Change-Id: I7de4fdecac7b7504da7389ab216b7349ea53d7be
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

This is only a small refactoring step towards a editor-free
CppEditorSupport.

Left are m_editor references that need to be addressed.

No functional change.

Change-Id: I9fc7b0abfe88f5b9a6ce3f57a5c4a922ae953ddf
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

...instead an ITextEditor.

The ITextEditor was only used to access the file path and QTextDocument
via the BaseTextEditorWidget.

Change-Id: Idef2b8162b992fe3718e16e2a046e2ff214846f2
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

No need to pass more than needed.

Change-Id: I047cc44f17342999e0809d022df35ba9a4a8dfb8
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

The function is unused

Change-Id: If12869fd51014a06d3feeb5f147e070f5f1ce463
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

This fixes the regression where two Location pages were added
before.

Change-Id: Ic3241d707ad8dcf6fc3063bb4613dce49001e196
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Add method to Utils::Wizard that allows to retrieve the first page
of a given type.

Change-Id: Id92496d09bf89cee6f6e8ef584409125b218ec83
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: Ie7b96b0309e1291f4bb002a1f01e0473b8e15239
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Get rid of a couple of useless Internal:: namespace prefixes

Change-Id: I99f44082a2ba0ec39b51d3f89db6697834414a99
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

This one will get some more functionality.

Change-Id: Ifba410e32f98c28cbfbb5a72493906ab3d681225
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

Also rename CustomWizardFactory to CustomWizardMetaFactory. These factories
are used to generate the actual factories that are visible in File->New, so
they are kind of Meta:-)

CustomWizardClassFactory was my first attempt, but since there are project-
and classwizards that are generated by this meta-factory that is not a good
name.

Get rid of a useless typedef while at it.

Change-Id: I3dc2cce3f52999af7e59d5f2ad5509153056ea49
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I2813babc838d5dd14e4b341954d6ce0d53bfcda7
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

It makes the code more complicated in one place just to make it harder
to follow in another...

Remove CustomWizardParameters::toString() while at it: It would need some
extending to work with this change and I do not think it is worth it, considering
that it is only used when the CustomWizard is set to verbose.

Change-Id: Ice14b197ed2fe98c54bc9c1c55ece3ed2222fbbf
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Removing the addWizardPage function makes the code easier to follow.

The old implementation did always name the CustomWizardPage as "Details",
which is just wrong. Getting rid of this makes the page use its title as
the progress text, which is what we want in general anyway.

Change-Id: Iaf44f467c45952ad881154f31f62521f4f562658
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

That avoids having to do it whenever the page is created.

Change-Id: Ie58288a66af79301295d74c3c9f892dc20a832af
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

That method was used to set a title in the progress view of the wizard
that is different from the page title. That is used exactly once and
there it adds more confusion than it helps. So get rid of the whole
thing.

Consistently set the "shortTitle" property instead for all wizards
that want to have a separate short title.

Change-Id: Ia4881e9c00891058629491f9e9de4ac421c59727
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I409699225b5f42db7ecb6c2dc8d150c1a66ee23c
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

Change-Id: If2ccfe9ff40539404ed0f40c06c71a73ee661e2d
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>

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

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

icon and iconPath are two different icons.

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