This is basically a continuation of the commits which
refactor code out of the base text editor. For instance,
36fa1de4 and
3a684586.

Also removed the doXXXX() forwarding methods.

The condition resolving now looks through typedefs and calls.

Task-number: QTCREATORBUG-2051
Reviewed-by: Christian Kamm

It was being reset based on the highlighter definitions.

Consider some refactoring with the related options as well to avoid
overriding user's settings.

Done-with: Thorbjorn Lindeijer

Reviewed-by: Thorbjorn Lindeijer

Reviewed-by: ckamm

... instead of running a sed script over a lot of files on each
version increment.

Clean icore.h.

- Unexport Core::Internal::MainWindow and remove its includes.
- Move RssFetcher from Core::Internal to Core.
- Unexport CopyTaskHandler.
- Move TextEditor's completion support and Refactor Overlay
  helper classes from TextEditor::Internal to TextEditor as they
  are exported.
- Move internal BaseTextBlockSelection into private header.
- Unexport TextEditorOverlay as they are not used.

qtLibraryTarget is only defined if used with TEMPLATE=lib, which makes
it fail in pri files that are used for app pro files. The tests still
don't run because of run time linking issues.

Remove internal markers from exported headers,
introduce private class for BaseTextDocument.

Reviewed-by: Leandro T. C. Melo <leandro.melo@nokia.com>

Task-number: QTCREATORBUG-2240
Reviewed-by: Erik Verbruggen

The debugger plugin opens an editor in this case, leading to fakevim and
cppeditor crashing.
This commit doesn't fix the problem's source, but adds safeguards that
are sensible anyhow: FakeVim should not register with editors during
shutdown, and CppEditor should not unconditionally assume an existing
editor manager in the destructor.

Reviewed-by: Friedemann Kleint

Reviewed-by: Christian Kamm

As they do not look nice in the layouts and several
options pages are involved in the case of the form
class wizard.

Reviewed-by: con
Task-number: QTCREATORBUG-2778

It is unnecessary to try hard to resolve a class forward declaration
when all you want to know is whether it is a valid link or not.

Task-number: QTCREATORBUG-2688
Reviewed-by: Thorbjorn Lindeijer

Reviewed-by: Roberto Raggi

Reviewed-by: hjk
Reviewed-by: Erik Verbruggen

Currently used by the QML outline and C++ type hierarchy.

Reviewed-by: kkoehne
Task-number: QTCREATORBUG-2583

The C++ hover handler should just iterate over the list of candidates, since
it was created by the element evaluator considering all possible name qualifications
and in the right order.

Reviewed-by: kh1

This is basically for consistency with the docs and with the html
extraction mechanism.

Reviewed-by: Roberto Raggi

This is actually a general improvement that splits the qualified names
so more help ids candidates are considered when searching in the docs.

Reviewed-by: kh1

Should have been done earlier. 2.1.80 was burned in the pre-2.1.0->2.0.1
days, and we'd like to have odd subminor for 'working' and even subminor
numbers for pure cherry-picking release branches anyway.

Reviewed-by: Christian Kamm

Reviewed-by: Roberto Raggi

Reviewed-by: Leandro Melo

Reviewed-by: Roberto Raggi

Make DebuggerRunner handle QProcess::startFailed
correctly on Windows (synchronous signal emission
within startDebugger()).
Remove debug output/warnings.

Reviewed-by: hjk

(cherry picked from commit 519f1d11947ff3109850e559fae868c4a55deb43)