refactoring to avoid red and possibly not compilable code clion/146.25 webstorm/146.24
authorEgor.Ushakov <egor.ushakov@jetbrains.com>
Thu, 18 Feb 2016 14:36:20 +0000 (17:36 +0300)
committerEgor.Ushakov <egor.ushakov@jetbrains.com>
Thu, 18 Feb 2016 14:36:51 +0000 (17:36 +0300)
java/debugger/impl/src/com/intellij/debugger/ui/breakpoints/BreakpointManager.java
platform/xdebugger-api/src/com/intellij/xdebugger/XDebuggerUtil.java
platform/xdebugger-impl/src/com/intellij/xdebugger/impl/XDebuggerUtilImpl.java

index e545dbc30301547caeb9ce085f68fb68ff8d786c..9e76d0157ec89c08864785ea57571dc5d1459e3f 100644 (file)
@@ -178,7 +178,7 @@ public class BreakpointManager {
       typeCls = JavaExceptionBreakpointType.class;
     }
     if (typeCls != null) {
-      XBreakpointType<XBreakpoint<?>, ?> type = XDebuggerUtil.getInstance().findBreakpointType(typeCls);
+      XBreakpointType type = XDebuggerUtil.getInstance().findBreakpointType(typeCls);
       ((XBreakpointManagerImpl)getXBreakpointManager()).getBreakpointDefaults(type).setSuspendPolicy(Breakpoint.transformSuspendPolicy(defaults.getSuspendPolicy()));
     }
   }
@@ -241,7 +241,7 @@ public class BreakpointManager {
   @NotNull
   public ExceptionBreakpoint addExceptionBreakpoint(@NotNull final String exceptionClassName, final String packageName) {
     ApplicationManager.getApplication().assertIsDispatchThread();
-    final JavaExceptionBreakpointType type = (JavaExceptionBreakpointType)XDebuggerUtil.getInstance().findBreakpointType(JavaExceptionBreakpointType.class);
+    final JavaExceptionBreakpointType type = XDebuggerUtil.getInstance().findBreakpointType(JavaExceptionBreakpointType.class);
     return ApplicationManager.getApplication().runWriteAction(new Computable<ExceptionBreakpoint>() {
       @Override
       public ExceptionBreakpoint compute() {
@@ -370,7 +370,7 @@ public class BreakpointManager {
               final Element breakpointElement = group.getChild("breakpoint");
               if (breakpointElement != null) {
                 XBreakpointManager manager = XDebuggerManager.getInstance(myProject).getBreakpointManager();
-                JavaExceptionBreakpointType type = (JavaExceptionBreakpointType)XDebuggerUtil.getInstance().findBreakpointType(JavaExceptionBreakpointType.class);
+                JavaExceptionBreakpointType type = XDebuggerUtil.getInstance().findBreakpointType(JavaExceptionBreakpointType.class);
                 XBreakpoint<JavaExceptionBreakpointProperties> xBreakpoint = manager.getDefaultBreakpoint(type);
                 Breakpoint breakpoint = getJavaBreakpoint(xBreakpoint);
                 if (breakpoint != null) {
index 5bf5b68344f42a1809196a9fe52b14a70403fd99..f290b12a65b076276bb4087590e98477c42517c9 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2015 JetBrains s.r.o.
+ * Copyright 2000-2016 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -70,7 +70,7 @@ public abstract class XDebuggerUtil {
 
   public abstract void removeBreakpoint(Project project, XBreakpoint<?> breakpoint);
 
-  public abstract <B extends XBreakpoint<?>> XBreakpointType<B, ?> findBreakpointType(@NotNull Class<? extends XBreakpointType<B, ?>> typeClass);
+  public abstract <T extends XBreakpointType> T findBreakpointType(@NotNull Class<T> typeClass);
 
   /**
    * Create {@link XSourcePosition} instance by line number
index dda2d745a3ddbba3f30e8bb70e48c33148db2c2f..968774ded3aa5ab0c2f7ff1c38513247621f7a35 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2015 JetBrains s.r.o.
+ * Copyright 2000-2016 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -282,7 +282,7 @@ public class XDebuggerUtilImpl extends XDebuggerUtil {
   }
 
   @Override
-  public <B extends XBreakpoint<?>> XBreakpointType<B, ?> findBreakpointType(@NotNull Class<? extends XBreakpointType<B, ?>> typeClass) {
+  public <T extends XBreakpointType> T findBreakpointType(@NotNull Class<T> typeClass) {
     if (myBreakpointTypeByClass == null) {
       myBreakpointTypeByClass = new THashMap<Class<? extends XBreakpointType>, XBreakpointType<?, ?>>();
       for (XBreakpointType<?, ?> breakpointType : XBreakpointUtil.getBreakpointTypes()) {
@@ -291,7 +291,7 @@ public class XDebuggerUtilImpl extends XDebuggerUtil {
     }
     XBreakpointType<?, ?> type = myBreakpointTypeByClass.get(typeClass);
     //noinspection unchecked
-    return (XBreakpointType<B, ?>)type;
+    return (T)type;
   }
 
   @Override