diff options
author | Marc Khouzam | 2011-03-11 15:06:55 +0000 |
---|---|---|
committer | Marc Khouzam | 2011-03-11 15:06:55 +0000 |
commit | ebb5a4dafdcbaf940d4ef56ac85aeb4b302be465 (patch) | |
tree | 6cb671b0734e809dd7a06b5586d8e1cbeeb8e6b2 /jtag | |
parent | aadf0c164a2b820a057cfb9a2a575bdaa6d23e7f (diff) | |
download | org.eclipse.cdt-ebb5a4dafdcbaf940d4ef56ac85aeb4b302be465.tar.gz org.eclipse.cdt-ebb5a4dafdcbaf940d4ef56ac85aeb4b302be465.tar.xz org.eclipse.cdt-ebb5a4dafdcbaf940d4ef56ac85aeb4b302be465.zip |
Bug 225805: Sourcelookup handling does not scale to large projects
Diffstat (limited to 'jtag')
-rw-r--r-- | jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java | 30 |
1 files changed, 18 insertions, 12 deletions
diff --git a/jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java b/jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java index 78127a5359f..029085f530d 100644 --- a/jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java +++ b/jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java @@ -340,19 +340,25 @@ public class GDBJtagDSFFinalLaunchSequence extends Sequence { requestMonitor.done(); } }}, - /* - * Setup the source paths - */ - new Step() { - @Override - public void execute(RequestMonitor requestMonitor) { - CSourceLookup sourceLookup = fTracker.getService(CSourceLookup.class); - CSourceLookupDirector locator = (CSourceLookupDirector)fLaunch.getSourceLocator(); - ISourceLookupDMContext sourceLookupDmc = (ISourceLookupDMContext)fCommandControl.getContext(); - sourceLookup.setSourceLookupPath(sourceLookupDmc, locator.getSourceContainers(), requestMonitor); - }}, - +// -environment-directory with a lot of paths could +// make setting breakpoint incredibly slow, which makes +// the debug session un-workable. We simply stop +// using it because it's usefulness is unclear. +// Bug 225805 +// /* +// * Setup the source paths +// */ +// new Step() { +// @Override +// public void execute(RequestMonitor requestMonitor) { +// CSourceLookup sourceLookup = fTracker.getService(CSourceLookup.class); +// CSourceLookupDirector locator = (CSourceLookupDirector)fLaunch.getSourceLocator(); +// ISourceLookupDMContext sourceLookupDmc = (ISourceLookupDMContext)fCommandControl.getContext(); +// +// sourceLookup.setSourceLookupPath(sourceLookupDmc, locator.getSourceContainers(), requestMonitor); +// }}, + // Below steps are specific to JTag hardware debugging /* |