Commit Graph

1272 Commits

Author SHA1 Message Date
hjk
4497d136ce Debugger: Leave GDB default setting for 'set overload resolution'
... and work around the pecularities on the dumper side instead.

Change-Id: I0c2703aae13dabc0f3f45efb46bd556c8741a15f
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-10-22 19:11:02 +02:00
Friedemann Kleint
50f522b2f7 Remove most trailing newlines from translated messages.
They are a hassle for translators and reviewers alike.

Change-Id: I07c1b61e8b6719e54fdc1f69cf63f573119a6776
Reviewed-by: hjk <hjk121@nokiamail.com>
Reviewed-by: Robert Loehning <robert.loehning@digia.com>
Reviewed-by: Eike Ziller <eike.ziller@digia.com>
2013-10-22 08:05:09 +02:00
hjk
1650aaae38 Debugger: Fold GdbToolTipContext into DebuggerToolTipIndex
Change-Id: Iee9201daa8d42ee4231895b1c78387acbc7caaac
Reviewed-by: David Schulz <david.schulz@digia.com>
2013-10-14 15:42:43 +02:00
hjk
30045bee78 Debugger: Simplify Tooltip manager interface
The Editor itself is not really used.

Task-number: QTCREATORBUG-10358
Change-Id: I94620ed7b3b1d6b02c4eef2a359f991217b0a00b
Reviewed-by: David Schulz <david.schulz@digia.com>
2013-10-14 09:17:23 +02:00
Nikolai Kosjar
b8dbac0b9c Rename "[Mm]ethod(s)" to "[Ff]unction(s)"
Only methods as programming functions are affected. Besides renaming
some actions like "Switch Between Function Declaration/Definition" this
mostly touches (api) code comments.

This is a follow-up patch to commit 872bfb7.

Change-Id: Icb65e8d73b59a022f8885b14df497169543a3b92
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-10-10 15:56:12 +02:00
Eike Ziller
63ff745213 Merge remote-tracking branch 'origin/2.8'
Conflicts:
	src/libs/extensionsystem/pluginmanager.cpp
	src/libs/extensionsystem/pluginspec.cpp
	src/plugins/coreplugin/basefilewizard.cpp
	src/plugins/coreplugin/coreplugin.pro
	src/plugins/coreplugin/coreplugin.qbs
	src/plugins/coreplugin/editormanager/ieditor.cpp
	src/plugins/coreplugin/externaltool.cpp
	src/plugins/coreplugin/icore.cpp
	src/plugins/cpptools/cppmodelmanager.cpp
	src/plugins/fakevim/fakevimhandler.cpp
	src/plugins/find/ifindfilter.cpp
	src/plugins/projectexplorer/buildstep.cpp
	src/plugins/projectexplorer/devicesupport/idevice.cpp
	src/plugins/projectexplorer/runconfiguration.cpp
	src/plugins/vcsbase/vcsbaseeditor.cpp
	tests/system/suite_debugger/tst_simple_analyze/test.py

Change-Id: I11dc9e60bfc14bad4f8af747d041fc7678a07c17
2013-09-27 10:44:32 +02:00
hjk
c372f7b601 Rename share/dumper to share/debugger and {l,g}bridge to {lldb,gdb}bridge
Change-Id: I56a09dd89d2ac086ee264432510fd39b5efe69c5
Reviewed-by: Christian Stenger <christian.stenger@digia.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-09-26 21:50:40 +02:00
Tim Sander
3fbeca21d2 add baremetal plugin
This patch adds the baremetal plugin. The baremetal plugin is for
debugging small targets with hardware debugger where the remote linux
plugin does not fit. It adds a new kit and device class of type
baremetal. The device allows entering custom gdb commands. Currently only
qmake builds are possible. To use this with smaller devices without qt
a fake-qt installation is needed. CMake and qbs is currently not
supported.

Change-Id: I86816f897a2e9ed8b95e3184387969cedf8a14d7
Reviewed-by: hjk <hjk121@nokiamail.com>
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
2013-09-20 14:24:32 +02:00
hjk
d3d66bb39d Debugger: Handle manually set catch breakpoint gracefully
Task-number: QTCREATORBUG-10096
Change-Id: Ia9458840247ae1815751e3c58e1f21990af0cc34
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-09-19 18:00:15 +02:00
hjk
7971b6e714 Debugger: Make dumpers a bit more Python 3.x friendly
Looks like Ubuntu 13.10 will ship GDB linked to Python 3.3.

Change-Id: I748e8461531a3554d27ebfbf7af33385f69586c0
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-09-19 12:09:52 +02:00
hjk
994075dde1 Debugger: Merge {Abstract,Local}GdbProcess to GdbProcess
Change-Id: I46e8b4347205791eeb6ebd3ebbde9a993d892e93
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-09-10 16:59:21 +02:00
hjk
8bffed4d10 Debugger: Merge {Abstract,Local}PlainAdapter to GdbPlainEngine
Change-Id: Idb2197e429df2fb166f042649d3e95e88c28203e
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-09-10 16:36:15 +02:00
hjk
ed1483741e Debugger: Remove RemoteGdbProcess and RemotePlainGdbAdapter
Should be superseded by using a LocalAdapter variant with
(remote) DeviceProcess.

Change-Id: I9834105ff677bfe5ca527778e9795efc3b677393
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
2013-09-10 16:35:55 +02:00
Leena Miettinen
2b573a6d82 Doc: edit debugger API docs
Remove \brief commands from function descriptions.
Use QDoc commands for notes and lists.
Write GDB in all caps.
Fix punctuation and style and grammar issues.

Change-Id: I7f5bf0f53ad643eb4e2981c0d4e39c453dff5558
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
2013-09-09 12:55:15 +02:00
Christian Kandeler
72d173829c Remove superfluous include paths from project files.
A lot of our build system files specify unneeded include
paths. These roughly fall into the following categories:
    a) Paths that are already set in more general files
       such as qtcreator.pri.
    b) Paths that serve no purpose at all, possibly
       left over from earlier versions of the project.
    c) Paths that act as workarounds for wrong include
       statements of the form '#include "xyz.h"', where
       xyz.h is not in the same directory as the including
      file.
This patch removes such path specifications and fixes the offending
include statements from case c).
Tested on Linux, Windows and OSX with qmake and qbs.

Change-Id: I039a8449f8a65df0d616b4c08081145c18ae4b15
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
2013-09-02 17:28:17 +02:00
hjk
de1ec22424 Debugger: Don't enable gdb remote cache by default
The reasons for the always using this are history.

Change-Id: If2b888eeea2af37fe56bdcb934b525e87976acbc
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
2013-08-30 16:28:31 +02:00
hjk
99f22de6ba TaskHub: Add convenience function to add a task and clean up users
Change-Id: I5c76f8af720092d4e47b1a9fa889fb3a7010f21f
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
2013-08-30 12:44:10 +02:00
Oswald Buddenhagen
c2e90c360b Merge remote-tracking branch 'origin/2.8'
Conflicts:
	src/plugins/cpptools/cppcompletion_test.cpp
	src/plugins/cpptools/cpptoolsplugin.h
	src/plugins/projectexplorer/customtoolchain.cpp
	src/plugins/vcsbase/command.cpp

Change-Id: Ie7b3c9e136c0748b41320227c757471259339b48
2013-08-21 14:39:18 +02:00
Nicolas Arnaud-Cormos
14d05eade8 Fix compilation errors
Change-Id: I6b0d4956b5dcbd13ee26c72081f77e5d8a4f219b
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
2013-08-19 12:47:59 +02:00
hjk
ad9c51a486 Debugger: Make Creator work with a GDB using Python 3.x
Even if current GDB git does not compile in that configuration,
some people seem to be able to get this up and running, leading
to dumper load failures and fallback to the no-more-existing
compiled dumpers.

Change-Id: I08ba6c43105c28b3406fce36049bfdf633e79e31
Reviewed-by: Kai Koehne <kai.koehne@digia.com>
Reviewed-by: David Schulz <david.schulz@digia.com>
2013-08-16 14:13:34 +02:00
Orgad Shaneh
fe4b6feeb3 Debugger: Pass data-directory to gdb on Windows
Required for gdb 7.6

Change-Id: I5d02a6b712fbd5c3f70ac4c83871b03197f95696
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-06-19 17:52:21 +02:00
hjk
e40308ab1b Debugger: Remove in-plugin dumper autotest handling
It's covered by tst_dumper nowadays.

Change-Id: Id6b91e5a4ad2ac2472d598c4f0ba4c95c334af6d
Reviewed-by: Christian Stenger <christian.stenger@digia.com>
2013-06-07 15:48:43 +02:00
hjk
cbcd0bee3f Debugger: Suppress GDB's memory-changed notification
Change-Id: I266b1cef33e53250da79fac0473c3d45eb740ddb
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-06-06 18:37:40 +02:00
hjk
c2d6081a6c Debugger: Don't check for existence of gdb.inferior.threads()
All supported versions have it in that context.

Change-Id: Iafff4760ba521ffc9abc4e05dbbc9263ed6fe21a
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
2013-06-06 14:06:16 +02:00
Orgad Shaneh
81fc8ff210 Doc: Remove \fn lines adjacent to functions
They are unneeded, some of them are unmaintained

Change-Id: I0ad87e152d8e2328b486b2f904f7cf9613004afe
Reviewed-by: Eike Ziller <eike.ziller@digia.com>
2013-06-05 10:07:14 +02:00
hjk
c7ea186df9 Debugger: Always check for availability of python with GDB
Trying to avoid that round-trip on versions that are known to
not have it puts us on the mercy of detecting the version,
which is unreliable by nature, and bad in practice, as proven
by the recent MinGW builds.

Change-Id: I122eb03fe0ddcb193ed8d45e672564f7113b662f
Reviewed-by: Ray Donnelly <mingw.android@gmail.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-06-04 08:29:55 +02:00
hjk
76fc972277 Debugger: Add specific customization hook for dumper initialization
We need to distiguish between start of GDB itself (a.k.a .gdbinit-style
customization) and after the dumper machinery is initialized, which is
nowadays often delayed until the first stop hook.

Change-Id: I40f1e7225c2043b8bcb7d50eef948bb3c9162bb6
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-06-03 17:27:39 +02:00
hjk
db4a8caf32 Debugger: Move warning of insufficent python version
It was in exactly the wrong branch.

Change-Id: I0679c18dc9c049bb89c2cba47cb0762ad81e3024
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-06-03 12:38:52 +02:00
hjk
38e98e60d5 Debugger: Fix typo when checking versions
Change-Id: Ic24e7f26228b4f1bc1e7ab3627e86036396584ed
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-28 13:12:16 +02:00
hjk
22c5d99fcb Debugger: Disable some commands on Mac
Version of gdb < 7.4 means we are on Mac, so we can
disable commands that have known results there.

Change-Id: I8fc97ed61c47af2c3d9e5cc2bf83e97661204d4f
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-28 12:38:10 +02:00
hjk
af69f732f1 Debugger: Hard-code version dependency of m_dissembleUsesComma
We don't support GDB in the range 7.0...7.3 anymore where the
outcome was unsure.

Change-Id: I1ce76f5c4b1febe45c4a3b6404208fed7d9c6868
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-28 12:37:27 +02:00
hjk
91ba674db4 Debugger: Remove workaround for gdb-7.1 bug
See sourceware.org/bugzilla/show_bug.cgi?id=12023.
The workaround can cause hickups in the output parser
of Mac gdb, as the error message is scattered within
regular output without any markup.

Change-Id: I835133e390403a0f8f6aa573d6f9c63cf0784187
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-28 12:37:02 +02:00
hjk
0a9665077a Debugger: Restrict use of system GDB pretty printer to version 7.4
Reduce compatibility issues e.g. on Mac. The previous versions
are not working well anyway.

Change-Id: If6cc7c1d4a85300d9bf054966216b52b35c0d1f2
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-26 15:31:02 +02:00
Eike Ziller
d8bd56bcd6 Merge remote-tracking branch 'origin/2.7' into 2.8
Conflicts:
	src/plugins/cpptools/cppcompletion_test.cpp
	src/plugins/cpptools/cpptoolsplugin.h
	src/plugins/qbsprojectmanager/qbsbuildstep.cpp
	src/plugins/qbsprojectmanager/qbscleanstep.cpp
	src/plugins/qbsprojectmanager/qbsnodes.cpp
	src/shared/qbs

Change-Id: I8fc97ed61c47af2c3d9e5cc2bf81e97661204d4e
2013-05-24 16:20:10 +02:00
Kai Koehne
0a471cc4fd Set breakpoints only by filename (again)
This essentially reverts 3aa0e50039 , which causes trouble
e.g. for network paths and other corner cases.

Change-Id: I12ceb5fbc68b7a3fc8010165acc910acd2d95e55
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-24 12:48:09 +02:00
Orgad Shaneh
05ef32f242 Remove Nokia leftovers
Mostly internal mimetypes

Change-Id: Ia84206f1334c1f83474cbc307e848719bb98adbc
Reviewed-by: Fawzi Mohamed <fawzi.mohamed@digia.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
2013-05-24 12:29:23 +02:00
hjk
4786cee969 Debugger: Issue "threadnames" command again
Was accidentally disabled in 066efcd6a6.

Change-Id: I64de0609a254aa2914142a4f75105b4140b6ae5b
Reviewed-by: Eike Ziller <eike.ziller@digia.com>
2013-05-23 15:07:21 +02:00
Friedemann Kleint
c6bee7e20e Debugger/GDB: Use short paths for breakpoints on UNIX.
Breakpoints by path are not hit if the path traverses
symbolic links.

Change-Id: I3ab5d1bead8333986444aec709c940840dfddbb4
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-05-15 10:20:33 +02:00
hjk
fa2ac44271 Debugger: Add a convenience function GdbMi::toInt()
Saves some line noise on the user side.

Change-Id: I9c6d7542195fc7485ac4ee466cd42a28be95b8da
Reviewed-by: David Schulz <david.schulz@digia.com>
2013-05-07 13:24:48 +02:00
hjk
9b8b8ba97b Debugger: Replace GdbMi::findChild() with an operator[]
Less noise.

Change-Id: I8e533c97207ff5b9c79182c4fb99993f1992154f
Reviewed-by: David Schulz <david.schulz@digia.com>
2013-05-06 14:41:08 +02:00
Kai Koehne
3aa0e50039 Debugger: Do not set absolute breakpoints for remote targets
Fixes a regression introduced by cb857b47a

Task-number: QTCREATORBUG-9226
Change-Id: I619df0eb8b5f016aac54a0260eb941fd91a348aa
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-30 11:02:42 +02:00
Kai Koehne
cb857b47a2 Debugger: Change default gdb engine breakpoint "path usage" to "full"
Setting breakpoints by file name and line does not work robustly when
using gdb, with neither the full path, nor with just the baseline. Since
typically one of the options work, Creator offers a per-breakpoint choice
of which approach to use. Traditionally, using the full path broke more
often, so the default was "short". Lately (gdb 7.3+) using the full path
seems to be more robust, so the default should be changed. Manual
overriding to "short" is still possible.

Change-Id: I9e857c86a63964bdacf9bebc5444ea752e5974f8
Reviewed-by: hjk <hjk121@nokiamail.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
2013-04-25 18:08:22 +02:00
hjk
ed890bc940 Debugger: Move dockwidget object names into a more private place
Change-Id: I5a3565179fdd8627387ba01370e6484d668e2160
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-23 17:36:36 +02:00
hjk
4d6ce81e0c Debugger: Don't scan directories that do not exist.
Change-Id: Ie79249039643041fd68da6766b0e6f3ca6080b86
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-19 15:26:43 +02:00
hjk
5b98167ebd Debugger: Remove common bridge.py
The engines know what they want, no need to guess again.

Change-Id: Ie21b0e08eff8b81b217a08d4c47c35c35ae831d7
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-11 18:17:21 +02:00
hjk
17e8631dfb Debugger: Reorganize python dumper loading
- Split off GDB and LLDB specific parts into separate files,
- Trigger loading of dumpers.py and qttypes.py from bridge.py
- Read start up script at startup, not as part of dumper loading

Change-Id: I7941ee535121fa0f43a466e5bb75a18c9bb19764
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-10 15:58:12 +02:00
hjk
87f2d83ec2 Debugger: Add basic breakpoint handling and stepping to LLDB backend
Change-Id: Ib700afa63739e6d26bdd97225265559d7112eadb
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-10 15:58:02 +02:00
Eike Ziller
65654e4a50 Merge remote-tracking branch 'origin/2.7'
Conflicts:
	src/plugins/android/androidrunner.cpp
	src/plugins/clearcase/clearcasesync.cpp
	src/plugins/fakevim/fakevimhandler.cpp
	tests/auto/ioutils/ioutils.pro

Change-Id: I31587b8a4dd6aacc1e76803159da51a972878370
2013-04-08 13:26:37 +02:00
hjk
066efcd6a6 Debugging on Android
This implements the host side of
https://codereview.qt-project.org/#change,50290

Change-Id: I13c7df29534a2a85202c2b295b139896443b0120
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@digia.com>
Reviewed-by: Daniel Teske <daniel.teske@digia.com>
Reviewed-by: BogDan Vatra <bogdan@kde.org>
Reviewed-by: hjk <hjk121@nokiamail.com>
2013-04-08 12:05:44 +02:00
hjk
6bb0024140 Debugger: Make adding tasks for missing debug info packages optional
Task-number: QTCREATORBUG-8949
Change-Id: If0cf1c1078a72a3e4a3d371d236bacac858b8046
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
2013-04-02 13:51:25 +02:00