Mark proxy finished *after* it sends the exit command, otherwise the exit is not send (patch by Martin Schovanek)

mkrauskopf · mkrauskopf · commit 184cf92170f8 · 2009-04-09T12:20:50.000Z
diff --git a/src/org/rubyforge/debugcommons/RubyDebuggerProxy.java b/src/org/rubyforge/debugcommons/RubyDebuggerProxy.java
@@ -447,7 +447,6 @@ public void finish(final boolean forced) {
                 // TBD rather detach
                 sendExit();
             }
-            finished = true;
             PROXIES.remove(RubyDebuggerProxy.this);
             if (forced) {
                 sendExit();
@@ -464,6 +463,7 @@ public void finish(final boolean forced) {
                     target.getProcess().destroy();
                 }
             }
+            finished = true;
         }
         fireDebugEvent(RubyDebugEvent.createTerminateEvent());
     }

Original file line number	Diff line number	Diff line change
`@@ -447,7 +447,6 @@ public void finish(final boolean forced) {`
`447`	`447`	`// TBD rather detach`
`448`	`448`	`sendExit();`
`449`	`449`	`}`
`450`		`- finished = true;`
`451`	`450`	`PROXIES.remove(RubyDebuggerProxy.this);`
`452`	`451`	`if (forced) {`
`453`	`452`	`sendExit();`
`@@ -464,6 +463,7 @@ public void finish(final boolean forced) {`
`464`	`463`	`target.getProcess().destroy();`
`465`	`464`	`}`
`466`	`465`	`}`
	`466`	`+ finished = true;`
`467`	`467`	`}`
`468`	`468`	`fireDebugEvent(RubyDebugEvent.createTerminateEvent());`
`469`	`469`	`}`