mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-24 01:15:29 +02:00
Code Issues Wiki Activity

[291754] Avoid race condition by waiting for the stopped event as soon as we have the launch

Browse source
This commit is contained in:
Marc Khouzam 2009-10-08 20:07:53 +00:00
parent f1088d2f5e
commit 91d786f13e
1 changed files with 12 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
dsf-gdb/org.eclipse.cdt.tests.dsf.gdb/src/org/eclipse/cdt/tests/dsf/gdb/framework/BaseTestCase.java
View file

@ -96,7 +96,18 @@ public class BaseTestCase {
fLaunch = (GdbLaunch)lc.launch(ILaunchManager.DEBUG_MODE, new NullProgressMonitor());
assert fLaunch != null;
try {
// Also wait for the program to stop before allowing tests to start
// This should be done as soon as we have the launch, to avoid missing the Stopped
// event. If we do miss it, we'll just have a 10 second delay.
final ServiceEventWaitor<MIStoppedEvent> eventWaitor =
new ServiceEventWaitor<MIStoppedEvent>(
fLaunch.getSession(),
MIStoppedEvent.class);
fInitialStoppedEvent = eventWaitor.waitForEvent(10000);
} catch (Exception e) {}
// If we started a gdbserver add it to the launch to make sure it is killed at the end
if (gdbserverProc != null) {
DebugPlugin.newProcess(fLaunch, gdbserverProc, "gdbserver");
@ -104,15 +115,6 @@ public class BaseTestCase {
// Now initialize our SyncUtility, since we have the launcher
SyncUtil.initialize(fLaunch.getSession());
try {
// Also wait for the program to stop before allowing tests to start
final ServiceEventWaitor<MIStoppedEvent> eventWaitor =
new ServiceEventWaitor<MIStoppedEvent>(
fLaunch.getSession(),
MIStoppedEvent.class);
fInitialStoppedEvent = eventWaitor.waitForEvent(10000);
} catch (Exception e) {}
}