GitLab

Change-Id: I0390e51a7d1fc4563eed791c9f8c8ed403398663
Reviewed-by: Leandro Melo <leandro.melo@nokia.com>

Change-Id: Ia68a121f61aab3667d1a4c99ba348fbfe2e98d4d
Reviewed-by: Bill King <bill.king@nokia.com>

Task-number: QTCREATORBUG-6132
Change-Id: I7c21e648cfc2e85803a3bf4df0a615a1e276f96a
Reviewed-by: Leandro Melo <leandro.melo@nokia.com>

Task-number: QTCREATORBUG-5978
Change-Id: Iab0bba3c16b4cd435f0dc6339db49f209f1359d3
Reviewed-by: Leandro Melo <leandro.melo@nokia.com>

* Generally changes the BasicProposalItemListModel's sort to take the
  BasicProposalItem::order member into account.
* Currently only the QML completion and quick fixes set the order.
* This means the 'Apply signature changes' quick fix is now further up
  than the 'add definition' quick fix.

Change-Id: I7b5bc82aa37fca232fddd630ab3273437e1bcc09
Reviewed-by: Leandro Melo <leandro.melo@nokia.com>

Change-Id: If18afb5d4665924e7d9250dccbc60a65e6daa75e
Reviewed-by: Eike Ziller <eike.ziller@nokia.com>

Reuse Function::argumentCount instead of reimplementing it.

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

Change-Id: Iaaa3907f709ea9ed567f59725399efd9024f3bc0
Reviewed-on: http://codereview.qt-project.org/5319

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

Change-Id: I890cccc25c5ba774691edee860845e22a7b7ec78
Reviewed-on: http://codereview.qt-project.org/4876

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

Change-Id: Ied3268bb624df4201ca42beb76d20b9a638d5918
Reviewed-on: http://codereview.qt-project.org/4746

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

Change-Id: I93abf773717b3f82fc6fa15e1916f48bc3edec95
Reviewed-on: http://codereview.qt-project.org/4675

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

Except for some SimpleSpecifiers and AttributeSpecifiers. Fixes a bug
where NamedTypeSpecifiers would not be replaced when the return type
was adjusted.

Change-Id: Ie093615ebf1b26d181b888fcbcc341e5bdf85fd1
Reviewed-on: http://codereview.qt-project.org/4749

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

You could accidentally delete specifiers like 'static' or 'virtual' with
the 'apply function signature changes' quick fix before.

Change-Id: Ia97a33f663a01063fad4d432551715ebf27a18d5
Reviewed-on: http://codereview.qt-project.org/4538

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

QTextCursor::selectedText() has null chars where the tokenizer expects
newlines.

Change-Id: I15ae87ef8525c89812a61b80abda91d36bf56576
Reviewed-on: http://codereview.qt-project.org/4450

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

This will allow using 'Follow Symbol' to jump back after the change.

Change-Id: I99fd3ebc63a2af58b12eee9d7055a4c2f249e97d
Reviewed-on: http://codereview.qt.nokia.com/4186

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

This way we don't need to override Return and Escape.

Change-Id: I1548118c06ee7338ba3dd9d84b82b9314ab16782
Reviewed-on: http://codereview.qt.nokia.com/4183

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

Change-Id: Ibc38898585f6e5be625ccfc0e1304ccb44a7f167
Reviewed-on: http://codereview.qt.nokia.com/3955

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

To allow FakeVim to pick up on the state and pass through Enter/Return.

Change-Id: Ia203a1b3fa8877d90cbaf4b46183bb01a6de1362
Reviewed-on: http://codereview.qt.nokia.com/3856

Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: hjk <qthjk@ovi.com>

Change-Id: Ib5e3a3a381568347a7a465f956f7daad15f10ab0
Reviewed-on: http://codereview.qt.nokia.com/3596

Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>

Change-Id: I7a95adcaafff5ff845ae0fdef5bea8c3846a6ffe
Reviewed-on: http://codereview.qt.nokia.com/3599

Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>

It's more robust if text before the target is changed in a minor way.

Change-Id: I48e27c5d194dd2dcff4b064bf59538b4660015d7
Reviewed-on: http://codereview.qt.nokia.com/3097

Reviewed-by: Roberto Raggi <roberto.raggi@nokia.com>

Previously RefactoringFiles were usually passed around by value.
However, since a RefactoringFile may sometimes own a QTextDocument
(when it was read from a file), that's not great and caused the
file to be reread after every copy.

With this change RefactoringFile becomes noncopyable and is always
owned by a shared pointer.

This change also allowed having const RefactoringFiles which is
useful because they can be safely used from other threads. See
CppRefactoringChanges::fileNoEditor.

Change-Id: I9045921d6d0f6349f9558ff2a3d8317ea172193b
Reviewed-on: http://codereview.qt.nokia.com/3084

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

Change-Id: Icc8861a5127272750c5aa0b6340967ddf0a5a085
Reviewed-on: http://codereview.qt.nokia.com/3016

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

When editing a function declaration or definition the code model
may realize the same changes have to be applied somewhere else. A
refactoring marker will pop up that can be clicked to perform the
changes. Alternatively, press enter to apply.

Change-Id: I2299a2ecfb6a8f87d4853fc7cfa99486f890a1d3
Reviewed-on: http://codereview.qt.nokia.com/2909

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