From e88ed4032e6d4cd4196e541a19315eb31d0ad5e3 Mon Sep 17 00:00:00 2001 From: Alexey Kudravtsev Date: Tue, 18 Apr 2017 18:15:51 +0300 Subject: [PATCH] cleanup --- .../intellij/ide/plugins/cl/PluginClassLoader.java | 13 +++++-------- .../util/src/com/intellij/openapi/util/Version.java | 8 +++----- 2 files changed, 8 insertions(+), 13 deletions(-) diff --git a/platform/core-impl/src/com/intellij/ide/plugins/cl/PluginClassLoader.java b/platform/core-impl/src/com/intellij/ide/plugins/cl/PluginClassLoader.java index 5aab969e8d8d..d965b2723079 100644 --- a/platform/core-impl/src/com/intellij/ide/plugins/cl/PluginClassLoader.java +++ b/platform/core-impl/src/com/intellij/ide/plugins/cl/PluginClassLoader.java @@ -100,7 +100,7 @@ public class PluginClassLoader extends UrlClassLoader { @Nullable private Class loadClassFromParents(final String name, Set visited) { for (ClassLoader parent : myParents) { - if (visited == null) visited = ContainerUtilRt.newHashSet(this); + if (visited == null) visited = ContainerUtilRt.newHashSet(this); if (!visited.add(parent)) { continue; } @@ -134,10 +134,7 @@ public class PluginClassLoader extends UrlClassLoader { try { c = _findClass(name); } - catch (IncompatibleClassChangeError e) { - throw new PluginException("While loading class " + name + ": " + e.getMessage(), e, myPluginId); - } - catch (UnsupportedClassVersionError e) { + catch (IncompatibleClassChangeError | UnsupportedClassVersionError e) { throw new PluginException("While loading class " + name + ": " + e.getMessage(), e, myPluginId); } if (c != null) { @@ -249,14 +246,14 @@ public class PluginClassLoader extends UrlClassLoader { @Override public String toString() { - return "PluginClassLoader[" + myPluginId + ", " + myPluginVersion + "]"; + return "PluginClassLoader[" + myPluginId + ", " + myPluginVersion + "] "+super.toString(); } private static class DeepEnumeration implements Enumeration { private final Enumeration[] myEnumerations; - private int myIndex = 0; + private int myIndex; - public DeepEnumeration(Enumeration[] enumerations) { + DeepEnumeration(@NotNull Enumeration[] enumerations) { myEnumerations = enumerations; } diff --git a/platform/util/src/com/intellij/openapi/util/Version.java b/platform/util/src/com/intellij/openapi/util/Version.java index 2a3328407252..28564b46c075 100644 --- a/platform/util/src/com/intellij/openapi/util/Version.java +++ b/platform/util/src/com/intellij/openapi/util/Version.java @@ -39,12 +39,12 @@ public class Version { return null; } - int minor = (versions.length > 1) ? parseNumber(versions[1], -1) : 0; + int minor = versions.length > 1 ? parseNumber(versions[1], -1) : 0; if (minor < 0) { return new Version(major, 0, 0); } - int patch = (versions.length > 2) ? parseNumber(versions[2], -1) : 0; + int patch = versions.length > 2 ? parseNumber(versions[2], -1) : 0; if (patch < 0) { return new Version(major, minor, 0); } @@ -105,9 +105,7 @@ public class Version { } public int compareTo(@Nullable Integer major, @Nullable Integer minor, @Nullable Integer bugfix) { - int result; - - result = doCompare(this.major, major); + int result = doCompare(this.major, major); if (result != 0) return result; result = doCompare(this.minor, minor);