1. 20 Aug, 2014 4 commits
  2. 19 Aug, 2014 1 commit
  3. 18 Aug, 2014 2 commits
  4. 07 Aug, 2014 3 commits
    • Nikolai Kosjar's avatar
      CppTools: Tests: Add "Find Errors Indexing" mode · 0f3032f8
      Nikolai Kosjar authored
      Adding QTC_FIND_ERRORS_INDEXING=1 to the run environment will Creator
      force to do some "extended indexing" in order to find bugs:
      
         1) The project files are parsed as if they would be opened in an
            editor. That is, parsing happens as precisely as possible, based on
            the ProjectPart information.
      
         2) Symbols are looked up by invoking CheckSymbols, the backend of the
            semantic highlighter.
      
         3) A "Task List File" (*.tasks) will be written with all diagnostic
            messages.
      
      This special indexing mode is meant for testing purposes. E.g. it can be
      run nightly on some bigger projects to find regressions.
      
      Note that Qt Creator will quit exactly after the first time some source
      files are indexed. E.g. that will happen if you open a file manually or
      if you open a new unconfigured project. Therefore it's required to
      configure projects as needed before invoking in this indexing mode.
      
      Change-Id: If25b83e67d24df9e28e107cb062f21cbf3b4c643
      Reviewed-by: default avatarErik Verbruggen <erik.verbruggen@digia.com>
      0f3032f8
    • Nikolai Kosjar's avatar
      CppTools: Builtin-Indexer: Fix potential memory leak · 8bf4a5b0
      Nikolai Kosjar authored
      In case the list of source files to be parsed is empty, the
      CppSourceProcessor is never deleted.
      
      This was never a real problem because of the guard in
      CppModelManager::updateSourceFiles().
      
      Change-Id: Icfd6962d11f2b2bf2ac28825f2fa0af0838a09c8
      Reviewed-by: default avatarErik Verbruggen <erik.verbruggen@digia.com>
      8bf4a5b0
    • Nikolai Kosjar's avatar
      CppTools: Builtin-Indexer: Remove superfluous member · 70ca7496
      Nikolai Kosjar authored
      Change-Id: Ia76a59f0c2739bec5e145edce3ca8711f54b5fa4
      Reviewed-by: default avatarErik Verbruggen <erik.verbruggen@digia.com>
      70ca7496
  5. 04 Aug, 2014 1 commit
  6. 01 Aug, 2014 2 commits
    • hjk's avatar
      TextEditor: Remove one stack of EditorWidget constructors · f5b0bd32
      hjk authored
      There are conceptually only two: one that operates a new document,
      and one that shares one. Being explicit makes moving data over to
      the Editor hierarchy easier. Convenience can be re-added there,
      later.
      
      Change-Id: I9b34ff26628c99ffff01201dcf99332d5e7253e9
      Reviewed-by: default avatarChristian Stenger <christian.stenger@digia.com>
      Reviewed-by: default avatarhjk <hjk121@nokiamail.com>
      f5b0bd32
    • Przemyslaw Gorszkowski's avatar
      C++: nested class with enclosing template class · ba42ceb0
      Przemyslaw Gorszkowski authored
      The parent of instantiation of nested class of template class
      should be the instantiation of enclosing template class.
      
      To prevent the infinite loop for case with local typedef of enclosing
      template we should not change a parent of typedefed instatiation of
      enclosing template. Example:
      template <typename T>
      struct Enclosing
      {
        typedef Enclosing<T> EnclosingT;// first case
        struct Nested
        {
          typedef Enclosing<T> EnclosingT;// second case
        };
      };
      
      Task-number: QTCREATORBUG-11752
      Task-number: QTCREATORBUG-11999
      Change-Id: Iadd7b5ef73ee0c4881f59c9dabfe03339f55827b
      Reviewed-by: default avatarNikolai Kosjar <nikolai.kosjar@digia.com>
      ba42ceb0
  7. 31 Jul, 2014 1 commit
  8. 30 Jul, 2014 5 commits
  9. 25 Jul, 2014 1 commit
  10. 24 Jul, 2014 3 commits
  11. 23 Jul, 2014 1 commit
    • hjk's avatar
      Merge ITextMark and BaseTextMark · cf8a6144
      hjk authored
      Turns out we don't even need two-phase initialization,
      by transparently postponing registration until we get
      a non-empty file name, either at constuction, or at
      file name change times.
      
      Change-Id: I3e87e47c820066e6707e946fc474ab9c1993e61f
      Reviewed-by: default avatarEike Ziller <eike.ziller@digia.com>
      cf8a6144
  12. 18 Jul, 2014 1 commit
  13. 17 Jul, 2014 1 commit
  14. 15 Jul, 2014 1 commit
  15. 08 Jul, 2014 2 commits
  16. 07 Jul, 2014 1 commit
  17. 04 Jul, 2014 7 commits
  18. 01 Jul, 2014 2 commits
    • Thiago Macieira's avatar
      Use the functions in QRefCount to check if the QString is in use · 56ddd194
      Thiago Macieira authored
      QStringLiteral() returns a string that has a refcount of -1 and it must
      be kept. There is no way to determine if it's still in use, so we'll
      just assume it is. Any QStringLiteral inserted into the structure will
      stay there forever.
      
      On the other hand, we must not accept unsharable strings, so Q_ASSERT on
      that.
      
      Change-Id: I5fbdc1046f0f00319f27fdfb7aa3ff87371ea668
      Reviewed-by: default avatarNikolai Kosjar <nikolai.kosjar@digia.com>
      56ddd194
    • Christian Kandeler's avatar
      Always pass Core::Id by value. · 93304df0
      Christian Kandeler authored
      Currently we pass in some places by value, elsewhere by const ref and
      for some weird reason also by const value in a lot of places. The latter
      is particularly annoying, as it is also used in interfaces and therefore
      forces all implementors to do the same, since leaving the "const" off is
      causing compiler warnings with MSVC.
      
      Change-Id: I65b87dc3cce0986b8a55ff6119cb752361027803
      Reviewed-by: default avatarhjk <hjk121@nokiamail.com>
      93304df0
  19. 30 Jun, 2014 1 commit