[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [2032] trunk/org.rubypeople.rdt.debug.ui/src/org/ rubypeople/rdt/internal/debug/ui/actions/ModifyCatchpointAction.java

[<mba...@us...>]

Revision: 2032
          http://svn.sourceforge.net/rubyeclipse/?rev=2032&view=rev
Author:   mbarchfe
Date:     2007-02-25 07:48:39 -0800 (Sun, 25 Feb 2007)

Log Message:
-----------
fix setting/clearing of catch point

Modified Paths:
--------------
    trunk/org.rubypeople.rdt.debug.ui/src/org/rubypeople/rdt/internal/debug/ui/actions/ModifyCatchpointAction.java

Modified: trunk/org.rubypeople.rdt.debug.ui/src/org/rubypeople/rdt/internal/debug/ui/actions/ModifyCatchpointAction.java
===================================================================
--- trunk/org.rubypeople.rdt.debug.ui/src/org/rubypeople/rdt/internal/debug/ui/actions/ModifyCatchpointAction.java	2007-02-25 15:47:31 UTC (rev 2031)
+++ trunk/org.rubypeople.rdt.debug.ui/src/org/rubypeople/rdt/internal/debug/ui/actions/ModifyCatchpointAction.java	2007-02-25 15:48:39 UTC (rev 2032)
@@ -57,11 +57,11 @@
         try {
 			if (existingBreakpoint == null) {
 				IBreakpoint breakpoint = new RubyExceptionBreakpoint(dialog.getException()); 
-				DebugPlugin.getDefault().getBreakpointManager().addBreakpoint(breakpoint) ;
 			}
 			else {
+				// This will modify the Breakpoint marker as well which then triggers a
+				// ResourceChange event which will be delivered to the RubyDebugTarget
 				existingBreakpoint.setException(dialog.getException()) ;
-				DebugPlugin.getDefault().getBreakpointManager().fireBreakpointChanged(existingBreakpoint) ;
 			}
 		} catch (CoreException e) {
 			RdtDebugCorePlugin.log(e) ;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.