summaryrefslogtreecommitdiff
path: root/src/plugins/debugger/qml/qmlcppengine.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Debugger: Remove unused fileshjk2019-05-131-692/+0
| | | | | Change-Id: I5b689bcdeaca9499daae3f8b28c8e67c045b9d88 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Debugger: Remove unused DebuggerEngine::isSynchronoushjk2018-07-241-5/+0
| | | | | Change-Id: Ia1eb3a65fabbf966a1766d68456144fd9e093ae8 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Fix combined debugger interrupt state machinehjk2018-05-231-0/+1
| | | | | | | | | This fixes the case where the mixed engine was used, and called one of the slave engine's interruptInferior() without setting the InterruptStopRequested state in the slave. Change-Id: I45adbdcebc02940d6fcfb4299ef9f4bef1341b32 Reviewed-by: David Schulz <david.schulz@qt.io>
* Debugger: Accept transition from InferiorStopOk to InferiorRunOkUlf Hermann2018-02-121-3/+7
| | | | | | | | | | This can happen if we continue after an automatic break point at application start. Set the state to InferiorRunRequested first, and then to InferiorRunOk. Change-Id: I0c4094ef0b335f31099cceb49b23aa3bc1a9a7af Task-number: QTCREATORBUG-19712 Reviewed-by: hjk <hjk@qt.io>
* Debugger: Combine the InferiorShutdown{Ok,Failed} stateshjk2017-12-141-7/+2
| | | | | | | ... into a InferiorShutdownFinished. Change-Id: Icb5394f38f52f7cf300dc83f4eb8f2f0777bbcba Reviewed-by: David Schulz <david.schulz@qt.io>
* Debugger: Combine the EngineShutdown{Ok,Failed} stateshjk2017-12-141-7/+2
| | | | | | | | | ... into a EngineShutdownFinished. They were never handled differently, and the only option is to proceed to DebuggerFinished anyway. So simplify the state machine a bit. Change-Id: Ied3be86fff6750abca578dc6788e4be1d895692b Reviewed-by: David Schulz <david.schulz@qt.io>
* Debugger: Merge EngineSetup and InferiorSetup start phaseshjk2017-12-061-21/+0
| | | | | | | | | It was needed in the past to trigger e.g. gdbserver setup in remote cases which is nowadays handled by separate RunWorkers. Change-Id: I30bce071dab0779cce2e7abef7b31550d8539461 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: David Schulz <david.schulz@qt.io>
* Debugger: Remove public access to QmlCppEngine::m_qmlEnginehjk2017-11-081-14/+9
| | | | | | | Not used anymore. Change-Id: I30605916bcdb4e5231d86ddc371f2d41e3d9d526 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Synchronize slave engine inferior setup phases againhjk2017-11-071-2/+2
| | | | | | | | | | | This reverts most of 4977c915 and applies the same pattern for setupEngine. Turns out that the removal was premature due to the special needs of Android setup. Task-number: QTCREATORBUG-19199 Change-Id: I4e2ff4f34a842f3296999a0651e7e8fd6470198c Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Pass interrupt request more directly to engineshjk2017-10-171-7/+1
| | | | | | | | | De-virtualize requestInterruptInferior() and let always the master engine react to the request. interruptInferior() diverts to the active slave in case of the combined engine. Change-Id: I5d62cf9379c8a452132827897a9ac51e4027f595 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Directly pass from engine to inferior setup in all engineshjk2017-10-171-2/+2
| | | | | | | | | This removes the setupSlaveInferior() synchronization point which wasn't really needed anymore as it was always triggered in response to a notifyEngineSetupOk() of the master engine. Change-Id: I24faec36b180fa961e10dfd88fe4d874f7310140 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Simplify QmlEngine constructionhjk2017-10-041-4/+4
| | | | | Change-Id: Id237998cf2d9a03f9c2929043fdb45ac49d4addb Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Centralize abort handlinghjk2017-09-211-2/+2
| | | | | | | | | ... and apply even more force on the second trying by forcing ramp down of the runControl itself instead of hoping that it would pick up hints. Change-Id: I9d0f4130cb9a137b91c9fa81c3d255f236f98be0 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Merge DebuggerStartParameter into *RunParametershjk2017-09-131-1/+0
| | | | | | | | | | The users (typically target specific DebuggerRunTool derived classes) are meant to use the individual setter functions nowadays, not the set up the full structure, so the members are a true implementation detail now. Change-Id: Ida04801e3230a2fe8bbadde8845e58c3077c87a5 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: De-virtualize *Engine::notify* functionshjk2017-09-131-30/+0
| | | | | | | | Most were never re-implemented, so just for debugging purposes in the mixed engine. Change-Id: If569e0e553e3fdd5ef83b3bdfdbd22fd0ad5b90f Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Remove remote setup sub-statemachineryhjk2017-06-221-17/+0
| | | | | | | | Not needed anymore in the world of RunWorkers. Change-Id: Id7fb24fece6acb03de12f2677dd99a05c513e7a4 Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>
* Debugger: Add missing breaks in switch statementsOrgad Shaneh2017-06-101-0/+1
| | | | | | | Detected by GCC7 Change-Id: Ic4a1f0adac486d3f6d0f7da15b24e983c17225f3 Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
* Debugger: Fix compile after 36ec37b29d22a5128d1ab088a73d6b5237ac56f5David Schulz2017-05-191-1/+1
| | | | | Change-Id: Ibd252961a5646ef3a9eec91db62c1b2fa74a47af Reviewed-by: hjk <hjk@qt.io>
* Debugger: Streamline ramping downhjk2017-05-191-11/+9
| | | | | | | | | There were only two used target states, and in case of mixed debugging all parts of the machinery better agree on the direction. So one bool in the (shared) runTool is sufficient. Change-Id: Iffbf1651b82dde707cfc37d8da9d3da573b34b76 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Move run parameters from engine to tool runnerhjk2017-05-181-19/+16
| | | | | | | | The parameters belong to the run control, they should not be triplicated in case of a combined engine. Change-Id: I4dd84220edbd7a44b902cc52627fe01d0568db75 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Base magic Qml connection discover on base run control signalhjk2017-04-201-9/+0
| | | | | | | | Less debugger specific code and less convoluted paths in debugger message handling. Change-Id: Ib298889c386d65f17acbdfc585188097bb20ed74 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Run fresh expression evaluator also in combined debugginghjk2017-02-211-0/+5
| | | | | | | | | doUpdateLocals() was called, but not re-implemented in the combined case. Task-number: QTCREATORBUG-17763 Change-Id: Id8c17ce0bda79cd61c57624aea5a05e4aad96c79 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger: Shut down QML engine on QmlCppEngine::shutdownEngine()Ulf Hermann2017-01-121-0/+1
| | | | | | | Otherwise the console's script evaluator is not removed. Change-Id: I0092e8571d6d44b208167a7bf6bc2a929055e09c Reviewed-by: hjk <hjk@qt.io>
* Debugger: Fix QML extra stack loading for GDBhjk2016-10-121-0/+6
| | | | | | Change-Id: I702bb8f183e93fe25857115225fa55607ffb1674 Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io> Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Debugger/BinEditor: Standardize interfacehjk2016-07-211-3/+7
| | | | | | | Change-Id: I4acf6bc7648e57c564e86023176ae3905a293a99 Reviewed-by: Orgad Shaneh <orgads@gmail.com> Reviewed-by: Christian Stenger <christian.stenger@qt.io> Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* Debugger: Remove debuggerstringutils.hhjk2016-06-091-6/+6
| | | | | | | | | | With QT_RESTRICTED_CAST_FROM_ASCII making GdbMi etc operate on QString is feasible again. Take this as opportunity to move debugger encoding handling closer to a 'conversion on input and output if needed, storage in QString only' scheme. Change-Id: I2f10c9fa8a6c62c44f4e6682efe3769e9fba30f7 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* Update License according to agreement with Free Qt FoundationTobias Hunger2016-01-191-17/+12
| | | | | | | * Update files in src/plugins Change-Id: Ia5d77fad7d19d4bb3498e78661982f68729adb22 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Debugger: Don't reset script evaluator in QmlCppEngineUlf Hermann2015-12-091-1/+0
| | | | | | | | If we don't set it, we shouldn't delete it, either. The QML slave engine will handle that. Change-Id: Idef54e6bbb4124ed195d88a367ef07c9bb8f6b3d Reviewed-by: hjk <hjk@theqtcompany.com>
* Move QmlConsole to Debuggerhjk2015-12-091-4/+2
| | | | | | | | | | | Now it is closer to its only user and possibly reusable for no-QML uses there. We also drop the QML/JS syntax checker. The application being debugged can already tell us about syntax errors. There is no need to duplicate that functionality. Change-Id: I2ba151f9f4c854c6119ba5462c21be40bddcebf9 Reviewed-by: Ulf Hermann <ulf.hermann@theqtcompany.com> Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Slim down CHECK_STATE() expansionhjk2015-09-151-8/+1
| | | | | | | Less generated code. Change-Id: Iafcafc875afa3ead12e10fd5f316d9663b2f9823 Reviewed-by: Niels Weber <niels.weber@theqtcompany.com>
* Debugger: Split update/expand paths also in mixed enginehjk2015-07-161-0/+8
| | | | | Change-Id: Ic341e4a3e974cfde3fbf4afd8cef41080b922067 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Split updateWatchData() pathshjk2015-07-161-3/+3
| | | | | | | | | There are two cases that do not coincide in asynchronous engines such as the QmlEngine: Inserting a new watch item, and expanding the children of an existing item. Change-Id: Ic98a5f1e89aca37146039a241de737c407606e83 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Rework start parameter completion.hjk2015-06-251-6/+8
| | | | | | | | | | | It's again a linear process now, with explicit entrance point for all users. Task-number: QTCREATORBUG-14618 Change-Id: I96c08947270ce34d7bc6c8be1d7f350dbfa14794 Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com> Reviewed-by: Orgad Shaneh <orgads@gmail.com> Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Remove the InferiorExited statehjk2015-06-171-6/+0
| | | | | | | | | | | | There are now several possible orders of events between sending a 'kill' and receiving something that indicated a properly shut down inferior. Coordinating the InferiorExited state during engine ramp down is not worthwhile as that state had only a forced transition to InferiorShutdownOk anyway. For the user (and user code), only the fact that ramp down is finished is interesting. Change-Id: Ic22131d5a1066dc62bd069f532c28b773f231088 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Consolidate GDB and LLDB "updateLocals" code pathshjk2015-06-101-11/+10
| | | | | | | | | | | | | | | | | This splits the bool setToolTipExpression() operation into a bool canHandleToolTip(), and the actual processing of the request, which is mostly identical to the handling of a watcher. Handling a watcher is now mostly the same as a full Locals update, except for the 'partial' flag. Pushing the handling of that down to the bridges gives identical code paths in the gdb and lldbengine. Move that to the DebuggerEngine base class. Change-Id: I3861b43e8630c7e7bd57fcd549b2a2387e3d4869 Reviewed-by: hjk <hjk@theqtcompany.com> Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Split off runtime-only items from DebuggerStartParametershjk2015-05-281-6/+6
| | | | | | | | | Right now, DebuggerStartParameter is part of the debugger plugin external interface. Better not make it fatter than necessary. Change-Id: I3b9205089f971dd314ccba1335b5ca2d4d9fe7c6 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
* Debugger: Remove most remaining occurrences of WatchDatahjk2015-03-201-10/+10
| | | | | | | | | ... in WatchHandler and DebuggerEngine interface. Adjust using code. Change-Id: I6371f3e96ba46a783e23a5767cdde24a10d3fce7 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Remove widget argument from tooltip requesthjk2015-03-061-9/+6
| | | | | | | The only information ever used is whether it's c++ or not. Change-Id: I4ca00663856dd66cbdf58c468f175a8c9e41d6a5 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Remove WatchUpdateFlagshjk2015-03-061-4/+3
| | | | | | | Only needed in the non-Python days. CDB didn't use it to start with. Change-Id: Ia609218cc1a694f797e278c4d99ead68e6212938 Reviewed-by: hjk <hjk@theqtcompany.com>
* Debugger: Avoid some undesired state transitions on shutdownhjk2015-03-051-0/+5
| | | | | | | | | Seeing an ill engine while trying to stop means that the stop failed. Also, a finished master engine doesn't need further updates. Task-number: QTCREATORBUG-14089 Change-Id: Icd5b090dfcdc0875953d8756a909b1405e05f71f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Mark unexpected state transitions of mixed enginehjk2015-03-051-40/+42
| | | | | | | ... prominently with red color in the log. Change-Id: Id9d86227bbcc269a0870ff93e372f28ab4b2618b Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Simplify *SlaveEngineType StartParameters membershjk2015-03-051-1/+1
| | | | | | | | The secondSlaveEngineType was not really used, and the first either NoEngineType or identical to the cppEngineType. Change-Id: I1725728ca38636f228ff9b9d8ca7d643a34c64e8 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Don't show a location marker in the mixed master enginehjk2015-03-051-5/+0
| | | | | | Task-number: QTCREATORBUG-14087 Change-Id: I7e8f116c97cdea0bd8234d060d068435229e6fc2 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: State machine cosmeticshjk2015-02-271-1/+1
| | | | | | | Remove one unused notification, rename notifyInferiorUnrunnable. Change-Id: I585d3a8606585fbdd8ad48e5eeecf629542bd93c Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Fix build.Christian Kandeler2015-02-261-1/+1
| | | | | Change-Id: Ic92e5854535a668f49f5e49a51d190132d27e9e6 Reviewed-by: hjk <hjk@theqtcompany.com>
* Clean exported headers of the Debugger plugin.Friedemann Kleint2015-02-261-0/+1
| | | | | Change-Id: I70ca6773e0f6aa6d80a441f8920d7f5d2418e1bf Reviewed-by: hjk <hjk@theqtcompany.com>
* Clean exported headers of the TextEditor plugin.Friedemann Kleint2015-02-261-0/+1
| | | | | Change-Id: I1e7dd34ba5a51fb0b34d137dc03add4457b32ed1 Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* Merge commit '3c85058694ee2e41658d17f524fb48f0b187d2fe'Eike Ziller2015-02-121-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/libs/utils/tooltip/tipcontents.cpp src/libs/utils/tooltip/tipcontents.h src/plugins/android/androiddeployqtstep.cpp src/plugins/baremetal/baremetalconstants.h src/plugins/baremetal/baremetaldevice.cpp src/plugins/baremetal/baremetaldevice.h src/plugins/baremetal/baremetaldeviceconfigurationwidget.cpp src/plugins/baremetal/baremetaldeviceconfigurationwidget.h src/plugins/baremetal/baremetaldeviceconfigurationwizard.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.h src/plugins/baremetal/baremetalplugin.cpp src/plugins/baremetal/baremetalplugin.h src/plugins/baremetal/baremetalruncontrolfactory.cpp src/plugins/baremetal/baremetalruncontrolfactory.h src/plugins/cppeditor/cppcodemodelinspectordialog.cpp src/plugins/cppeditor/cppdoxygen_test.cpp src/plugins/cppeditor/cppdoxygen_test.h src/plugins/debugger/breakpointmarker.cpp src/plugins/debugger/debuggeritemmodel.cpp src/plugins/debugger/debuggeritemmodel.h src/plugins/debugger/loadcoredialog.cpp src/plugins/genericprojectmanager/cppmodelmanagerhelper.cpp src/plugins/projectexplorer/addnewmodel.cpp src/plugins/projectexplorer/addnewmodel.h src/plugins/projectexplorer/jsonwizard/jsonfieldpage.cpp src/plugins/qmlprofiler/abstracttimelinemodel.cpp src/plugins/qmlprofiler/abstracttimelinemodel.h src/plugins/qmlprofiler/notesmodel.cpp src/plugins/qmlprofiler/qml/CategoryLabel.qml src/plugins/qmlprofiler/qml/MainView.qml src/plugins/qmlprofiler/qml/Overview.js src/plugins/qmlprofiler/qml/Overview.qml src/plugins/qmlprofiler/qml/TimeDisplay.qml src/plugins/qmlprofiler/qml/TimeMarks.qml src/plugins/qmlprofiler/qmlprofilertimelinemodelproxy.cpp src/plugins/qmlprofiler/sortedtimelinemodel.cpp src/plugins/qmlprofiler/sortedtimelinemodel.h src/plugins/qmlprofiler/timelinemodelaggregator.cpp src/plugins/qmlprofiler/timelinemodelaggregator.h src/plugins/qmlprofiler/timelinerenderer.cpp src/plugins/qmlprofiler/timelinerenderer.h src/plugins/qmlprojectmanager/QmlProjectManager.json.in src/plugins/texteditor/findinfiles.cpp src/plugins/vcsbase/vcsconfigurationpage.cpp src/shared/qbs src/shared/scriptwrapper/interface_wrap_helpers.h src/shared/scriptwrapper/wrap_helpers.h tests/auto/qmlprofiler/abstracttimelinemodel/tst_abstracttimelinemodel.cpp tests/system/suite_debugger/tst_debug_empty_main/test.py tests/system/suite_debugger/tst_qml_js_console/test.py tests/system/suite_debugger/tst_qml_locals/test.py Change-Id: I67540b648f8b162496f4aa606b04d50c7c9125c6
| * Update LicenseEike Ziller2015-01-161-6/+6
| | | | | | | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | Debugger: Re-work breakpoint storage handlinghjk2015-01-151-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | The actual data is now in a TreeModel. As interface to individual breakpoints there's a new Breakpoint class essentially providing a checked handle. On the user code side breakHandler()->foo(bpId) is replaced by bp.foo(). Change-Id: I82f435bad6301fce85a1d82bf6bf39e9ddba511e Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
View Lorry Controller Status