summaryrefslogtreecommitdiff
path: root/src/plugins/debugger
Commit message (Collapse)AuthorAgeFilesLines
* Debugger: Use base DebuggerCommand also in PdbEnginehjk2015-02-133-18/+12
| | | | | Change-Id: I92c2668353c05c00b1593439b55745d9b209a997 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Use base DebuggerResponse also in PdbEnginehjk2015-02-132-69/+41
| | | | | Change-Id: I1bcc62f191dfc7a201da7b5723c15587c7ba4e7f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Cleanup lldbbridge.pyhjk2015-02-131-10/+5
| | | | | | | | | Move reusable Breakpoint enum to base, use 'theDumper' uniformly across backends as name for the Dumper singleton. Signal engine setup only if the Dumper successfully initializes. Change-Id: If1b6be9d054f249d0d46061e4949cbecf4fd09cb Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Debugger: Start moving PdbEngine to common Gdb/Lldb base architecturehjk2015-02-133-5/+10
| | | | | | | | | | | | That's still not meant for "production", but just the "3rd use case needed to get abstractions right". For testing, remove all build steps, and use a custom run step with executable /usr/bin/python and argument pointing to the main .py file. Change-Id: I6ae6ed08597896ea979ee58e73c546c7892e8be2 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* Merge remote-tracking branch 'origin/3.3'Eike Ziller2015-02-123-3/+5
|\ | | | | | | | | | | | | | | | | | | Conflicts: src/plugins/debugger/watchhandler.cpp src/plugins/projectexplorer/kitmodel.cpp src/plugins/qbsprojectmanager/qbsprojectmanager.cpp src/shared/qbs Change-Id: I6a68090993a264e93ac7850858cc24ba6bdb5602
| * V8Debugger: Remove unused memberKai Koehne2015-02-041-1/+0
| | | | | | | | | | Change-Id: Ieaea81203cad0170d638dcee8b91b73bb3bc7144 Reviewed-by: hjk <hjk@theqtcompany.com>
| * V8Debugger: Don't send watcher requests if qml engine has not stoppedKai Koehne2015-02-021-0/+3
| | | | | | | | | | | | | | | | | | | | This fixes a regression introduced in a8ebf535b829dac. We shouldn't send watcher evaluation commands while the QML engine hasn't actually stopped, because this might lead to an assert on the server side. Task-number: QTCREATORBUG-13916 Change-Id: I300e2f73c0a1a03e0e6187a5357abeba408f72a0 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
| * Disable broken debugger tooltips for QMLKai Koehne2015-01-301-1/+1
| | | | | | | | | | | | | | | | | | | | Looks like the proper handling of tooltips while debugging a QML app is broken since commit 7f09d0b756ff3f9 . Just disable the functionality for now. Task-number: QTCREATORBUG-13854 Change-Id: I346058f6f9753475ec209ec8befdd5425bbd83d0 Reviewed-by: hjk <hjk@theqtcompany.com>
| * Debugger: Fix scoping bug in watch handler.Christian Kandeler2015-01-211-1/+1
| | | | | | | | | | | | | | Found by clang static analyzer. Change-Id: Ib25232b26c314d4da4707f7903f1b75b907b241d Reviewed-by: hjk <hjk@theqtcompany.com>
* | Merge commit '3c85058694ee2e41658d17f524fb48f0b187d2fe'Eike Ziller2015-02-12182-1090/+1090
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-16185-1108/+1108
| | | | | | | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | Debugger: Some code modernizationhjk2015-02-1215-158/+177
| | | | | | | | | | | | | | Qt 5 signal/slot, a few auto. Change-Id: Ia17f1b3dcd41dfe0a373c14e7c1d34d210d084f1 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Combine some watch related GDB and LLDB code pathshjk2015-02-125-35/+41
| | | | | | | | | | Change-Id: I20e5f6f04418c931354a07bc3597f6982fcaf942 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Convert bb extension to use runCommandhjk2015-02-122-50/+36
| | | | | | | | | | Change-Id: I528c51d2b02dccd91429778ef8c940fe5565345f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Move bbsetup over to runCommand()hjk2015-02-122-4/+6
| | | | | | | | | | Change-Id: I0a19b28d24cb570651ea84bf7cdbc9a761f85a0b Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Complete switch to Python for GDB stack generationhjk2015-02-112-23/+17
| | | | | | | | | | | | | | | | | | The iteration in Python was only used for the 'native mixed' case before. Seems reasonably fast and robust now to always enable it. Also, make the calling code use 'runCommand'. Change-Id: I10565a725dfaa9bf46c28739c69e9f2546498929 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Consolidate some GDB-through-Python interactionhjk2015-02-112-6/+13
| | | | | | | | | | | | | | | | | | Instead of calling a GDB extension command invoking a function on the well-known Dumper instance, call the Dumper function directly. This also makes the code more similar to the LLDB side. Change-Id: I4b23177eb72a904721b63c578ce7fbfe204f02a2 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Add tooltip to tooltip copy buttonhjk2015-02-111-0/+1
| | | | | | | | | | | | | | | | Hard to guess otherwise what it is meant for. Change-Id: I92221da8d0dbe92b579d0cf97a76224a57447ec0 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@theqtcompany.com>
* | Debugger: Go back to plain QTreeView for tooltipshjk2015-02-111-7/+8
| | | | | | | | | | | | | | | | | | Nothing of the BaseTreeView functionality is used, and there seems to be a off-by-1 in column width computation leading to unnecessary eliding. Change-Id: I4426070742801864edfdba4c912ad8cc8a15f7bb Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Using DebuggerCommand instead of bbedit callhjk2015-02-111-12/+8
| | | | | | | | | | | | | | | | Mid-term goal is to reduce the amount of similar code at least in GDB and LLDB engines. Change-Id: I4883686a102246be3135ddc694b837b1187ac9e9 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Fix off-by-one line in tooltip contexthjk2015-02-101-6/+3
| | | | | | | | | | | | | | | | The first function line (usually containing the function parameter) was attributed to the preceding function. Change-Id: Ia2b394cfa5a0edc909bc998bcce6bd92ae5b5b87 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Startup fix after 807c3a5ad2hjk2015-02-101-22/+21
| | | | | | | | | | Change-Id: I731a2916ba7580caaaeea6a7de72e748c30f31f1 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Pass RunControl in DebuggerStartParametershjk2015-02-103-7/+10
| | | | | | | | | | | | | | .. to simplify DebuggerRunControlFactory::doCreate() call. Change-Id: I4dd0c224968bb8a388ea7f095b940b66ee606ab1 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Start to disentangle DebuggerStartParameter setuphjk2015-02-101-36/+28
| | | | | | | | | | Change-Id: Ife19d506146806d965a2552b1390fae4545b3459 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Don't access model in view's reset()hjk2015-02-101-2/+4
| | | | | | | | | | | | | | | | | | This can legitimately be triggered by from a QAbstractItemViewPrivate::doDelayedReset() after the model has been destroyed. Change-Id: Id0b4f45c8386485a1128bfbeed3366caa87c6bfa Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Valgrind: Suppress messagebox on SIGTRAP for vgdbOrgad Shaneh2015-02-102-1/+5
| | | | | | | | | | Change-Id: I5af3a3a67a9c9903337e0035733a74ad482ba197 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Comment out unused variablesNikolai Kosjar2015-02-101-3/+3
| | | | | | | | | | | | | | | | | | debuggertooltipmanager.cpp:98:12: warning: unused variable 'modelElementC' debuggertooltipmanager.cpp:99:12: warning: unused variable 'modelColumnCountAttributeC' debuggertooltipmanager.cpp:100:12: warning: unused variable 'modelRowElementC' Change-Id: I141a3b86080e1855b83bb277a5d6a04428925e03 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Update watchers and return value views properly.hjk2015-02-091-0/+1
| | | | | | | | | | | | | | This was a regression after the watch model changes. Change-Id: I4cf369d972b9a6e2f52800556acbd3711744724c Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Rework Tooltips. Again.hjk2015-02-095-544/+394
| | | | | | | | | | | | | | | | | | | | | | | | | | Too much layers of complexity. Instead of keeping an eye on the "live" tree model and switch to a static StandardItemModel whenever live synchronization is not possible (and do all the tree view snake oil magic that's needed to make the switch appear "smooth") keep static copies of relevant parts of the live model, and update them whenever the the live model changes. Change-Id: I88a7de67f7703cd2fed041351346b1c7ada0839e Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com> Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Remove unused UpdateData::tooltipOnly memberhjk2015-02-063-10/+2
| | | | | | | | | | Change-Id: I15c6f16a258c0d664d2378150c4b3277a29c4b5c Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Remove unused UpdateParameters::tooltipExpression memberhjk2015-02-063-3/+0
| | | | | | | | | | Change-Id: I61360b151ac8e6181b0635437468b4cd897c9ee8 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Merge GdbEngine::GdbCommand into DebuggerCommandhjk2015-02-063-77/+71
| | | | | | | | | | | | | | Mostly the timestamp and mechanical renaming. Change-Id: I9a57302cba454b3c5cf6ce5c8279f817bf85cf2e Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Replace the QVariant callback cookies by direct parametershjk2015-02-0610-261/+160
| | | | | | | | | | | | | | Lambda makes it possible. Change-Id: I26a4df71dcd24b76a4f0d6d67545b2e1c6ba2412 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Remove GdbCommand::callbackNamehjk2015-02-056-27/+17
| | | | | | | | | | | | | | This was a debugging aid that has not been needed for a while. Change-Id: Ia79e8e91b6648bdb807836443b01dd325d819640 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Avoid some Gdb* in common type nameshjk2015-02-0514-275/+262
| | | | | | | | | | | | | | | | | | | | | | They had been created for use with GDB, but are more general nowadays. s/GdbResponse/DebuggerResponse/, s/GdbResultClass/ResultClass, etc Change-Id: Idfa3e280cdda882a8b3f02dff7e7d993e7e82d03 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Remove DebuggerCommand .arg chaining.hjk2015-02-053-49/+55
| | | | | | | | | | | | | | Less stylish options for more uniform user code. Change-Id: Ie9a6558101383a844efa25ba3ab59b4b824ad89f Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Enable 'native mixed' toogling also for LLDBhjk2015-02-053-4/+7
| | | | | | | | | | Change-Id: I50a026d6af2874aa78db98b0df1fcbe233c88b1b Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Fix broke slider connectionhjk2015-02-051-1/+1
| | | | | | | | | | Change-Id: Id9fd398e9f66494163d5a1e4d6f098044c2b9ff2 Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* | Debugger: Convert some connects to Qt 5 stylehjk2015-02-056-47/+52
| | | | | | | | | | Change-Id: Id0b374b4b0bd3d14fb73fae28269778db5ae3dc7 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Move QML stack access to DumperBasehjk2015-02-051-0/+6
| | | | | | | | | | | | | | ... and make it work with LLDB. Change-Id: Idaec029942dbcc726931781caff830173f7b126a Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Fix breakpoint property changes with LLDBhjk2015-02-051-0/+1
| | | | | | | | | | Change-Id: I68ee1b106e1cd6e8ad5ce99b226860c528698520 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Pass all breakpoint properties to *brigde.pyhjk2015-02-045-36/+23
| | | | | | | | | | Change-Id: I58ee1b106e1cd6e8ad5ce99b226860c528698520 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Native mixed fixeshjk2015-02-041-1/+1
| | | | | | | | | | Change-Id: I1b10b2f6a241c6061ac5ee77825aa8500cb7097f Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Remove unused codehjk2015-02-042-14/+0
| | | | | | | | | | Change-Id: Icce45f23abc026668531191e95c2e4d93d63fda7 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Implement native mixed breakpoints with LLDBhjk2015-02-041-5/+9
| | | | | | | | | | | | | | | | Breakpoints are hit, stack frames are identified as JS or native. No further data yet. Change-Id: I84a02422fd36dc7645003114dd8519bedd913c06 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Move some 'mixed native' code to DumperBasehjk2015-02-042-5/+17
| | | | | | | | | | | | | | Prepares reuse in LLDB. Change-Id: I2238a94937411b80a9ec84ab738aba2146bba50e Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Remove unneeded qualificationsOrgad Shaneh2015-02-0423-155/+152
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mostly done using the following ruby script: Dir.glob('**/*.cpp').each { |file| next if file =~ %r{src/shared/qbs|/qmljs/} s = File.read(file) s.scan(/^using namespace (.*);$/) { ns = $1 t = s.gsub(/^(.*)\b#{ns}::((?!Const)[A-Z])/) { |m| before = $1 char = $2 if before =~ /"|\/\/|\\|using|SIGNAL|SLOT|Q_/ m else before + char end } if t != s puts file File.open(file, 'w').write(t) end } } Change-Id: I1aa1a2b6ccbafeb1a8f3053fffa39b3f96992591 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Re-use base infrastructure for LLDB breakpoint handlinghjk2015-02-042-101/+87
| | | | | | | | | | Change-Id: If32b1f421e45dc4ee446e193e03c959d7c700948 Reviewed-by: hjk <hjk@theqtcompany.com>
* | Debugger: Move LldbEngine::Command to debuggerprotocolhjk2015-02-025-173/+175
| | | | | | | | | | | | | | Better re-usable over there. Change-Id: I75b70054d691771efaf5b19680d8a93d2943b5ba Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>
* | Debugger: Properly delete breakpoints when triggered from viewhjk2015-02-021-3/+1
| | | | | | | | | | | | | | | | | | If the removal is triggered from the view, the engine needs to be informed. If it's triggered from the engine, the view needs to be informed. Change-Id: I3505130c5b4b0b3752401a94599af9a06662f002 Reviewed-by: Christian Stenger <christian.stenger@theqtcompany.com>