Bug 416056 - Updating from I20130820-0800 to I20130827-0800 restarted many times with exceptions
- add comment about 23 return code
diff --git a/tests/org.eclipse.core.tests.harness/src/org/eclipse/core/tests/session/SessionTestRunner.java b/tests/org.eclipse.core.tests.harness/src/org/eclipse/core/tests/session/SessionTestRunner.java
index e2554b7..bbd2873 100644
--- a/tests/org.eclipse.core.tests.harness/src/org/eclipse/core/tests/session/SessionTestRunner.java
+++ b/tests/org.eclipse.core.tests.harness/src/org/eclipse/core/tests/session/SessionTestRunner.java
@@ -269,7 +269,9 @@
try {
int returnCode = setup.run();
if (returnCode == 23) {
- // asked to restart; for now just do this once
+ // asked to restart; for now just do this once.
+ // Note that 23 is our magic return code indicating that a restart is required.
+ // This can happen for tests that update framework extensions which requires a restart.
returnCode = setup.run();
}
if (returnCode != 0)
