summaryrefslogtreecommitdiff
path: root/src/plugins/debugger
Commit message (Collapse)AuthorAgeFilesLines
* Debugger: Fix crash on close while debuggingChristian Stenger2016-04-052-2/+11
| | | | | | Task-number: QTCREATORBUG-15987 Change-Id: I510add216ec16c0c6622870f26c68c16e99769ed Reviewed-by: hjk <hjk@theqtcompany.com>
* Icons: Move debug run/interrupt/continue/exit from core to elsewhereAlessandro Portale2016-04-0110-7/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Core contains the small variants of debug run/interrupt/continue/exit while the bigger icon variants are in projectexplorer or debugger. That does not seem to have aany reason, at least in today's state of Qt Creator architecture. But above all, it stands in the way when changing debugger icons as planned due to user feedback. This change moves: Core::Icons::DEBUG_START_SMALL to ProjectExplorer::Icons::DEBUG_START_SMALL Core::Icons::DEBUG_EXIT_SMALL to Debugger::Icons::DEBUG_EXIT_SMALL Core::Icons::DEBUG_INTERRUPT_SMALL to Debugger::Icons::DEBUG_INTERRUPT_SMALL Core::Icons::DEBUG_CONTINUE_SMALL to Debugger::Icons::DEBUG_CONTINUE_SMALL This change just moves icons across modules but does not change anything in the UI. Change-Id: I859b901c312d4d16b6c2f687395a7b48c90aab84 Reviewed-by: Alessandro Portale <alessandro.portale@theqtcompany.com>
* Debugger: Avoid setParent(0) on QDockWidgetsUlf Hermann2016-03-311-13/+13
| | | | | | | | | | This spells trouble as dock widgets which have a native window will assume they should be positioned relative to the outermost native window then. Change-Id: I5a8ce5870afacaabe26d4a2d7ac53ffee09328ed Task-number: QTCREATORBUG-15844 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Fix memory leaksOrgad Shaneh2016-03-262-4/+4
| | | | | | | Detected by memcheck. Change-Id: I56c0c39f3aa2251d6425ddc9388fdebc511d7f47 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Re-enable item selection in watches for core dumpOrgad Shaneh2016-03-241-7/+17
| | | | | | | | | | | | | | | In 7de7eb6bcada805eff176634e8131c4914e1213d the behavior on Unrunnable state was changed, probably by mistake. AddWatcherWhileRunningCapability should allow adding watchers while running, but it should not prevent selection when not supported and the inferior is not running. Another missing part was that Unrunnable was considered running, although it shouldn't. Change-Id: I7d27b81977a6921919327b3122a865b7ffa2d0bd Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Fix object leakage on shutdown and heap-use-after-free.hjk2016-03-243-50/+56
| | | | | | Task-number: QTCREATORBUG-15938 Change-Id: I437756705c33730398a129651fabe34c92334656 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Debugger: Also copy fallback data on Copy View Contents to Editorhjk2016-03-241-1/+1
| | | | | | | Getting e.g. an object's address is better than an empty value. Change-Id: Ia78335da8b9eadeadc2e6f1757399ae604eb6610 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Debugger: Move kit guessing out of run control creationhjk2016-03-232-46/+47
| | | | | | | | | It's only used when starting with a -debug command line and when attaching to a running run control. No need to clutter the normal codepaths with it. Change-Id: Ib374c64a7f63fa79e88967573c37a5da1f415d50 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Remove unneeded re-init of RunParameters::useTerminalhjk2016-03-231-4/+0
| | | | | | | ... and some dead code. Change-Id: I22fc6f8ecc09c4c3b928d7b7a510e762a54c0f8f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Reduce scope of scratch data in run control setuphjk2016-03-231-45/+38
| | | | | | | Keep magic guessing to a minimum. Change-Id: I3474b4406fa047dff468e853bd6fee7c962e065d Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Fix creation of task list entrieshjk2016-03-231-35/+41
| | | | | | | | | The debugger validation was operating on wrong language data, i.e. run unconditionally even on pure Qml projects that doesn't require an external debugger. Change-Id: Ic9c96d88f2014d62e8720f1e80134f561fc40bb9 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Don't re-evaluate language choiceshjk2016-03-231-7/+4
| | | | | Change-Id: Idca373abbca7c381bdb0af9c0db6763285700ed8 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Fix typo.Christian Kandeler2016-03-221-1/+1
| | | | | Change-Id: Ie81f9e0fc2ba8eb56019e2b9332298bdb60198a5 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Fix language/engine detectionChristian Stenger2016-03-221-0/+1
| | | | | | Task-number: QTCREATORBUG-15896 Change-Id: I5bc5ed11652932911bdf3b1107998c54353b1268 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Fix initialization of StartParameters::useTerminalhjk2016-03-221-5/+7
| | | | | Change-Id: I8b03b02323c6c4d9d714a6cc0ebc7ee40c0abf92 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Use the cdb process environment to search for symbol paths.David Schulz2016-03-221-1/+1
| | | | | Change-Id: Ia2cfa060c7c836db13bd1cabbde82c7630e4eb33 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Save one loop cycle for struct watchesOrgad Shaneh2016-03-211-1/+1
| | | | | Change-Id: I039dd5f32d4a1e25f1c3a6a7cec78ebdfb3f5e68 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Always set the cdb symbolpath.David Schulz2016-03-181-3/+12
| | | | | | | | | | | In newer cdb versions the default symbol path points to the microsoft symbol server. This can lead to a very long start up time for inferiors including many modules, because every module is verified against that symbol server. Unless the user does not explicitly set such a symbol server in the cdb settings do not use a symbol server. Change-Id: I0b24e2dd12f6cef98c55f4f57f0c800cecfc3c3f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Remove verbose option of the debugger log.David Schulz2016-03-186-44/+4
| | | | | | | | Saves an asking for debugger log roundtrip when the information is just visible in the verbose log. Was just used inside the cdbengine. Change-Id: I9801d3ccd8f273c3b4afcf053b00e7af0953a63f Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: fix crash in updateUiForTargetTim Jenssen2016-03-171-2/+2
| | | | | | | - it happens while removing the only existing kit Change-Id: Ifc2e9eae2836b496f9f696ff91e2999dc1904800 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Avoid switching to debug when the inferior started (or failed)Eike Ziller2016-03-171-3/+1
| | | | | | | | | | | | | We already switch to debug mode when starting the run control. Since loading of projects from the command line is delayed by the project explorer plugin, and that switches to edit or projects mode, we must delay the startup of the debugger when started from the command line until the project explorer plugin really finished all initialization. Change-Id: Icfae2b39878084b024c40fa30a2e35f6a0b94a89 Reviewed-by: hjk <hjk@theqtcompany.com> Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Debugger: Fix restoring of previous modeEike Ziller2016-03-173-1/+7
| | | | | | | | | | | | We have many code paths now that try to make sure that we really end up with debug mode when debugging, leading to multiple calls of activateDebugMode when starting the debugger. Separate ensuring debug mode from saving the previous mode to avoid that debug mode is always the previous mode. Change-Id: Ie9687e2de816b6ae55945b6108a5d4f760962f89 Reviewed-by: hjk <hjk@theqtcompany.com> Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Debugger: Make some of the plugin tests compilable againhjk2016-03-1611-281/+236
| | | | | Change-Id: I5c3885cb385dc1c15dce14f2ff52bf70edee8399 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Adjust error message for Qml Debugger Consolehjk2016-03-161-2/+2
| | | | | | | | | | | | Displaying live contents uses nowadays a feature of Qt 5.7. The previous display was not using the correct context. For Qt < 5.7, access is now only possible when the context is clear (e.g. because a breakpoint was hit). This patch adjusts the error message in the other case. Task-number: QTCREATORBUG-15831 Change-Id: Ic25a3f6bca05523b16063f3e7daed6817c0002e7 Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>
* Debugger: Move last separator in toolbar next to the views menuhjk2016-03-151-1/+1
| | | | | | | Looks more natural there. Change-Id: Id1c717869fbc903e55e4e4a2bc27dfe7db843423 Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
* Debugger: Fix copy to clipboard...Christian Stenger2016-03-151-11/+2
| | | | | | | | ...for Locals & Expressions. Task-number: QTCREATORBUG-15870 Change-Id: I5768908ab731adffd695627206fd03cc5ccce957 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Adapt to Android(?) GDB 7.10's changed register formathjk2016-03-151-2/+6
| | | | | | | The previous 'v4_int32' tag now shows as 'u32'. Look for either. Change-Id: Ie47fd564ec5ee1f08f44f3fb43180f5cdc495884 Reviewed-by: BogDan Vatra <bogdan@kdab.com>
* Debugger: Re-enable initial population of register viewhjk2016-03-151-18/+9
| | | | | | | | | Registers are only retrieved if the view is visible. When it becomes first visible, an extra poke to initiate population is needed. Change-Id: I67c3e166ce98e6405707f8457ebd32d0d1939229 Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
* Debugger: Re-enable retrieval of register contenthjk2016-03-155-8/+13
| | | | | | | Less use of magical object names. Change-Id: I2da17f1cefe1a72f4ea8be4c041e7443265def93 Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
* Debugger: Avoid proxy action loophjk2016-03-141-2/+2
| | | | | | | | Use the real start action data to Initialize the toolbar start proxy action with the real action, not the menu action. Change-Id: I498ba0069e9f259f90122eb6378c79f08e4d24d1 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Debugger: Compile fixhjk2016-03-111-1/+1
| | | | | Change-Id: I6a2019898ad2259c78ec19c6914ddc689f6ef40c Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
* Debugger: Warn on Inappropriate debugger.David Schulz2016-03-111-13/+37
| | | | | | Change-Id: Ib8777277032c5bd062ffaabd6281b6402576d0fe Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com> Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Move main splitter creation to DebugMainWindow setuphjk2016-03-113-68/+98
| | | | | | | | To allow other mode main windows to re-use the perspective concept. Change-Id: Icf00b4f0e4bd73a09ebec131ef9c578154f25eec Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Set default button in Unstarted App dialogOrgad Shaneh2016-03-111-0/+2
| | | | | | | + set initial focus to the executable. Change-Id: I3289e98293cffde6df5eaba844add807bafa6cd8 Reviewed-by: hjk <hjk@theqtcompany.com>
* PythonEditor: Ensure to set environmentChristian Stenger2016-03-111-0/+1
| | | | | Change-Id: Ie216b28b7effb2d44ba507dffa07bb7f463030e7 Reviewed-by: hjk <hjk@theqtcompany.com>
* ProjectExplorer: Remove project parameter from PEP::canRun()hjk2016-03-101-8/+7
| | | | | | | | | | | .. and rename it to canRunStartupProject(). It's only ever used with the startup project. No need to let all callers retrieve it by themselves. Change-Id: I24e56ce339bde70d96e28db51e6ddcf75c277ceb Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com> Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Debugger: Remove unused showPromptDialog functionhjk2016-03-101-21/+0
| | | | | Change-Id: Ie8f2cc905cb5e39c5419837e298c37ed4e1ec377 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* FancyMainWindow: Move toolbarDockWidget "property" to subclasshjk2016-03-092-13/+13
| | | | | | | | This was only used to accommodate the debugger main window and is not needed in the base class. Change-Id: Ifbe9937cd4c21f5bed2fab35b23d874f4f2dd0e4 Reviewed-by: Eike Ziller <eike.ziller@theqtcompany.com>
* Debugger: Use fixed size for perspective combo box...Christian Stenger2016-03-091-0/+10
| | | | | | | ...to avoid adjusting the width again and again while debugging. Change-Id: I67f68b2d54d9f3f9a32cbf1049b4f9e9bd63e010 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Remove one indirection in Debugger::registerActionhjk2016-03-091-29/+20
| | | | | | | ... and the now unused action id -> action description hash. Change-Id: I36ad3658b52fcf32b628620b5cc2bd98bc2ac849 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Do not switch to undefined perspectiveshjk2016-03-091-7/+4
| | | | | | | | | This implies switching to Debug mode and is currently triggered by the GammrayIntegration. Avoid it as long as GammaRay has a mode of its own. Change-Id: I6174554610038cb66fb1bc977dc537b47941a1d8 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Re-add language-dependent contextOrgad Shaneh2016-03-081-1/+8
| | | | | | | Lost in f3bd7412ef510c77a5e811d8ef85f371f0b67735. Change-Id: I6091db212eff8a4ac15c67e99d9e88fab5b11647 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Add a missing space in language listingOrgad Shaneh2016-03-081-1/+1
| | | | | Change-Id: I78a57c5edba1d7cfd59b8b4a93b41fb4ab3ae936 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Merge debug mode and analyze modehjk2016-03-0728-1958/+1385
| | | | | | | | | | | | | | | | | | | | | | | | On the user-visible side, only the 'Analyze' mode button disappears, and instead a combobox to switch between different tools in appears in the Debug mode toolbar. Internally, that's quite some re-organzition: The centralized 'Analyze mode is busy' flag is gone, allowing us to run e.g. ClangStaticAnalyzer and MemCheck in parallel. Analyzer tools and debugger now share the same mechanism to generate/load/save dock widgets. Analyzer tools now create and handle their own start/stop button when appropriate. In general, Analyzer tools can create/handle more than one run control at a time. Further consolidation is possible, e.g. RunControl state handling could be merged into the base ProjectExplorer::RunControl to avoid the still existing duplication in ~15 instances. Change-Id: I91e5940ebc4211f98056d507cf2f7b5f8efe7f07 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Update debugger details when executable is replacedOrgad Shaneh2016-03-074-5/+22
| | | | | | | | Store last modified time in persistent storage, and if it is changed, reinitialize the item. Change-Id: I15e8c843fecd3cebd528f2dadb64645828a2b221 Reviewed-by: hjk <hjk@theqtcompany.com>
* Merge remote-tracking branch 'origin/3.6' into 4.0Eike Ziller2016-03-031-5/+1
|\ | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/projectexplorer/applicationlauncher.cpp src/plugins/projectexplorer/applicationlauncher.h src/plugins/qmlprofiler/qmlprofilerclientmanager.cpp src/plugins/qmlprofiler/qmlprofilertool.cpp src/plugins/qtsupport/uicodemodelsupport.cpp src/plugins/qtsupport/uicodemodelsupport.h Change-Id: I6f6ae77422d99f4f422b108ade6b64273df9dd57
| * Debugger: Fix switching between threads not updating stack pane.Alexandru Croitor2016-02-241-5/+1
| | | | | | | | | | | | Task-number: QTCREATORBUG-15587 Change-Id: I41b2025e26d79ca0acf12d8d6468786f5dd121b2 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Rename Perspective::Split to Perspective::Operationhjk2016-03-033-26/+34
| | | | | | | | | | | | | | We have 'raise' in there now, that's not a split anymore. Change-Id: Id45c606056dfab1317fad37b2fe37216b0eacf85 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Make Perspective status label globalhjk2016-03-025-24/+16
| | | | | | | | | | | | | | | | After some discussion we agreed that the contents is ephemeral and does not need to survive perspective switching. Change-Id: I41de6a8f9478e4bd229c8b204ef7a3fa0a344b75 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Fix compilation without testsOrgad Shaneh2016-03-022-97/+96
| | | | | | | | | | | | | | | | * Some functions were inside WITH_TESTS section. * MSVC requires Perspective::Split to be explicitly exported. Change-Id: I272cefc5c4f446415b2714595f8b398f8ce010a4 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
View Lorry Controller Status