summaryrefslogtreecommitdiff
path: root/src/tools
Commit message (Collapse)AuthorAgeFilesLines
* Clang: Fix completion icon for members in member init listNikolai Kosjar2018-11-141-0/+1
| | | | | Change-Id: Ib02320dcf4de7cbb8ac6fbf29d1929edae77591a Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Add BuildDependenciesStorageMarco Bubke2018-11-0810-72/+78
| | | | | | Task-number: QTCREATORBUG-21378 Change-Id: Ibcb90239d240653b21f12a7b96a7775e5b0b4319 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Refactor FilePathIdMarco Bubke2018-11-082-4/+20
| | | | | | | | We don't need the directory id any more. It's not used widely any way. Task-number: QTCREATORBUG-21443 Change-Id: Ia95ea4c72fe9530ac56262f61f17faca04d313ba Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* ClangPchManager: Add BuildDependenciesProviderMarco Bubke2018-11-0610-9/+406
| | | | | | | | It is still unfinished but I want to add other changes first. Change-Id: Ife7ac56576f85d800633ea1e8f1c3f4f16486d5f Task-number: QTCREATORBUG-21377 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* LanguageClient: Improve Options iconAlessandro Portale2018-11-051-6/+6
| | | | | | | | Fits a bit better to the other icons. Change-Id: Ie73740a858a0683acb18ecf30d01fb816d928db3 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: David Schulz <david.schulz@qt.io>
* Clang: Fix unresolved #includes for ui_*.h headersNikolai Kosjar2018-10-3114-5/+153
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...with an extra parse. Previously, the creation of an e.g. "Qt Widgets Application" from the wizard could show code model errors in mainwindow.cpp. Depending on timing issues, the first error is either 1. 'ui_mainwindow.h' file not found (QTCREATORBUG-15187) The parse happened before the in-memory ui_mainwindow.h was generated by uic. The file system watcher can't help here as the #include was not resolved successfully. And libclang's reparse does not handle this case (it would need to remember all failed #include stats...). ==> Detect this case with the help of the include paths and trigger a full parse. 2. or: allocation of incomplete type... (QTCREATORBUG-15187) The parse happened after the generation of the in-memory ui_mainwindow.h, but before the clangbackend received the unsaved file. ==> Fix this by also writing the content of the unsaved file to our behind-the-scenes-created ui_mainwindow.h. Fixes: QTCREATORBUG-15187 Fixes: QTCREATORBUG-17002 Change-Id: I4f3a81adaa3d604746977a402c29f83fbc5b0e44 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Add unit-tests for function overloads completion orderIvan Donchevskii2018-10-261-1/+3
| | | | | | | | | | Make sure that the same function overloads with different priorities come together in the completions list. Fixes the case when we complete the method without '.' or '->'. Change-Id: Icaf7ea47f5e58b3ae5cc9764ad79c857f6f6e231 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* ClangPchManager: Add PchTaskGeneratorMarco Bubke2018-10-257-4/+242
| | | | | | Task-number: QTCREATORBUG-21358 Change-Id: Ia460ce82c3c896f4d0ac00638d03800a55dd27dc Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* ClangPchManager: Add build dependencies to include collectorMarco Bubke2018-10-259-60/+222
| | | | | | | | It's a first step but we have to refactor the interface later. Task-number: QTCREATORBUG-21379 Change-Id: Idda666bcaec950203f001c993c54a926779527e0 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Add top system includes to include collectorMarco Bubke2018-10-257-13/+47
| | | | | | | | | If we want to split the project part we have to filter out the top system includes. Task-number: QTCREATORBUG-21345 Change-Id: I258fa33ac39bd94b4699f4f39923d6ad274c1dc7 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Go through CLANG-UPGRADE-CHECKIvan Donchevskii2018-10-251-1/+1
| | | | | Change-Id: Iaaaea598d020a44bcdc60a8d06ee28a1f28386bb Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* Clang: Adapt priorities for the same method/constructor overloadsIvan Donchevskii2018-10-251-23/+24
| | | | | | | | | | | | CXXMethod and CXXConstructor may have different priorities depending ony their origin and attributes. To keep them together in the sorted list we adapt their priorities to have the same value if their names match. To continue keeping ClassCompletion before ConstructorCompletion change the order of the completion kinds for the sort purposes. Change-Id: I36efe5d5dbaa77d604a54b1dafe07d67f44db4c9 Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* Tests: Fix linking without lib toolingChristian Stenger2018-10-241-7/+7
| | | | | Change-Id: I51110089b1bed337de54dfa7167b2a5aff761837 Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* Clang: Remove pointless functionNikolai Kosjar2018-10-242-18/+1
| | | | | Change-Id: Ia4239f5b2aa1cfdd515ec0ff43fc10cb49c6f335 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Clang: Avoid running pointless UpdateDocumentJobsNikolai Kosjar2018-10-241-4/+5
| | | | | | | | | | | Opening a document (without typing in it), switching away and back to it resulted in running UpdateDocumentJobs for that document for nothing. As for documents we are going to reset, simply set opened documents initially dirty. Change-Id: I8edc2d1fb8f6f92950b4e2067f60609176bd5b26 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* Merge remote-tracking branch 'origin/4.7' into 4.8Eike Ziller2018-10-241-6/+11
|\ | | | | | | Change-Id: I6780d8103a88e31f42c674a96d0f6bcbba16e4bb
| * iOS: Work around mismatch of USB serial and iOS device idEike Ziller2018-10-231-6/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we track connection of devices in the iOS plugin via USB, using the USB serial number as a device identifier (iosdevice.h/cpp). On the other side, iostool uses the MobileDevice framework to identify iOS devices (iosdevicemanager.h/cpp). The assumption that the two identifiers are the same seems to be no longer true with the iPhone XS devices. These have a device identifier that contains a dash that is not present in the USB serial number. As a hotfix, just remove any dashes from the identifier on the iostool side because we only use it for the lookup deviceId -> AMDeviceRef anyhow. The longer term fix should be to use MobileDevice framework for the connection tracking of devices on the iOS plugin side as well, instead on relying on questionable assumptions. Change-Id: Iac3115a1c3f43a4f9e159aaa4ded1c671c55d888 Fixes: QTCREATORBUG-21291 Reviewed-by: Jason Hihn <jhihn@gmx.com> Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>
* | Fix frequent typoRobert Loehning2018-10-231-1/+1
| | | | | | | | | | | | | | can not -> cannot Change-Id: Ie872ada1bc9b4ed64bffb667c2e44dbb13b4ad11 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
* | Clang: Fix build with clang 7.0Ivan Donchevskii2018-10-175-3/+29
| | | | | | | | | | | | | | As an addition update the header with Clang-Tidy checks. Change-Id: Ib4a4d324e8cad95403a3e799e352205b05ed7475 Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Clang: Make SourceLocation lazily calculate its fieldsIvan Donchevskii2018-10-176-59/+68
| | | | | | | | | | | | | | | | Trigger path, line, column and offset evaluation only if one of them is accessed. Change-Id: Ib2f8b06ece94a3b7424db28523b5796628865202 Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Clang: Fix fix-its positions calculationIvan Donchevskii2018-10-163-9/+35
| | | | | | | | | | | | | | | | | | Code completion does not save the unsaved file contents in translation unit therefore we need to access unsaved file directly to calculate the proper UTF-16 column. Change-Id: I0f217b0d36e0ff94ebc483ed6bef94b55efd311a Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Clang: Fix SourceLocation column recalculationIvan Donchevskii2018-10-161-1/+1
| | | | | | | | | | | | | | | | Fix column conversion according to the fact that columns are 1-based. Change-Id: I75e9b966d96b9bb7648822b564e57f8838ebb3f8 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Clang: Split symbol storageMarco Bubke2018-10-1610-326/+422
| | | | | | | | | | | | | | | | | | We need UsedMacros and source related statements in the PCH manager too, so we have to split that class. Task-number: QTCREATORBUG-21289 Change-Id: Ie27d4b518b3d6d9174e93fcb243fdb55a09ddf51 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Remove hard-coded disabling of debug logsOrgad Shaneh2018-10-137-10/+4
| | | | | | | | | | | | | | | | | | | | | | Instead, set the default level of all logs to QtWarningMsg. The call to setFilterRules overrides the user preferences in qtlogging.ini. Change-Id: Id5f6cd550d14ff7f45ae04c5d3110e0bafb0f072 Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: Kai Koehne <kai.koehne@qt.io> Reviewed-by: hjk <hjk@qt.io>
* | ClangPchManager: Introduce UsedMacroAndSourcesCollectorMarco Bubke2018-10-128-109/+601
| | | | | | | | | | | | | | | | It's a subset to thesymbols collecter so we try to share some code. Change-Id: I058eee39289453e205f039b3850c1816f1d8ec1a Task-number: QTCREATORBUG-21257 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | icons/export.py: Make exporting of additional scale factors easierAlessandro Portale2018-10-051-5/+7
| | | | | | | | | | | | | | | | This changes export.py in a way that adding a scale factor becomes easy. Change-Id: Ib6649be931d36bf21df429fe5dd237b440913743 Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
* | LanguageClient: Add an options iconAlessandro Portale2018-10-051-0/+69
| | | | | | | | | | | | | | Server "can" with speech bubble saying "<>". Change-Id: I7eca0109aa5917f341f6310c8f175b1833ff9c00 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* | creatoricons.svg: Remove unused icons and adjust outdated idsAlessandro Portale2018-10-052-118/+21
| | | | | | | | | | Change-Id: I34989912a6e23de91a4ac2fa095c0b7826a4e578 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* | Debugger: Fix stopping of core file debugging using the fat buttonhjk2018-10-041-0/+21
| | | | | | | | | | | | Task-number: QTCREATORBUG-21094 Change-Id: If60f6bea93a583ab27e9e8b17c07402b2ac0854c Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* | Clang: Move the majority of completion items sorting to ClangBackendIvan Donchevskii2018-10-023-17/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | With this change ClangCodeModel only needs to sort completions by prefix. Also some other optimization have become possible and are implemented here: 1. Getting completions after '{' for constructor overloads by replacing it with '(' inside usaved file. 2. Checking for all overloads requires only previous item check because all Class completions are already sorted to go before all CXXConstructor completions. Since they are not mixed no extra search is required. Change-Id: Ie0187ad96a20857a63c1d71ddec74606b803f572 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Clang: Introduce Token class to work with CXToken pointersIvan Donchevskii2018-09-2616-244/+389
| | | | | | | | | | | | | | | | | | Make tokenization and tokens annotation simpler. Task-number: QTCREATORBUG-21143 Task-number: QTCREATORBUG-21144 Change-Id: I580091b7b63dd973228fd2e21cf2e74c7d0e7df2 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Clang: Remove project tracking on clangbackend sideNikolai Kosjar2018-09-2630-754/+91
| | | | | | | | | | | | | | | | | | | | | | | | ...as it is not needed. Just provide the compilation arguments as part of the Document. As a side effect, re-initializing the backend after a crash is cheaper and will not freeze the UI anymore (referenced bug). Task-number: QTCREATORBUG-21097 Change-Id: I866e25ef1fd5e4d318df16612a7564469e6baa11 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Clang: Rename member variables according to the code styleIvan Donchevskii2018-09-265-88/+88
| | | | | | | | | | Change-Id: I2ca6127d04321cfb5ac933475b54c669fbee729d Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Merge remote-tracking branch 'origin/4.7' into 4.8Eike Ziller2018-09-262-2/+2
|\ \ | |/ | | | | Change-Id: Iaae859601ae4f529c1fa7ada3a5c569820abb0d6
| * Fix incorrect grammar in 'does not exists'Kai Koehne2018-09-211-1/+1
| | | | | | | | | | | | Change-Id: I790d9bbdfc170808ff66e70be0f81d1c292c0bcb Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: Marco Bubke <marco.bubke@qt.io>
| * Clang: Remove version check for skipping inlude file warningsIvan Donchevskii2018-09-211-1/+1
| | | | | | | | | | | | | | | | The change is not upstreamed so the version number is not valid. Task-number: QTCREATORBUG-21139 Change-Id: I46fdb0a700780811b88083ae6ed55a241c705cb9 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | Clang: Fix include paths for clangrefactoringbackend for qbsNikolai Kosjar2018-09-251-1/+3
| | | | | | | | | | | | Change-Id: I36b5e28a91999559644bce002085eb929e6e82ef Reviewed-by: Marco Bubke <marco.bubke@qt.io> Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
* | ClangRefactoring: Use wildcard for pch manager qbs project fileMarco Bubke2018-09-251-19/+2
| | | | | | | | | | | | Change-Id: I2197bb6c1d65c8e04ac36ff21f1d604b9d35443e Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io> Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Clang: Reuse thread based pipeline for pch creationMarco Bubke2018-09-2439-1007/+939
| | | | | | | | | | | | | | | | | | | | | | The pch creation so far used signal and slots but there was no explicit pipeline. This patch is introducing the same architecture like the refactoring plugin. It is filtering out older project parts from the pipeline. Change-Id: Iaa6bd2ca1272231b97ebe1f5f7b2ce8e43bc590c Task-number: QTCREATORBUG-21111 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Merge remote-tracking branch 'origin/4.8'Orgad Shaneh2018-09-211-18/+30
|\ \ | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/debugger/debuggermainwindow.cpp Change-Id: I07496916dd37b842d57ce5c49f99c079a9095a5f
| * \ Merge remote-tracking branch 'origin/4.7' into 4.8Eike Ziller2018-09-211-18/+30
| |\ \ | | |/ | | | | | | | | | | | | | | | | | | Conflicts: qbs/modules/qtc/qtc.qbs qtcreator.pri Change-Id: Ia1d6035a335fd56069ec8d6d106d58124ee8798d
| | * qtcdebugger: Extend help textFriedemann Kleint2018-09-211-18/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Factor out a helper function to read out the current debugger. Display the currently registered debugger and the Qt version used in the help. Enable copying the message box text. This offers a convenient way of checking since the debugger settings often get overwritten by OS or MSVC updates. Change-Id: If61e30ae22802b71960cb6f3da96100f5fd9e47c Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: hjk <hjk@qt.io> Reviewed-by: David Schulz <david.schulz@qt.io>
| | * Clang: Reuse full type qualification from tooltipsIvan Donchevskii2018-08-203-27/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backported from master. Use qualification helper function from clangtooltipinfocollector.h instead of Unified Symbol Resolution (USR) not to deal with special symbols used in USR. Exception: handle anonymous namespaces via USR because they don't have displayName. Affects current document filter and symbol outline. Task-number: QTCREATORBUG-20917 Change-Id: I97f8fbc8a9f380d220d85837568f56a1a217f035 Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | | Clang/CppTools: Replace Q_DECL_NOEXCEPT with noexceptOrgad Shaneh2018-09-202-2/+2
|/ / | | | | | | | | Change-Id: I105a5ed5e4a5c647f947a413ae8123d02bfdae3f Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Clang: Fix "unhandled enums" warningOrgad Shaneh2018-09-181-0/+2
| | | | | | | | | | Change-Id: I5eb844b2599f3657b72f5f262b5d708f9fc593ac Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Clang: Remove redundant workaround for operator argumentsIvan Donchevskii2018-09-131-10/+3
| | | | | | | | | | | | | | With updated libclang 8c21a7a396 is not required anymore. Change-Id: Ied17a946e5e361441ebe2900dac59b7fe434f36e Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Highlighting: Add highlighting style for punctuationIvan Donchevskii2018-09-132-6/+4
| | | | | | | | | | | | | | | | | | Currently only operators have their own style but not punctuation tokens. Make possible to highlight both. Task-number: QTCREATORBUG-20666 Change-Id: I9533e0f1bef65b86c4e4f5c9756571103584124b Reviewed-by: Marco Bubke <marco.bubke@qt.io>
* | Clang: Add generated files support for pch managerMarco Bubke2018-09-126-39/+47
| | | | | | | | | | Change-Id: I69aae62dd1091a1958d4b0f23a37758c9f58c97b Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Clang: Add refactoring project updaterMarco Bubke2018-09-121-6/+1
| | | | | | | | | | | | | | | | The project parts are now first used by the pch manager and then notified by the pch manager to the refactoring plugin. Change-Id: I88074d8891cd0de9721497bbafee0deffc0b6339 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* | Clang: Enable follow symbol for 'auto' keywordIvan Donchevskii2018-09-121-1/+10
| | | | | | | | | | | | | | | | | | It will follow to the Type which is deduced from auto or to the pointee type if auto is deduced to pointer. Task-number: QTCREATORBUG-17191 Change-Id: I57db3f9b68c3da861691ab148fe39774ad5fceec Reviewed-by: Marco Bubke <marco.bubke@qt.io>