mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-23 17:05:26 +02:00
Code Issues Wiki Activity

Bug 225805: Sourcelookup handling does not scale to large projects

Browse source
This commit is contained in:
Marc Khouzam 2011-03-11 15:06:55 +00:00
parent aadf0c164a
commit ebb5a4dafd
2 changed files with 27 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/gdb/launching/FinalLaunchSequence.java
View file

@ -89,7 +89,15 @@ public class FinalLaunchSequence extends ReflectionSequence {
"stepSetNonStop", //$NON-NLS-1$
"stepSetAutoLoadSharedLibrarySymbols", //$NON-NLS-1$
"stepSetSharedLibraryPaths", //$NON-NLS-1$
"stepSetSourceLookupPath", //$NON-NLS-1$
// -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
//
// "stepSetSourceLookupPath", //$NON-NLS-1$
// For post-mortem launch only
"stepSpecifyCoreFile", //$NON-NLS-1$
// For remote-attach launch only

30
jtag/org.eclipse.cdt.debug.gdbjtag.core/src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java
View file

@ -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
/*