summaryrefslogtreecommitdiff
path: root/src/plugins/debugger
Commit message (Collapse)AuthorAgeFilesLines
* QmlEngine: Connect to port for remote setupAurindam Jana2012-06-122-3/+27
| | | | | | | | | | | | In the case of remote setup, the application output may take a while. The timeout is increased to 60s. Additionally in the case of mixed debugging, we check the state of the master engine to determine if we should keep waiting for the application output. Change-Id: I237a3501c1e70dbe62870929e50f12ede80e7c56 Reviewed-by: Thomas Hartmann <Thomas.Hartmann@nokia.com> Reviewed-by: Christian Kandeler <christian.kandeler@nokia.com>
* debugger: close source file of main event loop on debugger exithjk2012-06-121-8/+14
| | | | | | | | When opened by the debugger, but even if it is the "current". This hits the expected behaviour more often. Change-Id: Iaa488f350791473ecd4b10b43c90223411cf5217 Reviewed-by: hjk <qthjk@ovi.com>
* Debugging: RemotePlainGdbAdapter should not need remote setup.Christian Kandeler2012-06-111-4/+1
| | | | | Change-Id: I652c23efe83b2679b2adf0984d6fcd28cdb96b25 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: also accept external debug info as hint for a debug buildhjk2012-06-111-1/+4
| | | | | Change-Id: I97cc6cdf6b1f2b8f794ad166c0f2b3f8eadc98b8 Reviewed-by: Christian Kandeler <christian.kandeler@nokia.com>
* Debugger: resetting of newObjectsCreated done only for Qt5.xSimjees Abraham2012-06-111-1/+3
| | | | | | | | done to correct the regression introduced in 2574a5846983a3016e551432d24f8d8d4e3b97b5 Change-Id: I0d9fb8fabcd134f53f1a1c9d0fa04ca7b542a88c Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* Debugger: Adding objects to be watched correctedSimjees Abraham2012-06-111-0/+2
| | | | | | | | | Adding the objects to be watched corrected in case new objects were created. Task-number: QTCREATORBUG-7529 Change-Id: Ica5f852f47882242a0e97692c97c0ca4ee662a4e Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* debugger: read file type and machine type from binaryhjk2012-06-081-2/+11
| | | | | Change-Id: I3faa92ef8add807389ca3d87197e83df948f149a Reviewed-by: hjk <qthjk@ovi.com>
* debugger: more elf shufflinghjk2012-06-085-40/+89
| | | | | | | Also distinguish between debuglink and buildid. Change-Id: I1b55d1df42576c004050d319f46fb8c2ced9c85f Reviewed-by: hjk <qthjk@ovi.com>
* SSH: Streamline SshRemoteProcessRunner's output handling.Christian Kandeler2012-06-082-10/+8
| | | | | | | | | | Make it just like SshRemoteProcess (and QProcess). The current implementation annoyingly forces client code to establish additional signal/slot connections, even if they only want to evaluate the output at the end. Change-Id: Id8c30dd156574d7d26d848d8e0705856a16d3747 Reviewed-by: hjk <qthjk@ovi.com>
* SSH: Name SshRemoteProcess::ExitStatus values like the QProcess ones.Christian Kandeler2012-06-081-4/+4
| | | | | | | | (We cannot use QProcess::ExitStatus itself, because it does not contain FailedToStart.) Change-Id: I83ed6de621e2693e539562cd8e56ef478fa4fcb9 Reviewed-by: hjk <qthjk@ovi.com>
* Fix compilation.Christian Kandeler2012-06-081-1/+1
| | | | | | Change-Id: Iabaa6fd23ae3ab80c405a698e005eed6d5b4c9d1 Reviewed-by: Simjees Abraham <simjees.abraham@nokia.com> Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* QmlInspector: Update inspectorAurindam Jana2012-06-081-1/+4
| | | | | | | | | Update inpector when evaluating expression in inspector for mixed debugging. Change-Id: If2bad9d59b3ec3c0f2aa2a2d5f8a594701662bd0 Reviewed-by: Simjees Abraham <simjees.abraham@nokia.com> Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* Debugger: Fix automatic connection to QML when we don't get app outputKai Koehne2012-06-071-2/+1
| | | | | Change-Id: I34dff23401c34d3d0c940fac40a6d1d95e830ccb Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* debugger: handle new 'LoadRemoteCore' in switchhjk2012-06-061-0/+3
| | | | | Change-Id: I650d666a3d1eef23cbf34b0d7a3308666abcd42c Reviewed-by: hjk <qthjk@ovi.com>
* Windows compile fix for elfreaderJoerg Bornemann2012-06-061-2/+2
| | | | | Change-Id: Ie7c0d9d57cddc609fe621876cfc63e0c60b0fa39 Reviewed-by: hjk <qthjk@ovi.com>
* Debugger:Clearing the object tree corrected.Simjees Abraham2012-06-061-0/+1
| | | | | | | | newObjectsCreated reset while clearing the object tree. Task-number: QTCREATORBUG-7519 Change-Id: Ie5ee9bfd0faafb60f968a0c219b372eea6679761 Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* QmlDebugger: Edit breakpointsAurindam Jana2012-06-061-2/+3
| | | | | | | Remove and add breakpoint when a breakpoint is edited. Change-Id: Idb231c0e9d44fe295007b299660fcbe3a980a252 Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* Apply on Save: Infobar displayed cleared when debugging endsSimjees Abraham2012-06-062-0/+6
| | | | | | | | | Infobar displayed cleared if the user decides to end debugging without reloading Qml. Change-Id: Ic502c9b75a4f4e41aa3e843706f9e1fc0113f0e7 Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com> Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* debugger: rework core adapterhjk2012-06-069-100/+40
| | | | | | | | The name of the binary is now read using the elfreader instead of attaching gdb twice. Change-Id: I7a75bc64278c4f43e7085d2e479b53173602c966 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: add Load Remote Corehjk2012-06-058-7/+391
| | | | | Change-Id: I3ea5ddd608ff9e6764a995dc042466d436ca8474 Reviewed-by: hjk <qthjk@ovi.com>
* Debugger: Fetch object done only if parent Iname is known.Simjees Abraham2012-06-051-1/+6
| | | | | | | | When an object is created, the properties are fetched and added to the object tree only if the parent Iname is known. Change-Id: Id49a1873e9ffaf33689fe11297341cf56472c1be Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* Add ActionContainer::addSeparator(...)Eike Ziller2012-06-051-34/+8
| | | | | | | Gets rid of a lot of code duplication. Change-Id: I2ce38fb38a0b61cb821e0bdc7bcc9a7ccdf9da72 Reviewed-by: hjk <qthjk@ovi.com>
* Debugger: Prompt for reload if set/reset binding fails.Simjees Abraham2012-06-054-3/+24
| | | | | | | | | | QtCreator takes into account the result of message from qtdeclarative for setting, resetting and updating the method body. Task-number: QTCREATORBUG-3264 Change-Id: I15d1a33990175e86bb3f2fee08f75dd5b7b1b628 Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* Debugger:Correctly added the object to be watched.Simjees Abraham2012-06-051-3/+0
| | | | | | | | | Correction done for the bug introduced with commit fc1f265e874c1c2ba6828885dbd12339b2165650. Task-number: QTCREATORBUG-7498 Change-Id: I322a564208d70c7778a208fe0d9360ce47b75b81 Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* debugger: make startGdb response handling asyncronoushjk2012-06-0517-36/+100
| | | | | | | | This allows adapters to take all responses from the startup commands into account. Change-Id: I295cb211a4b69cfb8c59b34030aaee8120ffe98e Reviewed-by: hjk <qthjk@ovi.com>
* debugger: reshuffle sections in module datahjk2012-06-054-75/+36
| | | | | | | | Keep track of elf section headers, add a dumper for Utils::ElfSection etc. Change-Id: I06d01c0de01ffc6b827a4a79bdc91fdb3e761d75 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: make elfreader also report offset and size of sectionshjk2012-06-051-20/+33
| | | | | Change-Id: I8dd3d20ddc4652dc490d87558f46474f79300389 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: add watchpoint option to memoryview context menuhjk2012-06-042-0/+13
| | | | | Change-Id: I9511e15afcfe4664ae5077dd568e5a3637888e30 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: move watchpoint convenience functions to breakhandlerhjk2012-06-044-32/+34
| | | | | Change-Id: I7f214ff5fe6d996c4dd14c6ac181a482ad00c94e Reviewed-by: hjk <qthjk@ovi.com>
* debugger: more modulehandler polishinghjk2012-06-045-48/+121
| | | | | Change-Id: I2382a597783512b566118cccf772ff530cdf9026 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: small rework of the modules handlerhjk2012-06-013-90/+67
| | | | | | | Use new elfreader, also slimmer interface. Change-Id: Ie6387ff14bdefd109768d1ce4c395a17211da77d Reviewed-by: hjk <qthjk@ovi.com>
* debugger: use the elfreader instead of external objdumphjk2012-06-016-49/+79
| | | | | Change-Id: Ic5c5525703a6ef8924ac68e1a9ed33e411aada08 Reviewed-by: hjk <qthjk@ovi.com>
* Debugger: Remove 'Change Global Display Formats...'Kai Koehne2012-06-011-7/+4
| | | | | | | | | Remove 'Change Global Display Formats...' from the Locals & Expressions context menu. The dialog only works right now if one is currently debugging, changes are not stored across runs ... Change-Id: I7d16a7f8eaa9990e5fe1d7148022aa3db2c0fd05 Reviewed-by: hjk <qthjk@ovi.com>
* Debugger: Fix compile on Windows.Friedemann Kleint2012-05-311-5/+5
| | | | | Change-Id: I4994fdfd260d80eec8a0bfda903c553d2d563706 Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
* QmlDebugging: Save few cyclesAurindam Jana2012-05-312-20/+26
| | | | | Change-Id: I6d094605e3983128b844f2156909b98a4016509f Reviewed-by: hjk <qthjk@ovi.com>
* debugger: work on watchmodel bulk inserthjk2012-05-313-180/+69
| | | | | Change-Id: Ib92c8b02bc8e608810b3fa0f2d70d6547e428933 Reviewed-by: hjk <qthjk@ovi.com>
* debugger: save a few cycles in the watchmodelhjk2012-05-311-11/+27
| | | | | Change-Id: I56afd567361b2a65db4e0103a64540dc4305981c Reviewed-by: hjk <qthjk@ovi.com>
* QmlJSInspector: Check for null pointersAurindam Jana2012-05-302-20/+29
| | | | | Change-Id: I8d99b109909e596ead2e674eff0de576b2dca411 Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* QmlJSInspector: Improve performanceAurindam Jana2012-05-302-87/+118
| | | | | | | | Populating watchhandler can be expensive. Update with just the necessary data. Change-Id: I06c28e1c8f5437cc05d3f9cef139d3df9da10c81 Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* debugger: more precise matching for connection error messageshjk2012-05-301-10/+23
| | | | | Change-Id: Icd1821b4ded70242d8a7e09463461d340f277f6b Reviewed-by: hjk <qthjk@ovi.com>
* debugger: rework dumping arrays of plain datahjk2012-05-302-35/+100
| | | | | | | | | | | | Simple array data can be fetched in one go. The "normal" way gdb asks the server for contents is one item at a time, at ~3ms per round trip. Now we get 2000 in 200ms. This also introduces a re-usable readRawMemory function and removes a few checkAccess calls which will be triggered anyways. Change-Id: Ic07a3d6593fd2ea45f7a8058509118fe22a845bb Reviewed-by: hjk <qthjk@ovi.com>
* SSH: Use plain pointers to SshConnection objects.Christian Kandeler2012-05-292-7/+6
| | | | | | | | | | It used to be shared pointers so that existing connection objects could easily be passed around in order not to open a new connection to the same server. Since the introduction of the SshConnectionManager, this is no longer necessary. Change-Id: I13fd3eceaf35d562e6260e9969abbffb01edd6b5 Reviewed-by: Tobias Hunger <tobias.hunger@nokia.com>
* QmlJSInspector: Remove dependency on QmlJSToolsAurindam Jana2012-05-297-74/+51
| | | | | | | | | As Location information is already cached, there is no need to fetch the information from the QmlJS Semantic info. Task-number: QTCREATORBUG-7409 Change-Id: I9cc3dce0910632cfe517f7697d74691e6675793b Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>
* debugger: less scary crash messagehjk2012-05-291-2/+2
| | | | | Change-Id: I2c24d3a14cf2139a363100733280c4fd0c321524 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@nokia.com>
* debugger: code style, add missing spacehjk2012-05-291-1/+1
| | | | | Change-Id: I58ab4769dc26bea7234497660acf2d88c800fe46 Reviewed-by: hjk <qthjk@ovi.com>
* Debugging: Fix debugging for gdb versions < 7Aurindam Jana2012-05-291-5/+2
| | | | | | | | Revert changes in e11a3a7697e7b432ac061277694885cf3724f25c related to -file-exec-and-symbols command. Change-Id: I9d637fe931a6be2eaeb3d18ddcdf66e0ba4211dd Reviewed-by: hjk <qthjk@ovi.com>
* Debugger: Engines Queried on Reload of Qml.Simjees Abraham2012-05-292-1/+2
| | | | | Change-Id: I75de9fa584af8eb22a3eac6b0ab9d30b87d2ed69 Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* Debugger: Change in response for message "destroyObject".Simjees Abraham2012-05-294-4/+23
| | | | | | | | | Response for "destroyObject" now has the debugId of the destroyed object. This is used to fetch the parent object rather than querying the engine context. Change-Id: I2e5b7650cbe847117cce6952206dd479ee696f0e Reviewed-by: Aurindam Jana <aurindam.jana@nokia.com>
* QmlJSInspector: Set property valuesAurindam Jana2012-05-254-8/+56
| | | | | | | | | | | QML property values could be set in the inspector. Regression introduced by 7f09d0b756ff3f9bb52737d4aaf65ed84d751316. This patch is a partial fix. Only values can be assigned currently. For expressions, use the console. Change-Id: I945b11109cd7788dd0f5277af206bf9658844aee Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com> Reviewed-by: hjk <qthjk@ovi.com>
* QmlJSInspectorAgent: Refactor codeAurindam Jana2012-05-255-410/+254
| | | | | | | | Try to use the watchHandler for referring to the object tree instead of caching the object tree locally. Change-Id: I68624771ef49d50f2cf0d6d580b1b34d57483178 Reviewed-by: Christiaan Janssen <christiaan.janssen@nokia.com>