summaryrefslogtreecommitdiff
path: root/src/plugins/debugger/cdb
Commit message (Collapse)AuthorAgeFilesLines
* Utils: Unify CheckableMessageBox and make it look more nativeMarcus Tillmanns2023-05-171-7/+6
| | | | | | Change-Id: I5690c16f38cfd2058e01441283bec28d44cadf75 Reviewed-by: Eike Ziller <eike.ziller@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* All: Use the shorter access to aspect values in a few placeshjk2023-05-121-4/+4
| | | | | Change-Id: Ie0ff0e4e1cf4c2750897ea733dfccb743f44fa93 Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
* Proliferate pathListSeparator()Alessandro Portale2023-05-051-2/+3
| | | | | | Change-Id: I546107af6a88ad5901659a0a64485e4ebca3a164 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Utils: Rename qtcprocess.{cpp,h} -> process.{cpp,h}Jarek Kobus2023-05-042-2/+2
| | | | | | | | Follows QtcProcess -> Process rename. Change-Id: I97235a9a40cb7fd52944515b7ab878d96528f919 Reviewed-by: hjk <hjk@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
* Utils: Rename QtcProcess -> ProcessJarek Kobus2023-05-042-3/+3
| | | | | | | | Task-number: QTCREATORBUG-29102 Change-Id: Ibc264f9db6a32206e4097766ee3f7d0b35225a5c Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* CppEditor: Remove WorkingCopy::contains()Christian Kandeler2023-05-031-2/+2
| | | | | | | Most uses involved a double look-up. Change-Id: Ifeb62ea2361222ed0faad749f44a59735c8d6930 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Add automatic source path mapping for Qt PackagesDavid Schulz2023-04-211-1/+3
| | | | | Change-Id: Iae2a02fc134ac05a30241d7a69302294137d3844 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: support piping the python dumper to cdbDavid Schulz2023-04-202-13/+64
| | | | | | | Fixes: QTCREATORBUG-29000 Change-Id: I7454ded07829d06a33dc1b2ec95a838ca977ca95 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Debugger: Support piping dumpers into gdbhjk2023-04-051-1/+1
| | | | | | | | | | | This allows using dumpers available on the host being used from remotely running gdb. No lldb/cdb yet. Task-number: QTCREATORBUG-29000 Task-number: QTCREATORBUG-16246 Change-Id: Ib1a40a8c0284dcf41e8800d70ca3e632c699b2fa Reviewed-by: David Schulz <david.schulz@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* Merge remote-tracking branch 'origin/10.0'Eike Ziller2023-03-291-2/+2
|\ | | | | | | | | | | | | | | | | | | Conflicts: cmake/QtCreatorIDEBranding.cmake qbs/modules/qtc/qtc.qbs src/plugins/remotelinux/genericlinuxdeviceconfigurationwidget.cpp src/tools/perfparser Change-Id: Ie5643100e0eb00e0933359dce320169b876f5634
| * Debugger: Fix error messageDavid Schulz2023-03-221-2/+2
| | | | | | | | | | | | Change-Id: Idd6b63c8f814f1fbde31ee6cd5c0d4d3c461c895 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* | Debugger: Use FilePath for Moduleshjk2023-03-141-1/+2
| | | | | | | | | | | | Change-Id: Ib3c8cf0de3560fdc77775460aa6282d69dbfef9e Reviewed-by: Marcus Tillmanns <marcus.tillmanns@qt.io> Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* | Debugger: Use more FilePaths in module handling interfacehjk2023-03-142-4/+5
|/ | | | | | Change-Id: Ie058b928900015d0f71776151547bf7ba32a62bf Reviewed-by: David Schulz <david.schulz@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* Debugger: Disable source code display in debugger log for cdbDavid Schulz2023-03-101-1/+0
| | | | | | | | | | Printing the source code line the debugger stopped at causes issue in the general output handling and was just used to conveniently reading the log file. Fixes: QTCREATORBUG-25054 Change-Id: I8db54ef6aa101b2bc7e39984bbf93bd6c9e2dc41 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Add warning if cdb prevents loading of cdbextDavid Schulz2023-03-021-0/+13
| | | | | | | | | | This happens from time to time, but I do not have a way to reproduce it safely. In order to present the user a somewhat usable state after this error cancel the debug run and show a message box with potential fixes. Fixes: QTCREATORBUG-24323 Change-Id: I7e557001bd5dfb3b0abb19fa635bf2d20bb3b16c Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: optimize logging script messagesDavid Schulz2023-02-272-5/+12
| | | | | | | | Avoid parsing the gdbmi output twice just for printing out messages to the debugger log. Change-Id: I928fc3a7d55c318056afff09cfbc25d115932bc0 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: fix debugger log entryDavid Schulz2023-02-211-1/+1
| | | | | | | | Separate command and argurments with a space in the log output. Change-Id: Iaa486ce668ed268cb2a85e1267ca54aa42b07c20 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: hjk <hjk@qt.io>
* Translations: Change translation context prefix from "::" to "QtC::"Alessandro Portale2023-02-101-6/+6
| | | | | | | lupdate would be confused by translation contexts starting with :: Change-Id: Ie95e73436fd3cafc80a8e89f908efadc747e644c Reviewed-by: hjk <hjk@qt.io>
* Translations: Add missing "::" prefixes to QT_TRANSLATE_NOOPAlessandro Portale2023-02-081-6/+6
| | | | | | Change-Id: I7589a2d4003394e38e58c095b510f8eff33263b8 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Utils: Replace Environment.isValid() with .hasChanges()hjk2023-01-271-1/+1
| | | | | | | | | That's closer to the intended semantics. The "other" use in docker will be changed to an optional<Environment> as follow-up to keep this here mechanical. Change-Id: I43ef9da6c9c7731b28f9d6fab6413ce9c4f428b4 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Debugger: Adapt to change in FilePathChristian Stenger2023-01-181-1/+1
| | | | | Change-Id: I947f7c22bd59e445c94784fc50f29f6b571aad5a Reviewed-by: hjk <hjk@qt.io>
* Remove GPL-3.0+ from license identifiersKai Köhne2023-01-068-8/+8
| | | | | | | | | | | | | | | 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>
* plugins: Remove foreach usageArtem Sokolovskii2022-12-201-1/+1
| | | | | | | Change-Id: Idf60caed3b5f4ec4c6e990a9e3188487bcc824f8 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
* Debugger: Pass context object to lambda connectionsJarek Kobus2022-12-091-4/+2
| | | | | | | | | Remove some unneeded lambda () brackets. Glue lambda brackets with parameters brackets. Change-Id: I5df67cf01e497ad39c070a3f138a647762f2c33c Reviewed-by: hjk <hjk@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* Merge remote-tracking branch 'origin/9.0'Eike Ziller2022-12-074-64/+52
|\ | | | | | | | | | | | | | | Conflicts: src/plugins/cppeditor/cppeditorplugin.cpp src/plugins/docker/dockerdevice.cpp Change-Id: I41747ff9490e4ab20340d4d4acb4655100aaa3d2
| * Debugger: map reported breakpoint file locationsDavid Schulz2022-11-304-64/+52
| | | | | | | | | | | | Fixes: QTCREATORBUG-28521 Change-Id: I8229483df7fb0c51750e19d4558e81f5320e7f33 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* | Merge remote-tracking branch 'origin/9.0'Tim Jenssen2022-10-251-2/+2
|\ \ | |/ | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/qmldesigner/components/componentcore/modelnodeoperations.cpp src/plugins/qmldesigner/components/stateseditornew/propertychangesmodel.cpp src/plugins/qmldesigner/components/stateseditornew/propertymodel.cpp src/plugins/qmldesigner/qmldesignerprojectmanager.cpp src/plugins/qmldesigner/qmldesignerprojectmanager.h Change-Id: Ib029a830ee99190bc4ea2ad75d9300bfa86b42d9
| * UI text: Fix UI text to follow the guidelinesLeena Miettinen2022-10-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | - Fix typos - Replace "options" with "preferences" - Fix capitalization - Remove "please" - Do not use contractions Task-number: QTCREATORBUG-28334 Change-Id: Ie029eae435634aa2fb354e173fa107af72f7b025 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* | Get rid of unneeded includes of RunControlJarek Kobus2022-10-251-1/+0
|/ | | | | | | | | | Removed also some other includes that were marked with yellow triangles. In some cases includes of runcontrol.h were substituted with other includes, like qtcassert.h. Change-Id: Ica40f68198c7f8f70e6047acb2eddc8ef017e43d Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Debugger: Fix interrupting with cdbDavid Schulz2022-10-132-0/+27
| | | | | | | | | | | | It turns out sending ctrl+c events to cdb results in inconsistent behavior when trying to interrupt with the range from interrupting twice to not at all. Use the previous method of using signal operation for local debugging, and use the ctrl+c event only when cdb is attached to a remote server. Fixes: QTCREATORBUG-28279 Change-Id: Iccd2016685ba707b375aebfd88eccc253dde1d1d Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: remove can interrupt checkDavid Schulz2022-10-132-9/+0
| | | | | | | | We are now able to interrupt via Ctrl+C so we do not necessarily need a local inferior with a pid. Change-Id: I31997c5d2885b748699756908ef46686798ccafa Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Port from qAsConst() to std::as_const()Marc Mutz2022-10-072-4/+4
| | | | | | | | | | | | | | We've been requiring C++17 since Qt 6.0, and our qAsConst use finally starts to bother us (QTBUG-99313), so time to port away from it now. Since qAsConst has exactly the same semantics as std::as_const (down to rvalue treatment, constexpr'ness and noexcept'ness), there's really nothing more to it than a global search-and-replace. Task-number: QTBUG-99313 Change-Id: I88edd91395849574436299b8badda21bb93bea39 Reviewed-by: hjk <hjk@qt.io>
* FilePath: Return optional bytearray for file contentsEike Ziller2022-09-121-1/+1
| | | | | | | For differentiating between "error" and "empty file". Change-Id: I2c019ceac625e7be3180afa4d47ae3a24df91c1d Reviewed-by: hjk <hjk@qt.io>
* Debugger: Use qtcEnvironmentVariable* instead of qEnvironmentVariable*Eike Ziller2022-08-311-7/+5
| | | | | | | | | | | | And instead of qgetenv. Takes Qt Creator's setting at "Environment > System > Environment" into account, which makes it easier on some platforms to set them (e.g. macOS), can be configured differently in different settings paths, and potentially can be changed at runtime (depending on usage). Change-Id: I3ea7623fb528e13a202afa2f89b00e5ee83962d8 Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: David Schulz <david.schulz@qt.io>
* Use SPDX license identifiersLucie Gérard2022-08-268-192/+16
| | | | | | | | | 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>
* CdbEngine: Remove unused fieldJarek Kobus2022-08-022-4/+3
| | | | | | | | This field in only set, but never read. Change-Id: Ie42ccad32ca90e1b4a0dd9f4c2e213ea4c3ca180 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* Utils: Settle on lowerSnakeCase for LayoutBuilder::Setterhjk2022-07-271-6/+6
| | | | | | | Change-Id: Icfe487d8c7712cc8ad28f0f61e8ae4778a356fc3 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Utils: Introduce Layouting::{br,st}hjk2022-07-251-3/+3
| | | | | | | | | | | | | ... as "standard" ways to define line breaks and simple stretch. There have already been too many patterns to do it. Break() and Stretch() still work for the patches in flight, but they are planned to be removed. Change-Id: I9b70dcdc11244a904a496b0c55938dfb0b265fc8 Reviewed-by: Alessandro Portale <alessandro.portale@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* Utils: Make Layouting::Group { .. } less weirdhjk2022-07-221-12/+18
| | | | | | | | | It had a implicit vertical layout leading to unneded layout nesting in quite a few cases. The price is an added Column { ... } in those places where the implicit vertical layout was sufficient before. Change-Id: I3ae1f03f9c1d691bd0c563b0447edd03ee02bbd2 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Utils: More porting.h related changeshjk2022-07-141-1/+1
| | | | | Change-Id: I528a6950dfa6e09eb7f7ada265c8c41dba816bfd Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Debugger: Convert to Tr::trhjk2022-07-083-46/+32
| | | | | Change-Id: I5d2475c790851c68f9997ac6af72b5eaca58482d Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Debugger: use QtcProcess output bufferDavid Schulz2022-07-042-35/+4
| | | | | Change-Id: I03326879bbcf2411a1feab8b6826ddd4c0981286 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: remove unused CdbCommandDavid Schulz2022-07-042-13/+0
| | | | | Change-Id: Idad2b62ca60da1749970ced21445c2d82bc2a798 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: sort cdb engine includesDavid Schulz2022-07-041-8/+8
| | | | | Change-Id: Iddbf7dace0f8b5f36ad2ab9823815588ac087042 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Fix interrupting remote cdb processDavid Schulz2022-06-272-35/+1
| | | | | | | | | | Since we can use the ctrlc stub for cdb now we also can use the interrupt functionality to force the debugger to halt. This also works for remote sessions. Fixes: QTCREATORBUG-21657 Change-Id: I049aea781e984c31ac5744f87f9590f910234679 Reviewed-by: hjk <hjk@qt.io>
* Debugger: Fix compile with Qt5Christian Stenger2022-06-241-1/+1
| | | | | | | Amends f4ea9e09ec3. Change-Id: I642c99ffe9a932c6c74b771eab5d2122d01b1ee3 Reviewed-by: David Schulz <david.schulz@qt.io>
* CdbEngine: Don't call blocking waitForStarted()Jarek Kobus2022-06-232-11/+18
| | | | | | | | | | Connect to started() signal instead and continue setup in its handler. Handle failed to start case inside done() signal handler. Change-Id: I23fd222a6c73147ee439381cac79f29cffad560c Reviewed-by: David Schulz <david.schulz@qt.io> Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
* cdb: Use separate qtcreatorcdbext suffix for arm architectureCristian Adam2022-06-222-10/+17
| | | | | | | | This would allow arm64 to coexist with amd64 on the same installation. Change-Id: I578aab3d363bf93c2f61d4a7c93576ce19fba4b6 Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* CdbEngine: Connect to done() signalJarek Kobus2022-06-202-17/+14
| | | | | | | | Instead of connecting to errorOccurred() and finished() signals. Change-Id: Id5e92fbccd361c7bce7718704f33e221649ab47b Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: David Schulz <david.schulz@qt.io>
* Debugger: Use ProcessResultData a bit morehjk2022-06-161-1/+2
| | | | | | | Instead of its individual items. Change-Id: Icd668a61de1c2de676e0a44769f1c1bb5375c523 Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>