From 9acd76c5b0af0f18e182db219ba58c3ec01ddd51 Mon Sep 17 00:00:00 2001
From: Pawel Piech <pawel.piech@windriver.com>
Date: Thu, 29 Apr 2010 22:38:17 +0000
Subject: [PATCH] Bug 311122 -  [breakpoints] NPE in AbstractBreakpointVMNode.

---
 .../ui/viewmodel/breakpoints/BreakpointVMProvider.java | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/breakpoints/BreakpointVMProvider.java b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/breakpoints/BreakpointVMProvider.java
index 8e1e248e3e0..f59b147c289 100644
--- a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/breakpoints/BreakpointVMProvider.java
+++ b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/breakpoints/BreakpointVMProvider.java
@@ -283,9 +283,8 @@ public class BreakpointVMProvider extends AbstractVMProvider
             getBreakpointOrganizerVMCs(
                 (BreakpointOrganizerVMNode)_nestingOrganizerVmc.getVMNode(), path.getParentPath(),
                 new DataRequestMonitor<List<BreakpointOrganizerVMContext>>(getExecutor(), rm) {
-                @Override
-                protected void handleCompleted() {
-                    if (isSuccess()) {
+                    @Override
+                    protected void handleSuccess() {
                         int newVmcIdx = getData().indexOf(_nestingOrganizerVmc);
                         
                         if (newVmcIdx >= 0) {
@@ -293,10 +292,9 @@ public class BreakpointVMProvider extends AbstractVMProvider
                         } else {
                             rm.setStatus(new Status(IStatus.ERROR, DsfUIPlugin.PLUGIN_ID, IDsfStatusConstants.INVALID_HANDLE, "Breakpoint category not found", null)); //$NON-NLS-1$
                         }
+                        rm.done();
                     }
-                    rm.done();
-                }
-            });
+                });
         }
     }