having a go at quality issues on singleton enum Predicates

diff --git a/server/sonar-server/src/main/java/org/sonar/server/plugins/ServerPluginJarsInstaller.java b/server/sonar-server/src/main/java/org/sonar/server/plugins/ServerPluginJarsInstaller.java
index be8146b73f1c5da70e87ea5152d450f18eb9a5f3..cbed2cb36029a375ddba8be13d4570a0a74c30fc 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/plugins/ServerPluginJarsInstaller.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/plugins/ServerPluginJarsInstaller.java
@@ -33,7 +33,7 @@ import org.sonar.core.plugins.DefaultPluginMetadata;
 import org.sonar.server.platform.DefaultServerFileSystem;
 import org.sonar.updatecenter.common.PluginReferential;
 
-import javax.annotation.Nullable;
+import javax.annotation.Nonnull;
 import java.io.File;
 import java.io.IOException;
 import java.util.Arrays;
@@ -44,7 +44,6 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 
-import static com.google.common.base.Preconditions.checkNotNull;
 import static com.google.common.collect.Iterables.transform;
 import static com.google.common.collect.Lists.newArrayList;
 import static java.lang.String.format;
@@ -280,8 +279,8 @@ public class ServerPluginJarsInstaller {
     INSTANCE;
 
     @Override
-    public String apply(@Nullable File file) {
-      return checkNotNull(file).getName();
+    public String apply(@Nonnull File file) {
+      return file.getName();
     }
   }
 }

diff --git a/server/sonar-server/src/main/java/org/sonar/server/plugins/ws/PluginWSCommons.java b/server/sonar-server/src/main/java/org/sonar/server/plugins/ws/PluginWSCommons.java
index 4a81554765c7ed20555d8dd72bf5a5aca0c9c0c4..176f369e7270a05aa4d497caf084cb154742cdc0 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/plugins/ws/PluginWSCommons.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/plugins/ws/PluginWSCommons.java
@@ -24,7 +24,7 @@ import com.google.common.collect.Ordering;
 import org.sonar.api.platform.PluginMetadata;
 import org.sonar.api.utils.text.JsonWriter;
 
-import javax.annotation.Nullable;
+import javax.annotation.Nonnull;
 
 public class PluginWSCommons {
   static final String PROPERTY_KEY = "key";
@@ -83,10 +83,7 @@ public class PluginWSCommons {
     INSTANCE;
 
     @Override
-    public String apply(@Nullable PluginMetadata input) {
-      if (input == null) {
-        return null;
-      }
+    public String apply(@Nonnull PluginMetadata input) {
       return input.getName();
     }
   }
@@ -95,10 +92,7 @@ public class PluginWSCommons {
     INSTANCE;
 
     @Override
-    public String apply(@Nullable PluginMetadata input) {
-      if (input == null) {
-        return null;
-      }
+    public String apply(@Nonnull PluginMetadata input) {
       return input.getKey();
     }
   }

diff --git a/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java b/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
index ff90f7f127428d0420f2284eab51b24ca3a618f9..5af5f9c754f50a50afa744562a9d04b67b2dd56b 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
+++ b/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
@@ -25,13 +25,12 @@ import org.sonar.api.ServerComponent;
 import org.sonar.api.utils.SonarException;
 import org.sonar.updatecenter.common.PluginManifest;
 
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 import java.io.File;
 import java.io.IOException;
 import java.util.Arrays;
 
-import static com.google.common.base.Preconditions.checkNotNull;
-
 public abstract class PluginJarInstaller implements BatchComponent, ServerComponent {
 
   protected static final String FAIL_TO_INSTALL_PLUGIN = "Fail to install plugin: ";
@@ -95,14 +94,14 @@ public abstract class PluginJarInstaller implements BatchComponent, ServerCompon
 
   private final Function<File, DefaultPluginMetadata> jarFileToCorePlugin = new Function<File, DefaultPluginMetadata>() {
     @Override
-    public DefaultPluginMetadata apply(@Nullable File file) {
-      return extractMetadata(checkNotNull(file), true);
+    public DefaultPluginMetadata apply(@Nonnull File file) {
+      return extractMetadata(file, true);
     }
   };
   private final Function<File, DefaultPluginMetadata> jarFileToPlugin = new Function<File, DefaultPluginMetadata>() {
     @Override
-    public DefaultPluginMetadata apply(@Nullable File file) {
-      return extractMetadata(checkNotNull(file), false);
+    public DefaultPluginMetadata apply(@Nonnull File file) {
+      return extractMetadata(file, false);
     }
   };
 }