GitLab

To factor out things common to (most) tools.

Change-Id: I5a4605f230d35280eda71e5f195d49d7bb929ae0
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

Change-Id: I71205c45ee01eeaf1d2c991ec625f6a66be1e851
Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>

This is not printed anymore.

Change-Id: Ib55a06f642247ff33e092bee371276b187c6c8c8
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>

Change-Id: Ic0b6a36033830f7f67fb3ba5eb8554cdbc295ba0
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@theqtcompany.com>
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

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

Change-Id: Ia9b3ba12c8fdd6864ffc2e44fda584bf06ef549f
Reviewed-by: hjk <hjk@theqtcompany.com>
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>

Change-Id: Id56b94436bc2df984e4c61595524638167c09842
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

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

Change-Id: I18a6dd021bb29ca5eaa7727213c80458d903ee6c
Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>

Change-Id: Ida2e4f5397ee3e1f0a5d3ab0b48e419e2423aafa
Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>

Change-Id: I68fe61206e40e953554b2066ff7f8e45222720bf
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

Not all jobs of one command can run in the same directory, so
prepare for having individual working directories per job.

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

Change-Id: Ibed6302ce82fce984e42f45eb77a9bb9c1a81f7c
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

Expose the last visited directory as well as the project directory
stored by the DocumentManager to the MacroExpander.

Change-Id: I7cdbe5e5471b5c96954c74f21187786de7bb9aa3
Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>

QObject already has a virtual destructor.

Change-Id: Ie407226775d1690623ecc76adf29e1b3723abb0c
Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>

For non-editor documents it currently is not used, but for editors it
makes more sense to have that on the document instead of the editor.
Most actual implementations of "open" were done in the documents already
anyhow, because it is needed for reloading.

Change-Id: I29d4df2078995cbe80172b51a9bebeecb3afad3c
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>

Change-Id: I85ae3a273160dad6ead9e722fcf4fb6d1835eb0c
Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>

This makes us independent of libclang crashes for completion.
Re-parsing for highlighting still happens in the Qt Creator process.

Run in verbose mode:
    qtc.clangcodemodel.ipc=true

Run tests:
    -test "ClangCodeModel"

Task-number: QTCREATORBUG-14108
Task-number: QTCREATORBUG-12819
Change-Id: Id3e95bd2afdb6508bbd1d35fddc69534a909b905
Reviewed-by: Marco Bubke <marco.bubke@theqtcompany.com>

The saved bytesAvailable value has hidden the block size bug.

Change-Id: I751d6e11194a91c839be3dcf678e5fe4cc3f75b3
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

The default constructor has no noexcept specifier for gcc 4.7

Change-Id: If204c19e21bc368d755c8d6508fd490f71be6f6c
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

clang_installation.pri
* Remove pointless CLANG_COMPLETION, CLANG_HIGHLIGHTING defines
* Remove auto-detection of llvm-config-* variants in PATH. This does
  not make any sense if the user has to provide LLVM_INSTALL_DIR.
* If llvm-config is not found in LLVM_INSTALL_DIR, determine the clang
  version by looking into the lib dir (there should exist e.g.
  "clang/3.6.1")
* Check paths for existence
* Simplify logic
* Do not call error() in helper functions

clangcodemodel.pro
* Print detected version
* Sort HEADERS/SOURCES

Change-Id: I020d2e78aea37ef01a337e6da658d7f5e56b3ac5
Reviewed-by: Marco Bubke <marco.bubke@theqtcompany.com>

Change-Id: Ib00dfd415ca380039648d3c8333f7ff1c49ad9ec
Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>

It's unused, wrong, and unwanted.

Change-Id: I1ee5474338b4d7ea9455fa2c3b9671d6a83042c9
Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>

It is useful for other tree models as well.

Change-Id: I48262c266229a91d61579ec1dc068fc18bc33ee0
Reviewed-by: hjk <hjk@theqtcompany.com>

The usual Qt API for "takeXyz()" methods is to return the item that was
"taken".

Change-Id: Ie144051801487a301b3f13e2857735b65f58150b
Reviewed-by: hjk <hjk@theqtcompany.com>

This splits the bool setToolTipExpression() operation into
a bool canHandleToolTip(), and the actual processing of
the request, which is mostly identical to the handling of
a watcher.

Handling a watcher is now mostly the same as a full Locals
update, except for the 'partial' flag. Pushing the handling
of that down to the bridges gives identical code paths
in the gdb and lldbengine. Move that to the
DebuggerEngine base class.

Change-Id: I3861b43e8630c7e7bd57fcd549b2a2387e3d4869
Reviewed-by: hjk <hjk@theqtcompany.com>
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>

Change-Id: I198af839da35f4057fbfda6226f1314127e59e48
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>

Change-Id: Ib4fda739eb01a3098d03e0c2f12fd282d2759c62
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>

Change-Id: I945bee28f04c1bba225a8c6a6f785b6cc5c9ac95
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>

Was forgotten in eaf718bd

.

Change-Id: Ia11999d708ca2d9ffd8d8955945361ce888d4330
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>

The block size was one to big and so the command could only read after a
new command arrived.

Change-Id: I64ae47667d27d66e132031b10d4f130c53d3b0e9
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

Is fixed by the right block size.

Change-Id: If3b961625925cd82740fad0f469784b4d224bac1
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

This fixes std::vector, although it doesn't really resolve numeric
template arguments. It just picks the first specialization.

Use-case:
class Foo {};
template<class T1 = Foo> class Temp;
template<> class Temp<Foo> { int var; };
void func()
{
    Temp<> t;
    t.var; // var not highlighted
}

Task-number: QTCREATORBUG-8922
Change-Id: I593515beb3a6d901b6088db8bc1b8e16c39083d3
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

Use the template scope instead.

Change-Id: I8144427e14644697c709643da7c0ae0b0841e34d
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

* If a template type is specialized as a pointer, accept only pointers (of any
type)
* Same for references and arrays
* Only if the specialized type is not part of the template, match it
  against the input.

Fixes resolving of partial specialization with pointers.

Use-cases:
// 1
struct b {};
struct a : b {};
template<class X, class Y> struct s { float f; };
template<class X> struct s<X, b*> { int i; };
template<class X> struct s<X, a*> { char j; };

void f()
{
    s<int, a*> var;
    var.j; // j not highlighted
}

// 2
template <typename T> struct Temp { T variable; };
template <typename T> struct Temp<T &> { T reference; };
void func()
{
    Temp<int&> templ;
    templ.reference; // reference not highlighted
}

// 3
class false_type {};
class true_type {};
template<class T1, class T2> class and_type { false_type f; };
template<> class and_type<true_type, true_type> { true_type t; };
void func2()
{
    and_type<true_type, false_type> a;
    a.f; // f not highlighted
}

Task-number: QTCREATORBUG-14036
Change-Id: Idee5e3f41d15c0772318d3837cbcd442cb80293a
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

Change-Id: I1d504da25b44dc60be7dca2b07f5eadbc437826c
Reviewed-by: Tim Sander <tim@krieglstein.org>
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>

Use-case:
template<class T>
using Foo = Bar<T>; // T not highlighted

Task-number: QTCREATORBUG-9944
Change-Id: I04cb62ea6a21f158f7fb4fb7ac79ccd6eb1bbfbb
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

Task-number: QTCREATORBUG-12127
Change-Id: Ifb6ff22282f0f181e49a260e57307ec937f5dfb2
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

As a nice side effect superfluous new lines - introduced by quick fixes
that are using InsertionPointLocator::methodDefinition - vanished.

Task-number: QTCREATORBUG-13872
Change-Id: Ib3df2b2acbc22449f16f4444092a57ae93d53d35
Reviewed-by: Jochen Becher <jochen_becher@gmx.de>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@theqtcompany.com>

The original use case of running GDB itself remotely
is not present anymore.

Change-Id: I283a27216d6a8849fea7d9ca8df6b94571fb99fd
Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>