Fix the error case when contain the obsolete methods. (#147)

yaohaizh · web-flow · commit 7e082b5a7273 · 2018-01-22T15:52:02.000+08:00
diff --git a/com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/JavaHotCodeReplaceProvider.java b/com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/JavaHotCodeReplaceProvider.java
@@ -99,9 +99,9 @@ public class JavaHotCodeReplaceProvider implements IHotCodeReplaceProvider, IRes
 
     private PublishSubject<HotCodeReplaceEvent> eventSubject = PublishSubject.<HotCodeReplaceEvent>create();
 
-    private List<IResource> deltaResources = new ArrayList();
+    private List<IResource> deltaResources = new ArrayList<>();
 
-    private List<String> deltaClassNames = new ArrayList();
+    private List<String> deltaClassNames = new ArrayList<>();
 
     /**
      * Visitor for resource deltas.
@@ -296,7 +296,7 @@ public void onClassRedefined(Consumer<List<String>> consumer) {
     @Override
     public CompletableFuture<List<String>> redefineClasses() {
         return CompletableFuture.supplyAsync(() -> {
-            List<String> classNames = new ArrayList();
+            List<String> classNames = new ArrayList<>();
             synchronized (this) {
                 classNames.addAll(deltaClassNames);
                 doHotCodeReplace(deltaResources, deltaClassNames);
@@ -737,7 +737,7 @@ private boolean containsObsoleteMethods() throws DebugException {
             if (!thread.isSuspended()) {
                 continue;
             }
-            List<StackFrame> frames = getStackFrames(thread, false);
+            List<StackFrame> frames = getStackFrames(thread, true);
             if (frames == null || frames.isEmpty()) {
                 continue;
             }
