summaryrefslogtreecommitdiff
path: root/src/plugins/cppeditor/builtineditordocumentprocessor.cpp
Commit message (Collapse)AuthorAgeFilesLines
* CppEditor: Do not call Snapshot::filesDependingOn()Christian Kandeler2023-05-081-5/+17
| | | | | | | | | | | ... in document processor callback. The dependency table seems to get rebuilt more often than expected, making this operation expensive. Amends 6d805195f6d35df932d666f86af026c051c785f8. Change-Id: I1eace180b2428756b648fa1c58eb0fb66cce3c0c Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* Utils: Rename asynctask.{cpp,h} -> async.{cpp,h}Jarek Kobus2023-05-031-1/+1
| | | | | | | Follows AsyncTask -> Async rename. Change-Id: I37f18368ab826c9960a24087b52f6691bb33f225 Reviewed-by: hjk <hjk@qt.io>
* CppEditor: Force document parser run on change in header dependencyChristian Kandeler2023-05-031-0/+7
| | | | | | | | Fixes: QTCREATORBUG-24133 Change-Id: Ica89d4b0922dea5f2bea513ab8e4373bc52f7250 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* CppEditor: Remove WorkingCopy::contains()Christian Kandeler2023-05-031-6/+7
| | | | | | | Most uses involved a double look-up. Change-Id: Ifeb62ea2361222ed0faad749f44a59735c8d6930 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* CppEditor: Use QtConcurrent invocation for async runJarek Kobus2023-03-091-6/+4
| | | | | | Change-Id: Ibbac7f7788fe966c0dd846d68b7d17c43acadb0e Reviewed-by: Christian Kandeler <christian.kandeler@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* CppEditor: Add soft assert against impossible token locationChristian Kandeler2023-02-011-0/+4
| | | | | Change-Id: I1411c0f884c2f38b7e94b32714b8812210ea6d87 Reviewed-by: hjk <hjk@qt.io>
* Remove GPL-3.0+ from license identifiersKai Köhne2023-01-061-1/+1
| | | | | | | | | | | | | | | Since we also license under GPL-3.0 WITH Qt-GPL-exception-1.0, this applies only to a hypothetical newer version of GPL, that doesn't exist yet. If such a version emerges, we can still decide to relicense... While at it, replace (deprecated) GPL-3.0 with more explicit GPL-3.0-only Change was done by running find . -type f -exec perl -pi -e "s/LicenseRef-Qt-Commercial OR GPL-3.0\+ OR GPL-3.0 WITH Qt-GPL-exception-1.0/LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0/g" {} \; Change-Id: I5097e6ce8d10233993ee30d7e25120e2659eb10b Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* CppEditor: Remove Snapshot::remove(QString) overloadhjk2022-11-291-2/+1
| | | | | | | | ... and fix fallout. Change-Id: Ibbf865c3e4158fc98bb9b5372ce0633807b85576 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* CPlusPlus: Proliferate FilePath usehjk2022-11-221-10/+9
| | | | | | | | | | | | | | | | | | | The starts with CppDocument::filePath(), plus a bit of the fallout This is one patch of potentially many. It is hard to draw the line where to stop this kind of chunk, this here converts a few additional functions for which including it in the patch looked like less churn than without. Converting is mostly fromString/toString, with a few exceptions for "already seem" like caches, that use cheaper "path()" to avoid likely performance regressions (on Windows FilePath comparison is currently case-insenstive, and more expensive). There should be no difference for local operation with this patch. Change-Id: I7b35f98a0a6f0bfed4ea0f8f987faf586f7a8f2b Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Use SPDX license identifiersLucie Gérard2022-08-261-24/+2
| | | | | | | | | Replace the current license disclaimer in files by a SPDX-License-Identifier. Task-number: QTBUG-67283 Change-Id: I708fd1f9f2b73d60f57cc3568646929117825813 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* CPlusPlus: Microoptimizationshjk2022-08-101-4/+2
| | | | | | | | Inline some simple accessors, return references instead of copies in some getters, Change-Id: I136574823c79ad0c63ed354b78e1ad83908e7ae5 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* CppEditor: Remove foreach / Q_FOREACH usageArtem Sokolovskii2022-05-061-4/+6
| | | | | | Task-number: QTCREATORBUG-27464 Change-Id: I9915cf2c07bdcc570efc20601752d60f2ec82f34 Reviewed-by: David Schulz <david.schulz@qt.io>
* Remove leftover code for header diagnostic info barEike Ziller2022-05-041-1/+0
| | | | | | | | | | | | | ClangCodeModel used to show an info bar about errors in headers in the editor, but that was removed. Remove the remaining infrastructure that was used to show them. Amends b52fac7148fcd333ce410b99017b7aec52e2fee6 Change-Id: Id89f2bcb7778245f468923d29c577bf1ca68ee75 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* ClangCodeModel: Remove libclang-based references supportChristian Kandeler2022-05-021-9/+0
| | | | | | | Change-Id: I53d10c37eac1ec5a46b904a2e2eb959311f852fa Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: David Schulz <david.schulz@qt.io>
* ClangCodeModel: Remove some dead codeChristian Kandeler2022-05-021-9/+0
| | | | | | | Left over from recent libclang removals. Change-Id: I24eff39615d9e2701bc42dccc5f096a89b7fa783 Reviewed-by: David Schulz <david.schulz@qt.io>
* ClangCodeModel: Remove libclang-based diagnostics and highlightingChristian Kandeler2022-05-021-20/+20
| | | | | | Change-Id: Ib7c423884b76c27a6350ddea611919d3352fb80e Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* Merge CppTools into CppEditorChristian Kandeler2021-09-011-0/+354
There was no proper separation of responsibilities between these plugins. In particular, CppTools had lots of editor-related functionality, so it's not clear why it was separated out in the first place. In fact, for a lot of code, it seemed quite arbitrary where it was put (just one example: switchHeaderSource() was in CppTools, wheras switchDeclarationDefinition() was in CppEditor). Merging the plugins will enable us to get rid of various convoluted pseudo-abstractions that were only introduced to keep up the artificial separation. Change-Id: Iafc3bce625b4794f6d4aa03df6cddc7f2d26716a Reviewed-by: Christian Stenger <christian.stenger@qt.io>